kae/musicmuster
1
0
Fork 0
Code Issues 28 Pull Requests Projects Releases 1 Activity

Change PlaylistRows.row_number to row_no

Browse Source
This commit is contained in:
Keith Edmunds 2023-04-01 19:59:25 +01:00
parent 5cb6e83cd5
commit f9f1e5f237
2 changed files with 37 additions and 37 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

34
app/models.py
View File

@ -203,7 +203,7 @@ class Playlists(Base):
"PlaylistRows", "PlaylistRows",
back_populates="playlist", back_populates="playlist",
cascade="all, delete-orphan", cascade="all, delete-orphan",
order_by="PlaylistRows.row_number" order_by="PlaylistRows.row_no"
) )
def __repr__(self) -> str: def __repr__(self) -> str:
@ -372,7 +372,7 @@ class PlaylistRows(Base):
__tablename__ = 'playlist_rows' __tablename__ = 'playlist_rows'
id: int = Column(Integer, primary_key=True, autoincrement=True) id: int = Column(Integer, primary_key=True, autoincrement=True)
row_number: int = Column(Integer, nullable=False) row_no: int = Column(Integer, nullable=False)
note: str = Column(String(2048), index=False, default="", nullable=False) note: str = Column(String(2048), index=False, default="", nullable=False)
playlist_id: int = Column(Integer, ForeignKey('playlists.id'), playlist_id: int = Column(Integer, ForeignKey('playlists.id'),
nullable=False) nullable=False)
@ -385,7 +385,7 @@ class PlaylistRows(Base):
return ( return (
f"<PlaylistRow(id={self.id}, playlist_id={self.playlist_id}, " f"<PlaylistRow(id={self.id}, playlist_id={self.playlist_id}, "
f"track_id={self.track_id}, " f"track_id={self.track_id}, "
f"note={self.note}, row_number={self.row_number}>" f"note={self.note}, row_no={self.row_no}>"
) )
def __init__(self, def __init__(self,
@ -399,7 +399,7 @@ class PlaylistRows(Base):
self.playlist_id = playlist_id self.playlist_id = playlist_id
self.track_id = track_id self.track_id = track_id
self.row_number = row_number self.row_no = row_number
self.note = note self.note = note
session.add(self) session.add(self)
session.flush() session.flush()
@ -425,7 +425,7 @@ class PlaylistRows(Base):
).scalars().all() ).scalars().all()
for plr in src_rows: for plr in src_rows:
PlaylistRows(session, dst_id, plr.track_id, plr.row_number, PlaylistRows(session, dst_id, plr.track_id, plr.row_no,
plr.note) plr.note)
@staticmethod @staticmethod
@ -440,7 +440,7 @@ class PlaylistRows(Base):
delete(PlaylistRows) delete(PlaylistRows)
.where( .where(
PlaylistRows.playlist_id == playlist_id, PlaylistRows.playlist_id == playlist_id,
PlaylistRows.row_number > maxrow PlaylistRows.row_no > maxrow
) )
) )
session.flush() session.flush()
@ -454,11 +454,11 @@ class PlaylistRows(Base):
plrs = session.execute( plrs = session.execute(
select(PlaylistRows) select(PlaylistRows)
.where(PlaylistRows.playlist_id == playlist_id) .where(PlaylistRows.playlist_id == playlist_id)
.order_by(PlaylistRows.row_number) .order_by(PlaylistRows.row_no)
).scalars().all() ).scalars().all()
for i, plr in enumerate(plrs): for i, plr in enumerate(plrs):
plr.row_number = i plr.row_no = i
# Ensure new row numbers are available to the caller # Ensure new row numbers are available to the caller
session.commit() session.commit()
@ -476,7 +476,7 @@ class PlaylistRows(Base):
.where( .where(
cls.playlist_id == playlist_id, cls.playlist_id == playlist_id,
cls.id.in_(plr_ids) cls.id.in_(plr_ids)
).order_by(cls.row_number)).scalars().all() ).order_by(cls.row_no)).scalars().all()
return plrs return plrs
@ -486,7 +486,7 @@ class PlaylistRows(Base):
"""Return the last used row for playlist, or None if no rows""" """Return the last used row for playlist, or None if no rows"""
return session.execute( return session.execute(
select(func.max(PlaylistRows.row_number)) select(func.max(PlaylistRows.row_no))
.where(PlaylistRows.playlist_id == playlist_id) .where(PlaylistRows.playlist_id == playlist_id)
).scalar_one() ).scalar_one()
@ -518,7 +518,7 @@ class PlaylistRows(Base):
cls.playlist_id == playlist_id, cls.playlist_id == playlist_id,
cls.played.is_(True) cls.played.is_(True)
) )
.order_by(cls.row_number) .order_by(cls.row_no)
).scalars().all() ).scalars().all()
return plrs return plrs
@ -538,12 +538,12 @@ class PlaylistRows(Base):
cls.track_id.is_not(None) cls.track_id.is_not(None)
) )
if from_row is not None: if from_row is not None:
query = query.where(cls.row_number >= from_row) query = query.where(cls.row_no >= from_row)
if to_row is not None: if to_row is not None:
query = query.where(cls.row_number <= to_row) query = query.where(cls.row_no <= to_row)
plrs = ( plrs = (
session.execute((query).order_by(cls.row_number)).scalars().all() session.execute((query).order_by(cls.row_no)).scalars().all()
) )
return plrs return plrs
@ -563,7 +563,7 @@ class PlaylistRows(Base):
cls.track_id.is_not(None), cls.track_id.is_not(None),
cls.played.is_(False) cls.played.is_(False)
) )
.order_by(cls.row_number) .order_by(cls.row_no)
).scalars().all() ).scalars().all()
return plrs return plrs
@ -580,9 +580,9 @@ class PlaylistRows(Base):
update(PlaylistRows) update(PlaylistRows)
.where( .where(
(PlaylistRows.playlist_id == playlist_id), (PlaylistRows.playlist_id == playlist_id),
(PlaylistRows.row_number >= starting_row) (PlaylistRows.row_no >= starting_row)
) )
.values(row_number=PlaylistRows.row_number + move_by) .values(row_no=PlaylistRows.row_no + move_by)
) )

