ChangeSet ID: 31163
CVSROOT: /opt/cvs-commit
Module name: appdb
Changes by: wineowner(a)winehq.org 2007/06/07 19:42:06
Modified files:
cron : cleanup.php
Log message:
Chris Morgan <cmorgan(a)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
+ }
}
?>