Scarab commit: svn commit: r11063 - trunk/src/java/org/tigris/scarab/om/ScarabUserImpl.java

Author: dabbous
Date: 2011-12-21 19:54:31-0800
New Revision: 11063

Modified:
   trunk/src/java/org/tigris/scarab/om/ScarabUserImpl.java

Log:
fixed: Delete unwanted modules from the list.

Modified: trunk/src/java/org/tigris/scarab/om/ScarabUserImpl.java
Url: http://scarab.tigris.org/source/browse/scarab/trunk/src/java/org/tigris/scarab/om/ScarabUserImpl.java?view=diff&pathrev=11063&r1=11062&r2=11063
==============================================================================
--- trunk/src/java/org/tigris/scarab/om/ScarabUserImpl.java	(original)
+++ trunk/src/java/org/tigris/scarab/om/ScarabUserImpl.java	2011-12-21 19:54:31-0800
@@ -419,11 +419,12 @@
             
             try
             {
-                List scarabModules = ScarabModulePeer.doSelect(crit);
+                List initialScarabModules = ScarabModulePeer.doSelect(crit);
                 // check for permissions in global, if so get all modules
-                for (int i=scarabModules.size()-1; i>=0; i--) 
+                List<ScarabModule> scarabModules = new ArrayList<ScarabModule>();
+                for (int i=initialScarabModules.size()-1; i>=0; i--) 
                 {
-                    ScarabModule module = (ScarabModule)scarabModules.get(i);
+                    ScarabModule module = (ScarabModule)initialScarabModules.get(i);
                     Integer moduleId = module.getModuleId();
                     boolean hasRoles = this.hasAnyRoleIn(module);
                     String moduleName = module.getName();
@@ -459,6 +460,7 @@
                             scarabModules = ScarabModulePeer.doSelect(crit);
                             break;
                         }
+                        scarabModules.add(0, module);
                     }
                 }

------------------------------------------------------
http://scarab.tigris.org/ds/viewMessage.do?dsForumId=3577&dsMessageId=2898964