appdb/include maintainer.php

[WineHQ]

ChangeSet ID:	26881
CVSROOT:	/opt/cvs-commit
Module name:	appdb
Changes by:	[e-mail]	2006/07/27 11:38:59

Modified files:
    include        : maintainer.php

Log message:
    Chris Morgan 
    Simplify maintainer::getMaintainersForAppIdVersionId() query to be the same as the query used prior to the maintainer
    changes.  Fixes display of loads of maintainers due to permutations in a join that wasn't properly defined

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

Old revision  New revision  Changes     Path
 1.14          1.15          +2 -5       appdb/include/maintainer.php

Index: appdb/include/maintainer.php
diff -u -p appdb/include/maintainer.php:1.14 appdb/include/maintainer.php:1.15
--- appdb/include/maintainer.php:1.14	27 Jul 2006 16:38:59 -0000
+++ appdb/include/maintainer.php	27 Jul 2006 16:38:59 -0000
@@ -303,11 +303,8 @@ class maintainer

if($iVersionId)
         {
-            $hResult = query_parameters("SELECT appMaintainers.userId 
-                                 FROM appMaintainers, appVersion
-                                 WHERE appVersion.appId = appMaintainers.appId 
-                                 AND appVersion.versionId = '?' AND appMaintainers.queued = 'false'",
-                                        $iVersionId);
+            $hResult = query_parameters("SELECT userId from appMaintainers WHERE appMaintainers.queued = 'false' AND ".
+                                        "appMaintainers.versionId = '?'", $iVersionId);
         } 
         /*
          * If versionId was not supplied we fetch supermaintainers of application and maintainer of all versions.