appdb/include distribution.php

[WineHQ]

ChangeSet ID:	31297
CVSROOT:	/opt/cvs-commit
Module name:	appdb
Changes by:	wineowner at winehq.org	2007/07/30 22:31:07

Modified files:
	include        : distribution.php 

Log message:
	Chris Morgan <cmorgan at alum.wpi.edu>
	distribution::delete() should return false upon failure

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

Old revision  New revision  Changes     Path
 1.39          1.40          +2 -1       appdb/include/distribution.php

Index: appdb/include/distribution.php
diff -u -p appdb/include/distribution.php:1.39 appdb/include/distribution.php:1.40
--- appdb/include/distribution.php:1.39	31 Jul 2007  3:31: 7 -0000
+++ appdb/include/distribution.php	31 Jul 2007  3:31: 7 -0000
@@ -147,7 +147,7 @@ class distribution {
            deleted along with the last testData associated with it */
         if(!($this->canEdit() || (!sizeof($this->aTestingIds) &&
                 $this->sQueued != "false")))
-            return;
+            return false;
 
         /* Check for associated test results */
         if(sizeof($this->aTestingIds))
@@ -163,6 +163,7 @@ class distribution {
         if(!($hResult = query_parameters($sQuery, $this->iDistributionId)))
         {
             addmsg("Error removing the Distribution!", "red");
+            return false;
         }
 
         if(!$bSilent)