app/musicmuster.py

@@ -670,15 +670,11 @@ class Window(QMainWindow, Ui_MainWindow):
                 artist = tags['artist']
                 possible_matches = Tracks.search_titles(session, title)
                 if possible_matches:
-                    if len(possible_matches) > 5:
+                    txt += 'Similar to new track '
-                        txt = "More than five tracks look similar to "
+                    txt += f'"{title}" by "{artist} ({fname})":\n\n'
-                        txt += f'"{title}" by "{artist} ({fname})":\n\n'
+                    for track in possible_matches:
-                    else:
+                        txt += f' "{track.title}" by {track.artist}'
-                        txt += 'Similar to new track '
+                        txt += f' ({track.path})\n\n'
-                        txt += f'"{title}" by "{artist} ({fname})":\n\n'
-                        for track in possible_matches:
-                            txt += f' "{track.title}" by {track.artist}'
-                            txt += f' ({track.path})\n\n'
                     txt += "\n"
                     # Check whether to proceed if there were potential matches
                     txt += "Proceed with import?"
@@ -1035,7 +1031,7 @@ class Window(QMainWindow, Ui_MainWindow):
         """Scroll to show current track"""
 
         log.debug(f"KAE: musicmuster.show_current()")
-        if self.current_track_playlist_tab != self.visible_playlist_tab():
+        if self.current_track_playlist_tab != self.visible_playlist_tab:
             self.tabPlaylist.setCurrentWidget(self.current_track_playlist_tab)
         self.tabPlaylist.currentWidget().scroll_current_to_top()
 
@@ -1043,7 +1039,7 @@ class Window(QMainWindow, Ui_MainWindow):
         """Scroll to show next track"""
 
         log.debug(f"KAE: musicmuster.show_next()")
-        if self.next_track_playlist_tab != self.visible_playlist_tab():
+        if self.next_track_playlist_tab != self.visible_playlist_tab:
             self.tabPlaylist.setCurrentWidget(self.next_track_playlist_tab)
         self.tabPlaylist.currentWidget().scroll_next_to_top()
 
@@ -1155,12 +1151,8 @@ class Window(QMainWindow, Ui_MainWindow):
         # Update headers
         self.update_headers()
 
-        # Populate 'info' tabs with Wikipedia info, but queue it because
+        # Populate 'info' tabs with Wikipedia info
-        # it isn't quick
+        self.tabInfolist.open_in_wikipedia(track.title)
-        track_title = track.title
-        QTimer.singleShot(
-            1, lambda: self.tabInfolist.open_in_wikipedia(track_title)
-        )
 
     def tick(self) -> None:
         """

app/playlists.py

@@ -1727,25 +1727,10 @@ class PlaylistTab(QTableWidget):
         top.
         """
 
-        padding_required = Config.SCROLL_TOP_MARGIN
+        if row is not None:
-        top_row = row
+            top_row = max(0, row - Config.SCROLL_TOP_MARGIN + 1)
-
+            scroll_item = self.item(top_row, 0)
-        if row > Config.SCROLL_TOP_MARGIN:
+            self.scrollToItem(scroll_item, QAbstractItemView.PositionAtTop)
-            # We can't scroll to a hidden row. Calculate target_row as the
-            # one that is ideal to be at the top. Then count upwards from
-            # passed row until we either reach the target, pass it or reach
-            # row 0.
-            # target_row = max(0, row - Config.SCROLL_TOP_MARGIN + 1)
-            for i in range(row - 1, -1, -1):
-                if padding_required == 0:
-                    break
-                if self.isRowHidden(i):
-                    continue
-                top_row = i
-                padding_required -= 1
-
-        scroll_item = self.item(top_row, 0)
-        self.scrollToItem(scroll_item, QAbstractItemView.PositionAtTop)
 
     def _select_event(self) -> None:
         """

app/replace_files.py

@@ -87,8 +87,6 @@ def main():
 
     for new_fname in os.listdir(source_dir):
         new_path = os.path.join(source_dir, new_fname)
-        if not os.path.isfile(new_path):
-            continue
         new_tags = get_tags(new_path)
         new_title = new_tags['title']
         new_artist = new_tags['artist']