From 5db8bf03294fbc93e5e361be10355b45165160d4 Mon Sep 17 00:00:00 2001
From: Colin Powell <colin@unbl.ink>
Date: Sun, 2 Apr 2023 23:44:07 -0400
Subject: [PATCH] Move check for finish up a level

---
 vrobbler/apps/scrobbles/models.py | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/vrobbler/apps/scrobbles/models.py b/vrobbler/apps/scrobbles/models.py
index 65f54b9..28b465e 100644
--- a/vrobbler/apps/scrobbles/models.py
+++ b/vrobbler/apps/scrobbles/models.py
@@ -648,12 +648,12 @@ class Scrobble(TimeStampedModel):
         if scrobble_status == "resumed":
             self.resume()
 
+        check_scrobble_for_finish(self)
+
         for key, value in scrobble_data.items():
             setattr(self, key, value)
         self.save()
 
-        check_scrobble_for_finish(self)
-
         return self
 
     @classmethod
@@ -670,7 +670,7 @@ class Scrobble(TimeStampedModel):
     def stop(self, force_finish=False) -> None:
         self.in_progress = False
         self.save(update_fields=["in_progress"])
-        logger.info(f"{self.id} - {self.source}")
+        logger.info(f"stopping {self.id} from {self.source}")
 
         class_name = self.media_obj.__class__.__name__
         if class_name in LONG_PLAY_MEDIA.values():