[Fusionforge-commits] FusionForge branch 6.0 updated. 51bad1459535b14dc09f52d9c681b58868b7482a

Sylvain Beucler beuc-inria at fusionforge.org
Fri Feb 20 17:02:53 CET 2015


This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "FusionForge".

The branch, 6.0 has been updated
       via  51bad1459535b14dc09f52d9c681b58868b7482a (commit)
      from  875dcc88148919928ffb61bc82ad1eee1f5d5fd8 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit 51bad1459535b14dc09f52d9c681b58868b7482a
Author: Sylvain Beucler <sylvain.beucler at inria.fr>
Date:   Fri Feb 20 17:01:03 2015 +0100

    RBAC: in the dropdown list of external roles, place Anonymous and Logged roles first
    There's much confusion due to external projects creating their own "Anon" roles.
    Plus it makes sense to have the site-wide roles first.

diff --git a/src/common/include/RBAC.php b/src/common/include/RBAC.php
index 155e622..95ef9f5 100644
--- a/src/common/include/RBAC.php
+++ b/src/common/include/RBAC.php
@@ -1356,9 +1356,9 @@ class RoleComparator {
 		$ap = $a->getHomeProject() ;
 		$bp = $b->getHomeProject() ;
 		if ($ap == NULL && $bp != NULL) {
-			return 1 ;
-		} elseif ($ap != NULL && $bp == NULL) {
 			return -1 ;
+		} elseif ($ap != NULL && $bp == NULL) {
+			return 1 ;
 		} elseif ($ap == NULL && $bp == NULL) {
 			$tmp = strcoll ($a->getName(), $b->getName()) ;
 			return $tmp ;

-----------------------------------------------------------------------

Summary of changes:
 src/common/include/RBAC.php |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)


hooks/post-receive
-- 
FusionForge



More information about the Fusionforge-commits mailing list