move media to correct status and set progess_limit correctly (#19)

* move media to correct status and set progess_limit correctly

* remove redundent self

* change before save check to use status enum completed?

app/models/library_entry.rb

@@ -73,8 +73,8 @@ def current_marathon
 
   def progress_limit
     return unless progress
-    progress_cap = media.try(:progress_limit)
-    default_cap = media.try(:default_progress_limit)
+    progress_cap = media&.progress_limit
+    default_cap = media&.default_progress_limit
 
     if progress_cap
       if progress > progress_cap
@@ -100,9 +100,15 @@ def activity
   end
 
   before_save do
-    if status_changed? && status == :completed && media.progress_cap
+    if status_changed? && completed? && media.progress_limit
       # When marked completed, we try to update progress to the cap
-      self.progress = media.progress_cap
+      self.progress = media.progress_limit
+    elsif progress == media.progress_limit
+      # When in current and progress equals total episodes
+      self.status = :completed
+    elsif progress != media.progress_limit && completed?
+      # When in completed and episodes changed, strange case
+      self.status = :current
     end
   end