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
This commit is contained in:
@@ -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.
|
||||
|
||||
Reference in New Issue
Block a user