kae/musicmuster
1
0
Fork 0
Code Issues 28 Pull Requests Projects Releases 1 Activity

Warn when trying to delete playing or next track

Browse Source
This commit is contained in:
Keith Edmunds 2021-08-15 11:17:09 +01:00
parent 2c9f041838
commit d4f542cc29
2 changed files with 6 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
app/helpers.py
View File

@ -37,7 +37,7 @@ def get_relative_date(past_date, reference_date=None):
return f"{weeks} {weeks_str}, {days} {days_str} ago" return f"{weeks} {weeks_str}, {days} {days_str} ago"
def show_warning(self, title, msg): def show_warning(title, msg):
"Display a warning to user" "Display a warning to user"
QMessageBox.warning(None, title, msg, buttons=QMessageBox.Cancel) QMessageBox.warning(None, title, msg, buttons=QMessageBox.Cancel)

12
app/playlists.py
View File

@ -16,7 +16,7 @@ import os
from config import Config from config import Config
from datetime import datetime, timedelta from datetime import datetime, timedelta
from helpers import get_relative_date from helpers import get_relative_date, show_warning
from log import DEBUG, ERROR from log import DEBUG, ERROR
from model import ( from model import (
Notes, Playdates, Playlists, PlaylistTracks, Session, Settings, Tracks Notes, Playdates, Playlists, PlaylistTracks, Session, Settings, Tracks
@ -155,8 +155,8 @@ class PlaylistTab(QTableWidget):
self.menu.addSeparator() self.menu.addSeparator()
act_delete = self.menu.addAction('Delete') act_delete = self.menu.addAction('Delete')
act_delete.triggered.connect(lambda: self._delete_row(row)) act_delete.triggered.connect(lambda: self._delete_row(row))
act_delete = self.menu.addAction('Info') act_info = self.menu.addAction('Info')
act_delete.triggered.connect(lambda: self._info_row(row)) act_info.triggered.connect(lambda: self._info_row(row))
return super(PlaylistTab, self).eventFilter(source, event) return super(PlaylistTab, self).eventFilter(source, event)
@ -549,12 +549,10 @@ class PlaylistTab(QTableWidget):
DEBUG(f"playlist._delete_row({row})") DEBUG(f"playlist._delete_row({row})")
if row == self._meta_get_current(): if row == self._meta_get_current():
# TODO show_warning("Silly", "Can't delete playing track")
DEBUG("playlist._delete_row(): Can't delete playing track")
return return
elif row == self._meta_get_next(): elif row == self._meta_get_next():
# TODO show_warning("Safety", "Can't delete next track")
DEBUG("playlist._delete_row(): Can't delete next track")
return return
with Session() as session: with Session() as session: