Changeset 19639

Timestamp:

02/03/12 11:25:07 (4 months ago)

Author:

sys

Message:

extPoints.hasPermission() : isolement du controle des systemRights de celui des roles.

File:

• trunk/Xul_Commons/content/Commons/ExtPoints.jsm (modified) (7 diffs)

trunk/Xul_Commons/content/Commons/ExtPoints.jsm

@@ -686 +686 @@
          */
         hasPermission : function(pPerms, pRoleNames, pSystemRights){
-                //Si aucun role de paramétré dans l'extPoints, on ets dans un contexte hors getsion de droits, on valide la perm.
+                //Init perms
+                var vPerm, vOffset;
+                if(Array.isArray(pPerms)) {
+                        vPerm = pPerms[0];
+                        vOffset = 1;
+                } else {
+                        vPerm = pPerms;
+                }
+                
+                //System rights
+                if(pSystemRights != null) {
+                        while(vPerm) {
+                                //log.info("hasPermission:::"+pPerms+" for roles:::"+pRoleNames);
+                                var vPermRi = vPerm;
+                                while(vPermRi) {
+                                        if(vPermRi.getSystemRights()>0 && ~(~vPermRi.getSystemRights() | pSystemRights)!=0) return false;
+                                        vPermRi = vPermRi.getParent();
+                                }
+                                vPerm = vOffset>0 ? pPerms[vOffset++] : null;
+                        }
+                        //reinit des perm pour les roles.
+                        if(Array.isArray(pPerms)) {
+                                vPerm = pPerms[0];
+                                vOffset = 1;
+                        } else {
+                                vPerm = pPerms;
+                                vOffset = 0;
+                        }
+                }
+                
+                //Si aucun role de paramétré dans l'extPoints, on est dans un contexte hors gestion de droits, on valide la perm.
                 if (!this.hasRoles()) return true;
                 if(!pRoleNames || pRoleNames.length==0) return false;
@@ -696 +726 @@
                                 return vP2 - vP1;
                         });
-                }
-                var vPerm, vOffset;
-                if(Array.isArray(pPerms)) {
-                        vPerm = pPerms[0];
-                        vOffset==1;
-                } else {
-                        vPerm = pPerms;
                 }
                 while(vPerm) {
@@ -840 +863 @@
          * Teste s'il existe des rôles
          * 
-         * @returns true si au moins un role est défini 
+         * @return true si au moins un role est défini 
          */
         hasRoles : function() {
@@ -853 +876 @@
          * Retourne la liste des rôles définies
          * 
-         * @returns tableau des noms de rôles
+         * @return tableau des noms de rôles
          */
         getRoleNames : function () {
@@ -1234 +1257 @@
                 }
                 if(pRolePermList.deny) for each(var vPermName in pRolePermList.deny) {
-                        var vPRE = vR.fPREs[vPermName];
                         if(pOverwriteParent || this.fParentExtPoints.getRolePermLevel(vRoleName, vPermName) < pLevelAuthority) {
+                                var vPRE = vR.fPREs[vPermName];
                                 if(!vPRE || vPRE.fLevel < pLevelAuthority) {
                                         vR.fPREs[vPermName] = {fLevel : pLevelAuthority, fValue : false}
@@ -1242 +1265 @@
                 }
                 if(pRolePermList.allow) for each(var vPermName in pRolePermList.allow) {
-                        var vPRE = vR.fPREs[vPermName];
                         if(pOverwriteParent || this.fParentExtPoints.getRolePermLevel(vRoleName, vPermName) < pLevelAuthority) {
+                                var vPRE = vR.fPREs[vPermName];
                                 if(!vPRE || vPRE.fLevel < pLevelAuthority) {
                                         vR.fPREs[vPermName] = {fLevel : pLevelAuthority, fValue : true}
@@ -1250 +1273 @@
                 }
                 if(pRolePermList.erase) for each(var vPermName in pRolePermList.erase) {
-                        var vPRE = vR.fPREs[vPermName];
                         if(pOverwriteParent || this.fParentExtPoints.getRolePermLevel(vRoleName, vPermName) < pLevelAuthority) {
+                                var vPRE = vR.fPREs[vPermName];
                                 if(!vPRE || vPRE.fLevel < pLevelAuthority) {
                                         vR.fPREs[vPermName] = {fLevel : pLevelAuthority, fValue : null}