app/musicmuster.py

@@ -1609,13 +1609,8 @@ class DbDialog(QDialog):
             self.accept()
             return
 
-        if track:
+        self.parent().visible_playlist_tab().insert_track(
-            self.parent().visible_playlist_tab().insert_track(
+            self.session, track, note=self.ui.txtNote.text())
-                self.session, track, note=self.ui.txtNote.text())
-        else:
-            self.parent().visible_playlist_tab().insert_header(
-                self.session, note=self.ui.txtNote.text())
-
         # Save to database (which will also commit changes)
         self.parent().visible_playlist_tab().save_playlist(self.session)
         # Clear note field and select search text to make it easier for

app/playlists.py

@@ -669,7 +669,7 @@ class PlaylistTab(QTableWidget):
             self.update_display(session)
 
     def insert_track(self, session: scoped_session, track: Tracks,
-                     note: Optional[str] = None, repaint: bool = True) -> None:
+                     note: str = None, repaint: bool = True) -> None:
         """
         Insert track into playlist tab.
 
@@ -680,10 +680,10 @@ class PlaylistTab(QTableWidget):
         to do the heavy lifing.
         """
 
-        if not track:
+        if not track and track.id:
             log.debug(
-                f"insert_track({session=}, {note=}, {repaint=}"
+                f"insert_track({session=}, {track=}, {note=}, {repaint=}"
-                " called with no track"
+                " called with either no track or no track.id"
             )
             return
 
@@ -719,7 +719,7 @@ class PlaylistTab(QTableWidget):
         """
 
         current_row = self._get_current_track_row_number()
-        if current_row is None:
+        if not current_row:
             send_mail(Config.ERRORS_TO,
                       Config.ERRORS_FROM,
                       "MusicMuster unexpected failure",