appdb/include maintainer.php

[WineHQ]

ChangeSet ID:	31129
CVSROOT:	/opt/cvs-commit
Module name:	appdb
Changes by:	wineowner at winehq.org	2007/05/13 11:54:33

Modified files:
	include        : maintainer.php 

Log message:
	Alexander Nicolaysen Sørnes <alex at thehandofagony.com>
	Don't display supermaintainers twice on maintainer processing screen. When printing the list of
	non-super maintainers, only print the entry if the user is not a super maintainer

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

Old revision  New revision  Changes     Path
 1.41          1.42          +6 -1       appdb/include/maintainer.php

Index: appdb/include/maintainer.php
diff -u -p appdb/include/maintainer.php:1.41 appdb/include/maintainer.php:1.42
--- appdb/include/maintainer.php:1.41	13 May 2007 16:54:33 -0000
+++ appdb/include/maintainer.php	13 May 2007 16:54:33 -0000
@@ -537,7 +537,12 @@ class maintainer
             while(list($index, $iUserId) = each($aOtherMaintainers))
             {
                 $oUser = new User($iUserId);
-                echo "$oUser->sRealname<br />\n";
+
+                // because Version::getMaintainersUserIds() includes super maintainers
+                // we need to exclude these from the list of maintainers that we are
+                // building
+                if(!maintainer::isUserSuperMaintainer($oUser, $oVersion->iAppId))
+                    echo "$oUser->sRealname<br />\n";
             }
         }