appdb/include distribution.php

WineHQ wineowner at wine.codeweavers.com
Sun Aug 26 22:46:03 CDT 2007


ChangeSet ID:	31352
CVSROOT:	/opt/cvs-commit
Module name:	appdb
Changes by:	wineowner at winehq.org	2007/08/26 22:46:03

Modified files:
	include        : distribution.php 

Log message:
	Chris Morgan <cmorgan at alum.wpi.edu>
	Skip deleting a distributions test results if there are no test results for the distribution

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

Old revision  New revision  Changes     Path
 1.44          1.45          +6 -3       appdb/include/distribution.php

Index: appdb/include/distribution.php
diff -u -p appdb/include/distribution.php:1.44 appdb/include/distribution.php:1.45
--- appdb/include/distribution.php:1.44	27 Aug 2007  3:46: 3 -0000
+++ appdb/include/distribution.php	27 Aug 2007  3:46: 3 -0000
@@ -157,10 +157,13 @@ class distribution {
           return FALSE;
 
         // delete any test results this distribution has
-        foreach($this->aTestingIds as $iTestId)
+        if($this->aTestingIds)
         {
-          $oTestData = new TestData($iTestId);
-          $oTestData->delete();
+            foreach($this->aTestingIds as $iTestId)
+            {
+                $oTestData = new TestData($iTestId);
+                $oTestData->delete();
+            }
         }
 
         // now delete the Distribution 



More information about the wine-cvs mailing list