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