From aefbc6de57f1576a09f19372d05f74fd2f03e991 Mon Sep 17 00:00:00 2001
From: Chris Morgan <cmorgan@alum.wpi.edu>
Date: Thu, 24 May 2007 02:29:44 +0000
Subject: [PATCH] Add removeScreenshotsWithMissingFiles() to the nightly cron
 script. This will close a hole where we could potentially have screenshots in
 the database that don't have thumbnails or images.

---
 cron/cleanup.php | 57 ++++++++++++++++++++++++++++++++++++++++++++++++
 1 file changed, 57 insertions(+)

diff --git a/cron/cleanup.php b/cron/cleanup.php
index f002a5a..1fa6c1a 100644
--- a/cron/cleanup.php
+++ b/cron/cleanup.php
@@ -81,6 +81,8 @@ orphanSessionListCheck();
 /* report error log entries to admins and flush the error log after doing so */
 reportErrorLogEntries();
 
+/* remove screenshots that are missing their screenshot and thumbnail files */
+removeScreenshotsWithMissingFiles();
 
 /* Users that are unwarned and inactive since $iMonths */
 function unwarnedAndInactiveSince($iMonths)
@@ -236,4 +238,59 @@ function reportErrorLogEntries()
     error_log::flush();
 }
 
+// returns an array of iScreenshotIds of screenshots that are
+// missing their files
+function getMissingScreenshotArray()
+{
+  $aMissingScreenshotIds = array();
+
+  // retrieve all screenshots, not queued, not rejected
+  $hResult = Screenshot::objectGetEntries(false, false);
+
+  // go through each screenshot
+  while($oRow = mysql_fetch_object($hResult))
+  {
+    $iScreenshotId = $oRow->id;
+    $oScreenshot = new Screenshot($iScreenshotId);
+
+    // load the screenshot and thumbnail
+    $oScreenshot->load_image(true);
+    $oScreenshot->load_image(false);
+
+    // are the screenshot and thumbnail images not loaded? if so
+    // add this screenshot id to the array
+    if(!$oScreenshot->oScreenshotImage->isLoaded() &&
+       !$oScreenshot->oThumbnailImage->isLoaded())
+    {
+      // add the screenshot id to the array
+      $aMissingScreenshotIds[] = $iScreenshotId;
+    }
+  }
+
+  return $aMissingScreenshotIds;
+}
+
+function removeScreenshotsWithMissingFiles()
+{
+    $aMissingScreenshotIds = getMissingScreenshotArray();
+
+    // 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";
+
+    $sSubject = "Screenshots deleted\r\n";
+
+    $sEmail = User::get_notify_email_address_list(null, null); /* get list admins */
+    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
+    //    }
+}
 ?>