svn commit: r728266 - /ofbiz/trunk/applications/securityext/script/org/ofbiz/securityext/securitygroup/SecurityGroupServices.xml

Author: jleroux
Date: Sat Dec 20 03:06:50 2008
New Revision: 728266

URL: http://svn.apache.org/viewvc?rev=728266&view=rev
Log:
A modified patch from Bruno Busco "bug when adding new permissions to group" (https://issues.apache.org/jira/browse/OFBIZ-2093) - OFBIZ-2093

Modified:
    ofbiz/trunk/applications/securityext/script/org/ofbiz/securityext/securitygroup/SecurityGroupServices.xml

Modified: ofbiz/trunk/applications/securityext/script/org/ofbiz/securityext/securitygroup/SecurityGroupServices.xml
URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/securityext/script/org/ofbiz/securityext/securitygroup/SecurityGroupServices.xml?rev=728266&r1=728265&r2=728266&view=diff
==============================================================================
--- ofbiz/trunk/applications/securityext/script/org/ofbiz/securityext/securitygroup/SecurityGroupServices.xml (original)
+++ ofbiz/trunk/applications/securityext/script/org/ofbiz/securityext/securitygroup/SecurityGroupServices.xml Sat Dec 20 03:06:50 2008
@@ -71,10 +71,11 @@
 
         <make-value value-name="newEntity" entity-name="SecurityGroupPermission"/>
         <set-pk-fields map-name="parameters" value-name="newEntity"/>
-        <create-value value-name="newEntity"/>
-        
+      
         <!-- clear the org.ofbiz.security.Security object's custom cache by newEntity -->
         <call-bsh><![CDATA[ org.ofbiz.security.Security.securityGroupPermissionCache.remove(newEntity); ]]></call-bsh>
+        
+        <create-value value-name="newEntity"/>        
     </simple-method>
     <simple-method method-name="removeSecurityPermissionFromSecurityGroup" short-description="Remove SecurityPermission From SecurityGroup">
         <check-permission permission="SECURITY" action="_DELETE"><fail-message message="Security Error: to run removeSecurityPermissionFromSecurityGroup you must have the SECURITY_DELETE or SECURITY_ADMIN permission"/></check-permission>
@@ -142,11 +143,11 @@
         <make-value value-name="newEntity" entity-name="ProtectedView"/>
         <set-pk-fields map-name="parameters" value-name="newEntity"/>
         <set-nonpk-fields map-name="parameters" value-name="newEntity"/>
-        
-        <create-value value-name="newEntity"/>
-        
+              
         <!-- clear the org.ofbiz.security.Security object's custom cache by newEntity -->
         <call-bsh><![CDATA[ org.ofbiz.security.Security.userLoginSecurityGroupByUserLoginId.remove(newEntity); ]]></call-bsh>
+        
+        <create-value value-name="newEntity"/>        
     </simple-method>
     <simple-method method-name="updateProtectedViewToSecurityGroup" short-description="Update ProtectedView to SecurityGroup">
         <check-permission permission="SECURITY" action="_UPDATE"><fail-message message="Security Error: to run updateProtectedViewToSecurityGroup you must have the SECURITY_UPDATE or SECURITY_ADMIN permission"/></check-permission>