Default config definition more consistent

[mpd-sima.git] / sima / plugins / core / history.py

diff --git a/sima/plugins/core/history.py b/sima/plugins/core/history.py
index b85c0dd923b1cb302237fa299dbcd301b3d14cf6..6c3ad907ce6a5c19ed453f6a42dea50104656426 100644 (file)
--- a/sima/plugins/core/history.py
+++ b/sima/plugins/core/history.py
@@ -34,7 +34,6 @@ class History(Plugin):
     """
     def __init__(self, daemon):
         Plugin.__init__(self, daemon)
-        self.sdb = daemon.sdb
         self._last_clean = time()
 
     def shutdown(self):
@@ -45,14 +44,14 @@ class History(Plugin):
     def callback_player(self):
         current = self.player.current
         last_hist = next(self.sdb.get_history(), None)
-        if last_hist[3] == current.file:
+        if last_hist and last_hist[3] == current.file:
             return
         if not current:
             self.log.debug('Cannot add "%s" to history (empty or missing file)', current)
             return
         self.log.debug('add history: "%s"', current)
         self.sdb.add_history(current)
-        if self._last_clean - time() > 86400:
+        if time() - self._last_clean > 86400:
             self.shutdown()
             self._last_clean = time()