diff --git a/app/helpers.py b/app/helpers.py index 7da6bff..c7869aa 100644 --- a/app/helpers.py +++ b/app/helpers.py @@ -37,7 +37,7 @@ def get_relative_date(past_date, reference_date=None): 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" QMessageBox.warning(None, title, msg, buttons=QMessageBox.Cancel) diff --git a/app/playlists.py b/app/playlists.py index c26ebb9..818e0c9 100644 --- a/app/playlists.py +++ b/app/playlists.py @@ -16,7 +16,7 @@ import os from config import Config 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 model import ( Notes, Playdates, Playlists, PlaylistTracks, Session, Settings, Tracks @@ -155,8 +155,8 @@ class PlaylistTab(QTableWidget): self.menu.addSeparator() act_delete = self.menu.addAction('Delete') act_delete.triggered.connect(lambda: self._delete_row(row)) - act_delete = self.menu.addAction('Info') - act_delete.triggered.connect(lambda: self._info_row(row)) + act_info = self.menu.addAction('Info') + act_info.triggered.connect(lambda: self._info_row(row)) return super(PlaylistTab, self).eventFilter(source, event) @@ -549,12 +549,10 @@ class PlaylistTab(QTableWidget): DEBUG(f"playlist._delete_row({row})") if row == self._meta_get_current(): - # TODO - DEBUG("playlist._delete_row(): Can't delete playing track") + show_warning("Silly", "Can't delete playing track") return elif row == self._meta_get_next(): - # TODO - DEBUG("playlist._delete_row(): Can't delete next track") + show_warning("Safety", "Can't delete next track") return with Session() as session: