You are viewing a plain text version of this content. The canonical link for it is here.
Posted to jetspeed-dev@portals.apache.org by we...@apache.org on 2003/11/03 19:25:01 UTC
cvs commit: jakarta-jetspeed/src/java/org/apache/jetspeed/modules/actions/portlets/security RoleUpdateAction.java
weaver 2003/11/03 10:25:01
Modified: src/java/org/apache/jetspeed/modules/actions/portlets/security
RoleUpdateAction.java
Log:
Use JetspeedRoleFactory instead of directly using the
implementation class's constructor (not pluggable)
Revision Changes Path
1.11 +5 -4 jakarta-jetspeed/src/java/org/apache/jetspeed/modules/actions/portlets/security/RoleUpdateAction.java
Index: RoleUpdateAction.java
===================================================================
RCS file: /home/cvs/jakarta-jetspeed/src/java/org/apache/jetspeed/modules/actions/portlets/security/RoleUpdateAction.java,v
retrieving revision 1.10
retrieving revision 1.11
diff -u -r1.10 -r1.11
--- RoleUpdateAction.java 23 Jul 2003 19:50:13 -0000 1.10
+++ RoleUpdateAction.java 3 Nov 2003 18:25:01 -0000 1.11
@@ -63,6 +63,7 @@
import org.apache.turbine.util.DynamicURI;
// jetspeed om
+import org.apache.jetspeed.om.security.JetspeedRoleFactory;
import org.apache.jetspeed.om.security.Role;
import org.apache.jetspeed.om.security.BaseJetspeedRole;
// jetspeed services
@@ -208,9 +209,9 @@
//
// generate a new role
- //
- // TODO: replace with Role factory
- role = new BaseJetspeedRole();
+ //
+
+ role = JetspeedRoleFactory.getInstance();
role.setName(name);
//
---------------------------------------------------------------------
To unsubscribe, e-mail: jetspeed-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: jetspeed-dev-help@jakarta.apache.org