40
app/playlists.py
View File

@ -531,7 +531,7 @@ class PlaylistTab(QTableWidget):
Insert passed playlist row (plr) into playlist tab. Insert passed playlist row (plr) into playlist tab.
""" """
row_number = plr.row_number row_number = plr.row_no
bold = True bold = True
self.insertRow(row_number) self.insertRow(row_number)
_ = self._set_row_plr_id(row_number, plr.id) _ = self._set_row_plr_id(row_number, plr.id)
@ -695,7 +695,7 @@ class PlaylistTab(QTableWidget):
for plr in playlist.rows: for plr in playlist.rows:
self.insert_row(session, plr, update_track_times=False, self.insert_row(session, plr, update_track_times=False,
played=plr.row_number in played_rows) played=plr.row_no in played_rows)
# Scroll to top # Scroll to top
if scroll_to_top: if scroll_to_top:
@ -756,7 +756,7 @@ class PlaylistTab(QTableWidget):
plr = self._get_row_plr(session, row_number) plr = self._get_row_plr(session, row_number)
if not plr: if not plr:
continue continue
plr.row_number = row_number plr.row_no = row_number
plr.playlist_id = self.playlist_id plr.playlist_id = self.playlist_id
# Any rows in the database for this playlist that has a row # Any rows in the database for this playlist that has a row
@ -1086,7 +1086,7 @@ class PlaylistTab(QTableWidget):
f"Really delete {row_count} row{plural}?"): f"Really delete {row_count} row{plural}?"):
return return
rows_to_delete = [plr.row_number for plr in plrs] rows_to_delete = [plr.row_no for plr in plrs]
# Delete rows from database. Would be more efficient to # Delete rows from database. Would be more efficient to
# query then have a single delete. # query then have a single delete.
@ -1133,11 +1133,11 @@ class PlaylistTab(QTableWidget):
starting_row = 0 starting_row = 0
track_rows = [ track_rows = [
p.row_number for p in PlaylistRows.get_rows_with_tracks( p.row_no for p in PlaylistRows.get_rows_with_tracks(
session, self.playlist_id) session, self.playlist_id)
] ]
played_rows = [ played_rows = [
p.row_number for p in PlaylistRows.get_played_rows( p.row_no for p in PlaylistRows.get_played_rows(
session, self.playlist_id) session, self.playlist_id)
] ]
for row_number in range(starting_row, self.rowCount()): for row_number in range(starting_row, self.rowCount()):
@ -1194,8 +1194,8 @@ class PlaylistTab(QTableWidget):
""" """
return [ return [
p.row_number for p in PlaylistRows.get_played_rows( p.row_no for p in PlaylistRows.get_played_rows(
session, self.playlist_id) if p.row_number is not None session, self.playlist_id) if p.row_no is not None
] ]
def _get_row_artist(self, row_number: int) -> str: def _get_row_artist(self, row_number: int) -> str:
@ -1407,16 +1407,16 @@ class PlaylistTab(QTableWidget):
new_row_number: int) -> None: new_row_number: int) -> None:
"""Move playlist row to new_row_number using parent copy/paste""" """Move playlist row to new_row_number using parent copy/paste"""
if plr.row_number is None: if plr.row_no is None:
return return
# Remove source row # Remove source row
self.removeRow(plr.row_number) self.removeRow(plr.row_no)
# Fixup plr row number # Fixup plr row number
if plr.row_number < new_row_number: if plr.row_no < new_row_number:
plr.row_number = new_row_number - 1 plr.row_no = new_row_number - 1
else: else:
plr.row_number = new_row_number plr.row_no = new_row_number
self.insert_row(session, plr) self.insert_row(session, plr)
self.save_playlist(session) self.save_playlist(session)
self.hide_or_show_played_tracks() self.hide_or_show_played_tracks()
@ -1728,7 +1728,7 @@ class PlaylistTab(QTableWidget):
# Unmark next track # Unmark next track
if old_plr and old_plr.playlist_id == self.playlist_id: if old_plr and old_plr.playlist_id == self.playlist_id:
self._set_row_colour_default(old_plr.row_number) self._set_row_colour_default(old_plr.row_no)
# Mark next track # Mark next track
if new_plrid: if new_plrid:
@ -1737,7 +1737,7 @@ class PlaylistTab(QTableWidget):
log.error(f"_reset_next({new_plrid=}): plr not found") log.error(f"_reset_next({new_plrid=}): plr not found")
return return
if new_plr.playlist_id == self.playlist_id: if new_plr.playlist_id == self.playlist_id:
self._set_row_colour_next(new_plr.row_number) self._set_row_colour_next(new_plr.row_no)
# Update start/stop times # Update start/stop times
self._update_start_end_times(session) self._update_start_end_times(session)
@ -2082,7 +2082,7 @@ class PlaylistTab(QTableWidget):
""" """
plr_tracks = PlaylistRows.get_rows_with_tracks( plr_tracks = PlaylistRows.get_rows_with_tracks(
session, self.playlist_id, from_plr.row_number, to_plr.row_number) session, self.playlist_id, from_plr.row_no, to_plr.row_no)
total_time = 0 total_time = 0
total_time = sum([a.track.duration for a in plr_tracks total_time = sum([a.track.duration for a in plr_tracks
@ -2134,7 +2134,7 @@ class PlaylistTab(QTableWidget):
total_time = self._track_time_between_rows( total_time = self._track_time_between_rows(
session, from_plr, to_plr) session, from_plr, to_plr)
time_str = self._get_section_timing_string(total_time) time_str = self._get_section_timing_string(total_time)
self._set_row_header_text(session, from_plr.row_number, self._set_row_header_text(session, from_plr.row_no,
from_plr.note + time_str) from_plr.note + time_str)
# Update section end # Update section end
@ -2144,12 +2144,12 @@ class PlaylistTab(QTableWidget):
section_header_cleanup_re, '', from_plr.note, section_header_cleanup_re, '', from_plr.note,
).strip() + "]" ).strip() + "]"
) )
self._set_row_header_text(session, to_plr.row_number, self._set_row_header_text(session, to_plr.row_no,
new_text) new_text)
except IndexError: except IndexError:
# This ending row may have a time left from before a # This ending row may have a time left from before a
# starting row above was deleted, so replace content # starting row above was deleted, so replace content
self._set_row_header_text(session, plr.row_number, self._set_row_header_text(session, plr.row_no,
plr.note) plr.note)
continue continue
@ -2163,7 +2163,7 @@ class PlaylistTab(QTableWidget):
from_plr, to_plr) from_plr, to_plr)
time_str = self._get_section_timing_string(total_time, time_str = self._get_section_timing_string(total_time,
no_end=True) no_end=True)
self._set_row_header_text(session, from_plr.row_number, self._set_row_header_text(session, from_plr.row_no,
from_plr.note + time_str) from_plr.note + time_str)
def _update_start_end_times(self, session: scoped_session) -> None: def _update_start_end_times(self, session: scoped_session) -> None: