From 8d61f9b497c07422cb41377f0c81b1c0358efdbf Mon Sep 17 00:00:00 2001 From: Chris Morgan Date: Tue, 31 Jul 2007 03:31:07 +0000 Subject: [PATCH] distribution::delete() should return false upon failure --- include/distribution.php | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/include/distribution.php b/include/distribution.php index 7fff8b5..bcc868d 100644 --- a/include/distribution.php +++ b/include/distribution.php @@ -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)