Changeset 381

Timestamp:

21/05/08 12:48:21 (2 years ago)

Author:

stuarth

Message:

Marked as compatible with SqueezeCenter? 7.1.
Kick off cleanup after a rescan has been triggered (this is supposed to remove the old items).
Marked plugin as only being compatible with Linux (for now).

Location:

trunk

Files:

• Makefile (modified) (1 diff)
• Plugin.pm (modified) (3 diffs)
• install.xml (modified) (1 diff)

trunk/Makefile

@@ -20 +20 @@
 # $Id$
 
-VERSION=1.0b3
+VERSION=1.0b4
 PERLSOURCE=Plugin.pm Settings.pm
 HTMLSOURCE=HTML/EN/plugins/AutoRescan/settings/basic.html

trunk/Plugin.pm

@@ -220 +220 @@
         $inotify->poll;
 
+        # Flag of whether any rescanning was performed.
+        my $scan_done = 0;
+
         # We don't perform any rescanning if a scan is currently underway - we
         # defer it until it's finished.
@@ -231 +234 @@
                         if ($touchedDirs{$dir} < $triggerTime) {
                                 $log->info("Triggering RESCAN of folder: $dir");
+                                $scan_done = 1;
 
                                 # Rescan the changed directory.
@@ -255 +259 @@
                         }
                 }
+        }
+
+        # If a rescan was performed then do a database cleanup. This is necessary
+        # to remove items from the database that no longer exist - eg the old file
+        # if the file has been renamed or moved.
+        # Note that this might be slow since it has to traverse every file in the
+        # database - that shouldn't be too much of an annoyance, though, since
+        # the user is assumed to have just moved music around and it won't happen
+        # too often.
+        if ($scan_done) {
+                $log->info("One or more scans have been performed. Now performing cleanup");
+                Slim::Schema->cleanupStaleTrackEntries;
         }
 

trunk/install.xml

@@ -41 +41 @@
                 <id>SqueezeCentre</id>
                 <minVersion>7.0a</minVersion>
-                <maxVersion>7.0+</maxVersion>
+                <maxVersion>7.1+</maxVersion>
         </targetApplication>
+
+        <targetPlatform>Linux</targetPlatform>
 </extension>