app/playlistmodel.py

@@ -363,15 +363,12 @@ class PlaylistModel(QAbstractTableModel):
 
         Need to delete them in contiguous groups wrapped in beginRemoveRows / endRemoveRows
         calls. To keep it simple, if inefficient, delete rows one by one.
-
-        TODO: delete in blocks
-
-        Delete from highest row back so that not yet deleted row numbers don't change.
         """
 
+        log.info(f"delete_rows({row_numbers=}")
+
         with Session() as session:
-            for row_number in sorted(row_numbers, reverse=True):
-                log.info(f"delete_rows(), {row_number=}")
+            for row_number in row_numbers:
                 super().beginRemoveRows(QModelIndex(), row_number, row_number)
                 # We need to remove data from the underlying data store,
                 # which is the database, but we cache in
@@ -582,7 +579,7 @@ class PlaylistModel(QAbstractTableModel):
             for a in self.playlist_rows.values()
             if not a.played and a.track_id is not None
         ]
-        log.debug(f"get_unplayed_rows() returned: {result=}")
+        log.info(f"get_unplayed_rows() returned: {result=}")
         return result
 
     def headerData(