appdb/cron cleanup.php

[WineHQ]

ChangeSet ID:	31163
CVSROOT:	/opt/cvs-commit
Module name:	appdb
Changes by:	wineowner at winehq.org	2007/06/07 19:42:06

Modified files:
	cron           : cleanup.php 

Log message:
	Chris Morgan <cmorgan at alum.wpi.edu>
	Activte the deletion portion of the 'screenshots with missing files' cleanup

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

Old revision  New revision  Changes     Path
 1.34          1.35          +6 -8       appdb/cron/cleanup.php

Index: appdb/cron/cleanup.php
diff -u -p appdb/cron/cleanup.php:1.34 appdb/cron/cleanup.php:1.35
--- appdb/cron/cleanup.php:1.34	8 Jun 2007  0:42: 6 -0000
+++ appdb/cron/cleanup.php	8 Jun 2007  0:42: 6 -0000
@@ -283,8 +283,7 @@ function removeScreenshotsWithMissingFil
 
     // build the email to admins about what we are doing
     $sMsg = "Found ".count($aMissingScreenshotIds)." screenshots with missing files.\r\n";
-    //FIXME: uncomment the below line when we uncomment the below lines in the script
-    //    $sMsg.= "Deleting these screenshots.\r\n";
+    $sMsg.= "Deleting these screenshots.\r\n";
 
     $sSubject = "Screenshots deleted\r\n";
 
@@ -292,12 +291,11 @@ function removeScreenshotsWithMissingFil
     if($sEmail)
         mail_appdb($sEmail, $sSubject, $sMsg);
 
-    //FIXME: activate this after we see the results from the nightly cron script email
     // 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(true); // delete the screenshot silently
+    }
 }
 ?>