appdb/cron cleanup.php
WineHQ
wineowner at wine.codeweavers.com
Sun Jun 10 12:13:49 CDT 2007
ChangeSet ID: 31172
CVSROOT: /opt/cvs-commit
Module name: appdb
Changes by: wineowner at winehq.org 2007/06/10 12:13:49
Modified files:
cron : cleanup.php
Log message:
Chris Morgan <cmorgan at alum.wpi.edu>
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
+// }
}
?>
More information about the wine-cvs
mailing list