appdb/include distribution.php

[WineHQ]

ChangeSet ID:	31352
CVSROOT:	/opt/cvs-commit
Module name:	appdb
Changes by:	[e-mail]	2007/08/26 22:46:03

Modified files:
    include        : distribution.php

Log message:
    Chris Morgan 
    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