From 8248346630cb9b15a5621f6b8a00b59dd1561488 Mon Sep 17 00:00:00 2001 From: Chris Morgan Date: Thu, 27 Jul 2006 16:38:59 +0000 Subject: [PATCH] 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 --- include/maintainer.php | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/include/maintainer.php b/include/maintainer.php index 6096c7f..8e7c524 100644 --- a/include/maintainer.php +++ b/include/maintainer.php @@ -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.