distribution::delete() should return false upon failure
This commit is contained in:
@@ -147,7 +147,7 @@ class distribution {
|
|||||||
deleted along with the last testData associated with it */
|
deleted along with the last testData associated with it */
|
||||||
if(!($this->canEdit() || (!sizeof($this->aTestingIds) &&
|
if(!($this->canEdit() || (!sizeof($this->aTestingIds) &&
|
||||||
$this->sQueued != "false")))
|
$this->sQueued != "false")))
|
||||||
return;
|
return false;
|
||||||
|
|
||||||
/* Check for associated test results */
|
/* Check for associated test results */
|
||||||
if(sizeof($this->aTestingIds))
|
if(sizeof($this->aTestingIds))
|
||||||
@@ -163,6 +163,7 @@ class distribution {
|
|||||||
if(!($hResult = query_parameters($sQuery, $this->iDistributionId)))
|
if(!($hResult = query_parameters($sQuery, $this->iDistributionId)))
|
||||||
{
|
{
|
||||||
addmsg("Error removing the Distribution!", "red");
|
addmsg("Error removing the Distribution!", "red");
|
||||||
|
return false;
|
||||||
}
|
}
|
||||||
|
|
||||||
if(!$bSilent)
|
if(!$bSilent)
|
||||||
|
|||||||
Reference in New Issue
Block a user