appdb/include distribution.php

[WineHQ]

ChangeSet ID:	31343
CVSROOT:	/opt/cvs-commit
Module name:	appdb
Changes by:	wineowner(a)winehq.org	2007/08/23 22:49:33

Modified files:
	include        : distribution.php 

Log message:
	Chris Morgan <cmorgan(a)alum.wpi.edu>
	Enable deletion of a distribution's test results along with the distribution, if the user has
	admin privileges.

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

Old revision  New revision  Changes     Path
 1.42          1.43          +9 -4       appdb/include/distribution.php

Index: appdb/include/distribution.php
diff -u -p appdb/include/distribution.php:1.42 appdb/include/distribution.php:1.43
--- appdb/include/distribution.php:1.42	24 Aug 2007  3:49:33 -0000
+++ appdb/include/distribution.php	24 Aug 2007  3:49:33 -0000
@@ -151,11 +151,16 @@ class distribution {
                 $this->sQueued != "false")))
             return false;
 
-        /* Check for associated test results */
-        if(sizeof($this->aTestingIds))
+        // if the distribution has test results only enable an admin to delete
+        // the distribution
+        if(sizeof($this->aTestingIds) && !$_SESSION['current']->hasPriv("admin"))
+          return FALSE;
+
+        // delete any test results this distribution has
+        foreach($this->aTestingIds as $iTestId)
         {
-            addmsg("This distribution still has associated test results", "red");
-            return FALSE;
+          $oTestData = new TestData($iTestId);
+          $oTestData->delete();
         }
 
         // now delete the Distribution