[cvs] expresso commit by lhamel: use string constant

JCorporate Ltd jcorp at jcorp2.servlets.net
Sat Nov 20 13:48:15 PST 2004


Log Message:
-----------
use string constant

Modified Files:
--------------
    expresso/expresso-web/WEB-INF/src/com/jcorporate/expresso/services/dbobj:
        UserGroup.java

Revision Data
-------------
Index: UserGroup.java
===================================================================
RCS file: /home/javacorp/.cvs/expresso/expresso/expresso-web/WEB-INF/src/com/jcorporate/expresso/services/dbobj/UserGroup.java,v
retrieving revision 1.24
retrieving revision 1.25
diff -Lexpresso-web/WEB-INF/src/com/jcorporate/expresso/services/dbobj/UserGroup.java -Lexpresso-web/WEB-INF/src/com/jcorporate/expresso/services/dbobj/UserGroup.java -u -r1.24 -r1.25
--- expresso-web/WEB-INF/src/com/jcorporate/expresso/services/dbobj/UserGroup.java
+++ expresso-web/WEB-INF/src/com/jcorporate/expresso/services/dbobj/UserGroup.java
@@ -71,6 +71,7 @@
 import com.jcorporate.expresso.core.i18n.Messages;
 
 import java.util.Iterator;
+import java.util.List;
 import java.util.Vector;
 
 
@@ -88,6 +89,7 @@
     public static final String GROUP_DESCRIPTION = "Descrip";
     public static final int GROUP_NAME_MAX_LEN = 10;
     public static final int GROUP_DESCRIP_MAX_LEN = 80;
+    public static final String TABLE_NAME = "USERROLES";
 
     /**
      * used as default group for all
@@ -206,7 +208,7 @@
      */
     protected synchronized void setupFields()
             throws DBException {
-        setTargetTable("USERROLES");
+        setTargetTable(TABLE_NAME);
         setDescription("DBuserGroup");
         setCharset("ISO-8859-1");
         addField(GROUP_NAME_FIELD, "char", GROUP_NAME_MAX_LEN, false, "groupName");
@@ -309,4 +311,13 @@
         return getField(GROUP_DESCRIPTION);
     }
 
+    /**
+     * convenience method
+     * @return list of all groups
+     * @throws DBException upon error
+     */
+    public static List getAllGroups() throws DBException {
+        UserGroup search = new UserGroup();
+        return search.searchAndRetrieveList();
+    }
 } /* UserGroup */


More information about the cvs mailing list