appdb/cron cleanup.php

[WineHQ]

ChangeSet ID:	31172
CVSROOT:	/opt/cvs-commit
Module name:	appdb
Changes by:	[e-mail]	2007/06/10 12:13:49

Modified files:
    cron           : cleanup.php

Log message:
    Chris Morgan 
    Disable screenshot deletion and add screenshot ids to the cleanup email so we can verify that
    the cleanup script is removing the correct screenshots.

Patch: http://cvs.winehq.org/patch.py?id=31172

Old revision  New revision  Changes     Path
 1.35          1.36          +14 -5      appdb/cron/cleanup.php

Index: appdb/cron/cleanup.php
diff -u -p appdb/cron/cleanup.php:1.35 appdb/cron/cleanup.php:1.36
--- appdb/cron/cleanup.php:1.35	10 Jun 2007 17:13:49 -0000
+++ appdb/cron/cleanup.php	10 Jun 2007 17:13:49 -0000
@@ -285,6 +285,15 @@ function removeScreenshotsWithMissingFil
     $sMsg = "Found ".count($aMissingScreenshotIds)." screenshots with missing files.\r\n";
     $sMsg.= "Deleting these screenshots.\r\n";

+    // add the screenshot ids to the email so we can see which screenshots are
+    // going to be deleted
+    $sMsg.="\r\n";
+    $sMsg.="Screenshot IDs:\r\n";
+    foreach($aMissingScreenshotIds as $iScreenshotId)
+    {
+        $sMsg.=$iScreenshotId."\r\n";
+    }
+
     $sSubject = "Screenshots deleted\r\n";

$sEmail = User::get_notify_email_address_list(null, null); /* get list admins */
@@ -292,10 +301,10 @@ function removeScreenshotsWithMissingFil
         mail_appdb($sEmail, $sSubject, $sMsg);

// remove the screenshots with missing files
-    foreach($aMissingScreenshotIds as $iScreenshotId)
-    {
-        $oScreenshot = new Screenshot($iScreenshotId);
-        $oScreenshot->delete(true); // delete the screenshot silently
-    }
+//    foreach($aMissingScreenshotIds as $iScreenshotId)
+//    {
+//        $oScreenshot = new Screenshot($iScreenshotId);
+//        $oScreenshot->delete(); // delete the screenshot
+//    }
 }
 ?>