Author: taylor
Date: Tue Oct 30 14:25:12 2007
New Revision: 590471
URL: http://svn.apache.org/viewvc?rev=590471&view=rev
Log:
intermittently the pref is coming back as null, even though its default value
is ""
I can't reproduce here, but I did debug over netmeeting and reproduce it
Modified:
portals/jetspeed-2/trunk/applications/j2-admin/src/java/org/apache/jetspeed/portlets/security/users/UserDetailsPortlet.java
Modified:
portals/jetspeed-2/trunk/applications/j2-admin/src/java/org/apache/jetspeed/portlets/security/users/UserDetailsPortlet.java
URL:
http://svn.apache.org/viewvc/portals/jetspeed-2/trunk/applications/j2-admin/src/java/org/apache/jetspeed/portlets/security/users/UserDetailsPortlet.java?rev=590471&r1=590470&r2=590471&view=diff
==============================================================================
---
portals/jetspeed-2/trunk/applications/j2-admin/src/java/org/apache/jetspeed/portlets/security/users/UserDetailsPortlet.java
(original)
+++
portals/jetspeed-2/trunk/applications/j2-admin/src/java/org/apache/jetspeed/portlets/security/users/UserDetailsPortlet.java
Tue Oct 30 14:25:12 2007
@@ -1164,7 +1164,7 @@
}
String requiredRole =
actionRequest.getPreferences().getValue("requiredRole", "");
- if (!requiredRole.equals("") && user != null)
+ if (!SecurityUtil.isEmpty(requiredRole) && user != null)
{
roleManager.addRoleToUser(userName, requiredRole);
}
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
