Don't grow window when track title too long

Use an elided text box, set wrapping and max height for label.

Fixes #26

app/model.py

@@ -463,6 +463,12 @@ class Tracks(Base):
 
         return [a[0] for a in session.query(Tracks.path).all()]
 
+    @staticmethod
+    def get_all_tracks(session):
+        "Return a list of all tracks"
+
+        return session.query(Tracks).all()
+
     @classmethod
     def get_or_create(cls, session, path):
         DEBUG(f"Tracks.get_or_create(path={path})")

app/musicmuster.py

@@ -10,11 +10,13 @@ from datetime import datetime, timedelta
 from log import DEBUG, EXCEPTION
 
 from PyQt5.QtCore import Qt, QTimer
+from PyQt5.QtGui import QFontMetrics, QPainter
 from PyQt5.QtWidgets import (
     QApplication,
     QDialog,
     QFileDialog,
     QInputDialog,
+    QLabel,
     QListWidgetItem,
     QMainWindow,
 )
@@ -32,6 +34,21 @@ from ui.dlg_SelectPlaylist_ui import Ui_dlgSelectPlaylist
 from ui.main_window_ui import Ui_MainWindow
 
 
+class ElideLabel(QLabel):
+    """
+    From https://stackoverflow.com/questions/11446478/
+    pyside-pyqt-truncate-text-in-qlabel-based-on-minimumsize
+    """
+
+    def paintEvent(self, event):
+        painter = QPainter(self)
+
+        metrics = QFontMetrics(self.font())
+        elided = metrics.elidedText(self.text(), Qt.ElideRight, self.width())
+
+        painter.drawText(self.rect(), self.alignment(), elided)
+
+
 class Window(QMainWindow, Ui_MainWindow):
     def __init__(self, parent=None):
         super().__init__(parent)

app/songdb.py

@@ -115,10 +115,75 @@ def create_track_from_file(session, path):
     return track
 
 
-def full_update_db():
+def full_update_db(session):
     "Rescan all entries in database"
 
-    print("Full scan not yet implemented")
+    def log(msg):
+        INFO(f"full_update_db(): {msg}")
+
+    def check_change(track_id, title, attribute, old, new):
+        if new > (old * 1.1) or new < (old * 0.9):
+            log(
+                "\n"
+                f"track[{track_id}] ({title}) "
+                f"{attribute} updated from {old} to {new}"
+            )
+
+    # Start with normal update to add new tracks and remove any missing
+    # files
+    log("update_db()")
+    update_db(session)
+
+    # Now update track length, silence and fade for every track in
+    # database
+
+    tracks = Tracks.get_all_tracks(session)
+    total_tracks = len(tracks)
+    log(f"Processing {total_tracks} tracks")
+    track_count = 0
+    for track in tracks:
+        track_count += 1
+        print(f"\rTrack {track_count} of {total_tracks}", end='')
+
+        # Sanity check
+        tag = get_music_info(track.path)
+        if not tag['title']:
+            log(f"track[{track.id}] {track.title=}: No tag title")
+            continue
+        if not tag['artist']:
+            log(f"track[{track.id}] {track.artist=}: No tag artist")
+            continue
+
+        # Update title and artist
+        if track.title != tag['title']:
+            track.title = tag['title']
+        if track.artist != tag['artist']:
+            track.artist = tag['artist']
+
+        # Update numbers; log if more than 10% different
+        duration = int(round(
+            tag['duration'], Config.MILLISECOND_SIGFIGS) * 1000)
+        check_change(track.id, track.title, "duration", track.duration,
+                     duration)
+        track.duration = duration
+
+        audio = get_audio_segment(track.path)
+
+        start_gap = leading_silence(audio)
+        check_change(track.id, track.title, "start_gap", track.start_gap,
+                     start_gap)
+        track.start_gap = start_gap
+
+        fade_at = fade_point(audio)
+        check_change(track.id, track.title, "fade_at", track.fade_at,
+                     fade_at)
+        track.fade_at = fade_at
+
+        silence_at = trailing_silence(audio)
+        check_change(track.id, track.title, "silence_at", track.silence_at,
+                     silence_at)
+        track.silence_at = silence_at
+        session.commit()
 
 
 def get_audio_segment(path):
