cco/py-scopes
3
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix tracks.remove(): use track as parameter, not trackId

Browse source
This commit is contained in:
Helmut Merz 2024-04-15 12:39:41 +02:00
parent 5dac623011
commit 5044978acc
2 changed files with 3 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
scopes/storage/tracking.py
View file

@ -171,8 +171,8 @@ class Container(object):
return track.trackId return track.trackId
return self.insert(track, withTrackId=True) return self.insert(track, withTrackId=True)
def remove(self, trackId): def remove(self, track):
stmt = self.table.delete().where(self.table.c.trackid == trackId) stmt = self.table.delete().where(self.table.c.trackid == track.trackId)
n = self.session.execute(stmt).rowcount n = self.session.execute(stmt).rowcount
if n > 0: if n > 0:
self.storage.mark_changed() self.storage.mark_changed()

2
tests/tlib_storage.py
View file

@ -50,7 +50,7 @@ def test_tracking(self, config):
tr03 = storage.getItem('rec-31') tr03 = storage.getItem('rec-31')
self.assertEqual(tr03.trackId, 31) self.assertEqual(tr03.trackId, 31)
n = tracks.remove(31) n = tracks.remove(tr03)
self.assertEqual(n, 1) self.assertEqual(n, 1)
self.assertEqual(tracks.get(31), None) self.assertEqual(tracks.get(31), None)