appdb/include maintainer.php
WineHQ
wineowner at wine.codeweavers.com
Thu Jul 27 11:38:59 CDT 2006
ChangeSet ID: 26881
CVSROOT: /opt/cvs-commit
Module name: appdb
Changes by: wineowner at winehq.org 2006/07/27 11:38:59
Modified files:
include : maintainer.php
Log message:
Chris Morgan <cmorgan at alum.wpi.edu>
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.
More information about the wine-cvs
mailing list