@@ -196,22 +261,6 @@ def fade_point(audio_segment, fade_threshold=0,
     return int(trim_ms)
 
 
-# Current unused (1 June 2021)
-# def rescan_database(session):
-#
-#     tracks = Tracks.get_all_tracks(session)
-#     total_tracks = len(tracks)
-#     track_count = 0
-#     for track in tracks:
-#         track_count += 1
-#         print(f"Track {track_count} of {total_tracks}")
-#         audio = get_audio_segment(track.path)
-#         track.start_gap = leading_silence(audio)
-#         track.fade_at = fade_point(audio)
-#         track.silence_at = trailing_silence(audio)
-#         session.commit()
-
-
 def trailing_silence(audio_segment, silence_threshold=-50.0,
                      chunk_size=Config.AUDIO_SEGMENT_CHUNK_SIZE):
     return fade_point(audio_segment, silence_threshold, chunk_size)

app/ui/main_window.ui

@@ -149,6 +149,9 @@ border: 1px solid rgb(85, 87, 83);</string>
           <property name="text">
            <string/>
           </property>
+          <property name="wordWrap">
+           <bool>true</bool>
+          </property>
          </widget>
         </item>
         <item>
@@ -166,10 +169,25 @@ border: 1px solid rgb(85, 87, 83);</string>
           <property name="text">
            <string/>
           </property>
+          <property name="wordWrap">
+           <bool>true</bool>
+          </property>
          </widget>
         </item>
         <item>
-         <widget class="QLabel" name="hdrNextTrack">
+         <widget class="ElideLabel" name="hdrNextTrack">
+          <property name="minimumSize">
+           <size>
+            <width>0</width>
+            <height>39</height>
+           </size>
+          </property>
+          <property name="maximumSize">
+           <size>
+            <width>16777215</width>
+            <height>39</height>
+           </size>
+          </property>
           <property name="font">
            <font>
             <family>Sans</family>
@@ -183,6 +201,9 @@ border: 1px solid rgb(85, 87, 83);</string>
           <property name="text">
            <string/>
           </property>
+          <property name="wordWrap">
+           <bool>true</bool>
+          </property>
          </widget>
         </item>
        </layout>
@@ -392,13 +413,13 @@ border: 1px solid rgb(85, 87, 83);</string>
         </spacer>
        </item>
        <item>
-        <widget class="QPushButton" name="btnFade">
+        <widget class="QPushButton" name="btnStop">
          <property name="text">
-          <string>Fade</string>
+          <string>Stop</string>
          </property>
          <property name="icon">
           <iconset resource="icons.qrc">
-           <normaloff>:/icons/fade</normaloff>:/icons/fade</iconset>
+           <normaloff>:/icons/stopsign</normaloff>:/icons/stopsign</iconset>
          </property>
          <property name="iconSize">
           <size>
@@ -409,13 +430,13 @@ border: 1px solid rgb(85, 87, 83);</string>
         </widget>
        </item>
        <item>
-        <widget class="QPushButton" name="btnStop">
+        <widget class="QPushButton" name="btnFade">
          <property name="text">
-          <string>Stop</string>
+          <string>Fade</string>
          </property>
          <property name="icon">
           <iconset resource="icons.qrc">
-           <normaloff>:/icons/stopsign</normaloff>:/icons/stopsign</iconset>
+           <normaloff>:/icons/fade</normaloff>:/icons/fade</iconset>
          </property>
          <property name="iconSize">
           <size>
@@ -984,6 +1005,13 @@ border: 1px solid rgb(85, 87, 83);</string>
    </property>
   </action>
  </widget>
+ <customwidgets>
+  <customwidget>
+   <class>ElideLabel</class>
+   <extends>QLabel</extends>
+   <header>musicmuster</header>
+  </customwidget>
+ </customwidgets>
  <resources>
   <include location="icons.qrc"/>
  </resources>

app/ui/main_window_ui.py

@@ -85,6 +85,7 @@ class Ui_MainWindow(object):
         self.hdrPreviousTrack.setStyleSheet("background-color: #f8d7da;\n"
 "border: 1px solid rgb(85, 87, 83);")
         self.hdrPreviousTrack.setText("")
+        self.hdrPreviousTrack.setWordWrap(True)
         self.hdrPreviousTrack.setObjectName("hdrPreviousTrack")
         self.verticalLayout.addWidget(self.hdrPreviousTrack)
         self.hdrCurrentTrack = QtWidgets.QLabel(self.centralwidget)
@@ -95,9 +96,12 @@ class Ui_MainWindow(object):
         self.hdrCurrentTrack.setStyleSheet("background-color: #d4edda;\n"
 "border: 1px solid rgb(85, 87, 83);")
         self.hdrCurrentTrack.setText("")
+        self.hdrCurrentTrack.setWordWrap(True)
         self.hdrCurrentTrack.setObjectName("hdrCurrentTrack")
         self.verticalLayout.addWidget(self.hdrCurrentTrack)
-        self.hdrNextTrack = QtWidgets.QLabel(self.centralwidget)
+        self.hdrNextTrack = ElideLabel(self.centralwidget)
+        self.hdrNextTrack.setMinimumSize(QtCore.QSize(0, 39))
+        self.hdrNextTrack.setMaximumSize(QtCore.QSize(16777215, 39))
         font = QtGui.QFont()
         font.setFamily("Sans")
         font.setPointSize(20)
@@ -105,6 +109,7 @@ class Ui_MainWindow(object):
         self.hdrNextTrack.setStyleSheet("background-color: #fff3cd;\n"
 "border: 1px solid rgb(85, 87, 83);")
         self.hdrNextTrack.setText("")
+        self.hdrNextTrack.setWordWrap(True)
         self.hdrNextTrack.setObjectName("hdrNextTrack")
         self.verticalLayout.addWidget(self.hdrNextTrack)
         self.horizontalLayout_3.addLayout(self.verticalLayout)
@@ -181,20 +186,20 @@ class Ui_MainWindow(object):
         self.horizontalLayout.addWidget(self.btnSetNext)
         spacerItem3 = QtWidgets.QSpacerItem(69, 20, QtWidgets.QSizePolicy.Expanding, QtWidgets.QSizePolicy.Minimum)
         self.horizontalLayout.addItem(spacerItem3)
-        self.btnFade = QtWidgets.QPushButton(self.frame_5)
-        icon6 = QtGui.QIcon()
-        icon6.addPixmap(QtGui.QPixmap(":/icons/fade"), QtGui.QIcon.Normal, QtGui.QIcon.Off)
-        self.btnFade.setIcon(icon6)
-        self.btnFade.setIconSize(QtCore.QSize(30, 30))
-        self.btnFade.setObjectName("btnFade")
-        self.horizontalLayout.addWidget(self.btnFade)
         self.btnStop = QtWidgets.QPushButton(self.frame_5)
-        icon7 = QtGui.QIcon()
-        icon7.addPixmap(QtGui.QPixmap(":/icons/stopsign"), QtGui.QIcon.Normal, QtGui.QIcon.Off)
-        self.btnStop.setIcon(icon7)
+        icon6 = QtGui.QIcon()
+        icon6.addPixmap(QtGui.QPixmap(":/icons/stopsign"), QtGui.QIcon.Normal, QtGui.QIcon.Off)
+        self.btnStop.setIcon(icon6)
         self.btnStop.setIconSize(QtCore.QSize(30, 30))
         self.btnStop.setObjectName("btnStop")
         self.horizontalLayout.addWidget(self.btnStop)
+        self.btnFade = QtWidgets.QPushButton(self.frame_5)
+        icon7 = QtGui.QIcon()
+        icon7.addPixmap(QtGui.QPixmap(":/icons/fade"), QtGui.QIcon.Normal, QtGui.QIcon.Off)
+        self.btnFade.setIcon(icon7)
+        self.btnFade.setIconSize(QtCore.QSize(30, 30))
+        self.btnFade.setObjectName("btnFade")
+        self.horizontalLayout.addWidget(self.btnFade)
         self.spnVolume = QtWidgets.QSpinBox(self.frame_5)
         self.spnVolume.setMaximum(100)
         self.spnVolume.setProperty("value", 100)
@@ -486,8 +491,8 @@ class Ui_MainWindow(object):
         self.btnAddFile.setText(_translate("MainWindow", "Add file"))
         self.btnAddNote.setText(_translate("MainWindow", "Add note"))
         self.btnSetNext.setText(_translate("MainWindow", "Set next"))
-        self.btnFade.setText(_translate("MainWindow", "Fade"))
         self.btnStop.setText(_translate("MainWindow", "Stop"))
+        self.btnFade.setText(_translate("MainWindow", "Fade"))
         self.label_2.setText(_translate("MainWindow", "Started at:"))
         self.label_start_tod.setText(_translate("MainWindow", "00:00:00"))
         self.label_3.setText(_translate("MainWindow", "Silent at:"))
@@ -540,4 +545,5 @@ class Ui_MainWindow(object):
         self.actionSelect_previous_track.setShortcut(_translate("MainWindow", "K"))
         self.actionSelect_played_tracks.setText(_translate("MainWindow", "Select played tracks"))
         self.actionSelect_unplayed_tracks.setText(_translate("MainWindow", "Select unplayed tracks"))
+from musicmuster import ElideLabel
 import icons_rc