From d8072ae73f4234442c3365ba7385f3447f77d32e Mon Sep 17 00:00:00 2001
From: Keith Edmunds <kae@tiger-computing.co.uk>
Date: Mon, 23 Aug 2021 09:23:18 +0100
Subject: [PATCH] Remove TODOs from code.

Fixes #57
---
 app/model.py       |  4 ++--
 app/musicmuster.py | 10 +++-------
 2 files changed, 5 insertions(+), 9 deletions(-)

diff --git a/app/model.py b/app/model.py
index d4ccd9d..862ee3e 100644
--- a/app/model.py
+++ b/app/model.py
@@ -1,5 +1,6 @@
 #!/usr/bin/python3
 
+import os.path
 import sqlalchemy
 
 from datetime import datetime
@@ -500,8 +501,7 @@ class Tracks(Base):
         DEBUG(f"Tracks.get_track_from_filename({filename=})")
         try:
             track = session.query(Tracks).filter(Tracks.path.ilike(
-                # TODO: filename separator is hardcoded here
-                f'%/{filename}')).one()
+                f'%{os.path.sep}{filename}')).one()
             return track
         except (NoResultFound, MultipleResultsFound):
             return None
diff --git a/app/musicmuster.py b/app/musicmuster.py
index f697d04..72a4b1b 100755
--- a/app/musicmuster.py
+++ b/app/musicmuster.py
@@ -132,7 +132,9 @@ class Window(QMainWindow, Ui_MainWindow):
         if self.music.playing():
             DEBUG("closeEvent() ignored as music is playing")
             event.ignore()
-            # TODO notify user
+            helpers.show_warning(
+                "Track playing",
+                "Can't close application while track is playing")
         else:
             DEBUG("closeEvent() accepted")
 
@@ -516,12 +518,6 @@ class Window(QMainWindow, Ui_MainWindow):
                 silence_at - self.current_track.fade_at
             ))
 
-    def play_previous(self):
-        "Resume playing last track"
-
-        # TODO
-        pass
-
     def search_database(self):
         with Session() as session:
             dlg = DbDialog(self, session)