appdb/ admin/adminMaintainerQueue.php include/ ...

[WineHQ]

ChangeSet ID:	26327
CVSROOT:	/opt/cvs-commit
Module name:	appdb
Changes by:	wineowner(a)winehq.org	2006/07/07 11:44:41

Modified files:
	admin          : adminMaintainerQueue.php 
	include        : maintainer.php version.php 

Log message:
	Chris Morgan <cmorgan(a)alum.wpi.edu>
	Move getMaintainersUserIds...() to version::getMaintainersUserIds() and fixup callers of this function

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

Old revision  New revision  Changes     Path
 1.29          1.30          +2 -1       appdb/admin/adminMaintainerQueue.php
 1.11          1.12          +0 -24      appdb/include/maintainer.php
 1.66          1.67          +24 -2      appdb/include/version.php

Index: appdb/admin/adminMaintainerQueue.php
diff -u -p appdb/admin/adminMaintainerQueue.php:1.29 appdb/admin/adminMaintainerQueue.php:1.30
--- appdb/admin/adminMaintainerQueue.php:1.29	7 Jul 2006 16:44:41 -0000
+++ appdb/admin/adminMaintainerQueue.php	7 Jul 2006 16:44:41 -0000
@@ -68,7 +68,8 @@ if ($aClean['sSub'])
 
         $firstDisplay = true; /* if false we need to fix up table rows appropriately */
 
-        $other_users = getMaintainersUserIdsFromAppIdVersionId($oRow->versionId);
+        $oVersion = new Version($oRow->versionId);
+        $other_users = $oVersion->getMaintainersUserIds();
         if($other_users)
         {
             $foundMaintainers = true;
Index: appdb/include/maintainer.php
diff -u -p appdb/include/maintainer.php:1.11 appdb/include/maintainer.php:1.12
--- appdb/include/maintainer.php:1.11	7 Jul 2006 16:44:41 -0000
+++ appdb/include/maintainer.php	7 Jul 2006 16:44:41 -0000
@@ -4,30 +4,6 @@
 /*****************************/
 
 /*
- * get the userIds of maintainers for a versionId
- */
-function getMaintainersUserIdsFromAppIdVersionId($versionId)
-{
-    $retval = array();
-
-    /* early out if the versionId isn't valid */
-    if($versionId == 0)
-        return $retval;
-    
-    $sQuery = "SELECT userId FROM ".
-        "appMaintainers WHERE versionId = '?';";
-    $hResult = query_parameters($sQuery, $versionId);
-    $c = 0;
-    while($oRow = mysql_fetch_object($hResult))
-    {
-        $retval[$c] = $oRow->userId;
-        $c++;
-    }
-
-    return $retval;
-}
-
-/*
  * get the userIds of super maintainers for this appId
  */
 function getSuperMaintainersUserIdsFromAppId($appId)
Index: appdb/include/version.php
diff -u -p appdb/include/version.php:1.66 appdb/include/version.php:1.67
--- appdb/include/version.php:1.66	7 Jul 2006 16:44:41 -0000
+++ appdb/include/version.php	7 Jul 2006 16:44:41 -0000
@@ -32,7 +32,7 @@ class Version {
     var $aTestingIds;         // an array that contains the testingId of every test result linked to this version
     var $aMonitorIds;         // an array that contains the monitorId of every monitor linked to this version
 
-    /**    
+    /**
      * constructor, fetches the data.
      */
     function Version($iVersionId = null)
@@ -677,7 +677,7 @@ class Version {
         // display all maintainers of this application
         echo "<tr class=\"color0\"><td align=\"left\" colspan=\"2\"><b>Maintainers of this version:</b>\n";
         echo "<table width=\"250\" border=\"0\">";
-        $aMaintainers = getMaintainersUserIdsFromAppIdVersionId($this->iVersionId);
+        $aMaintainers = $this->getMaintainersUserIds();
         $aSupermaintainers = getSuperMaintainersUserIdsFromAppId($this->iAppId);
         $aAllMaintainers = array_merge($aMaintainers,$aSupermaintainers);
         $aAllMaintainers = array_unique($aAllMaintainers);
@@ -926,6 +926,28 @@ class Version {
             echo html_frame_end("Click the Version Name to view the details of that Version");
         }
     }
+
+    /* returns the maintainers of this version in an array */
+    function getMaintainersUserIds()
+    {
+        $aMaintainers = array();
+
+        /* early out if the versionId isn't valid */
+        if($this->iVersionId == 0)
+            return $aMaintainers;
+    
+        $sQuery = "SELECT userId FROM ".
+            "appMaintainers WHERE versionId = '?';";
+        $hResult = query_parameters($sQuery, $this->iVersionId);
+        $iCount = 0;
+        while($oRow = mysql_fetch_object($hResult))
+        {
+            $aMaintainers[$iCount] = $oRow->userId;
+            $iCount++;
+        }
+
+        return $aMaintainers;
+    }
 }
 
 ?>