.envrc

@@ -8,6 +8,12 @@ branch=$(git branch --show-current)
 if on_git_branch master; then
         export MM_ENV="PRODUCTION"
         export MM_DB="mysql+mysqldb://musicmuster:musicmuster@localhost/musicmuster_prod"
+elif on_git_branch carts; then
+        export MM_ENV="DEVELOPMENT"
+        export MM_DB="mysql+mysqldb://dev_musicmuster:dev_musicmuster@localhost/dev_musicmuster_carts"
+elif on_git_branch newcarts; then
+        export MM_ENV="DEVELOPMENT"
+        export MM_DB="mysql+mysqldb://dev_musicmuster:dev_musicmuster@localhost/dev_musicmuster_carts"
 else
         export MM_ENV="DEVELOPMENT"
         export MM_DB="mysql+mysqldb://dev_musicmuster:dev_musicmuster@localhost/dev_musicmuster"

app/log.py

@@ -2,8 +2,7 @@
 
 import logging
 import logging.handlers
-import os
+import stackprinter
-import stackprinter                         # type: ignore
 import sys
 import traceback
 
@@ -76,7 +75,7 @@ def log_uncaught_exceptions(_ex_cls, ex, tb):
     logging.critical(''.join(traceback.format_tb(tb)))
     print("\033[1;37;40m")
     print(stackprinter.format(ex, style="darkbg2", add_summary=True))
-    if os.environ["MM_ENV"] != "DEVELOPMENT":
+    if os.environ("MM_ENV") != "DEVELOPMENT":
         msg = stackprinter.format(ex)
         send_mail(Config.ERRORS_TO, Config.ERRORS_FROM,
                   "Exception from musicmuster", msg)

app/models.py

@@ -2,7 +2,7 @@
 
 import os.path
 import re
-import stackprinter                         # type: ignore
+import stackprinter
 
 from dbconfig import Session
 
@@ -224,14 +224,11 @@ class Playlists(Base):
 
     __tablename__ = "playlists"
 
-    id = Column(Integer, primary_key=True, autoincrement=True)
+    id: int = Column(Integer, primary_key=True, autoincrement=True)
-    name = Column(String(32), nullable=False, unique=True)
+    name: str = Column(String(32), nullable=False, unique=True)
     last_used = Column(DateTime, default=None, nullable=True)
-    tab = Column(Integer, default=None, nullable=True, unique=True)
+    tab = Column(Integer, default=False, nullable=True, unique=True)
-    sort_column = Column(Integer, default=None, nullable=True, unique=False)
     is_template = Column(Boolean, default=False, nullable=False)
-    query = Column(String(256), default=None, nullable=True, unique=False)
-    deleted = Column(Boolean, default=False, nullable=False)
     rows = relationship(
         "PlaylistRows",
         back_populates="playlist",
@@ -266,17 +263,8 @@ class Playlists(Base):
     def close(self, session: Session) -> None:
         """Mark playlist as unloaded"""
 
-        # Closing this tab will mean all higher-number tabs have moved
-        # down by one
-        closed_idx = self.tab
         self.tab = None
 
-        session.execute(
-            update(Playlists)
-            .where(Playlists.tab > closed_idx)
-            .values(tab=Playlists.tab - 1)
-        )
-
     @classmethod
     def create_playlist_from_template(cls,
                                       session: Session,

app/musicmuster.py

@@ -358,7 +358,7 @@ class Window(QMainWindow, Ui_MainWindow):
 
     def close_tab(self, tab_index: int) -> None:
         """
-        Close playlist tab unless it holds the current or next track.
+        Close active playlist tab unless it holds the curren or next track.
         Called from close_playlist_tab() or by clicking close button on tab.
         """
 
@@ -375,12 +375,6 @@ class Window(QMainWindow, Ui_MainWindow):
                 "Can't close next track playlist", 5000)
             return
 
-        # Record playlist as closed and update remaining playlist tabs
-        with Session() as session:
-            playlist_id = self.tabPlaylist.widget(tab_index).playlist_id
-            playlist = session.get(Playlists, playlist_id)
-            playlist.close(session)
-
         # Close playlist and remove tab
         self.tabPlaylist.widget(tab_index).close()
         self.tabPlaylist.removeTab(tab_index)
@@ -426,7 +420,8 @@ class Window(QMainWindow, Ui_MainWindow):
         self.btnStop.clicked.connect(self.stop)
         self.hdrCurrentTrack.clicked.connect(self.show_current)
         self.hdrNextTrack.clicked.connect(self.show_next)
-        self.tabPlaylist.currentChanged.connect(self.tab_change)
+        self.tabPlaylist.currentChanged.connect(
+            lambda: self.tabPlaylist.currentWidget().tab_visible())
         self.tabPlaylist.tabCloseRequested.connect(self.close_tab)
         self.tabBar = self.tabPlaylist.tabBar()
         self.tabBar.tabMoved.connect(self.move_tab)
@@ -1212,15 +1207,6 @@ class Window(QMainWindow, Ui_MainWindow):
         # Run end-of-track actions
         self.end_of_track_actions()
 
-    def tab_change(self):
-        """Called when active tab changed"""
-
-        try:
-            self.tabPlaylist.currentWidget().tab_visible()
-        except AttributeError:
-            # May also be called when last tab is closed
-            pass
-
     def this_is_the_next_track(self, session: Session,
                                playlist_tab: PlaylistTab,
                                track: Tracks) -> None:

app/playlists.py

@@ -203,6 +203,16 @@ class PlaylistTab(QTableWidget):
 
 # ########## Events other than cell editing ##########
 
+    def closeEvent(self, event) -> None:
+        """Handle closing playist tab"""
+
+        with Session() as session:
+            # Record playlist as closed
+            playlist = session.get(Playlists, self.playlist_id)
+            playlist.close(session)
+
+        event.accept()
+
     def dropEvent(self, event: QDropEvent) -> None:
         """
         Handle drag/drop of rows

migrations/versions/07dcbe6c4f0e_add_sort_column_deleted_and_query_to_.py

@@ -1,32 +0,0 @@
-"""Add sort_column, deleted and query to playlists table
-
-Revision ID: 07dcbe6c4f0e
-Revises: 4a7b4ab3354f
-Create Date: 2022-12-25 10:26:38.200941
-
-"""
-from alembic import op
-import sqlalchemy as sa
-
-
-# revision identifiers, used by Alembic.
-revision = '07dcbe6c4f0e'
-down_revision = '4a7b4ab3354f'
-branch_labels = None
-depends_on = None
-
-
-def upgrade():
-    # ### commands auto generated by Alembic - please adjust! ###
-    op.add_column('playlists', sa.Column('sort_column', sa.Integer(), nullable=True))
-    op.add_column('playlists', sa.Column('query', sa.String(length=256), nullable=True))
-    op.add_column('playlists', sa.Column('deleted', sa.Boolean(), nullable=False))
-    # ### end Alembic commands ###
-
-
-def downgrade():
-    # ### commands auto generated by Alembic - please adjust! ###
-    op.drop_column('playlists', 'deleted')
-    op.drop_column('playlists', 'query')
-    op.drop_column('playlists', 'sort_column')
-    # ### end Alembic commands ###