[cvs] expresso commit by lhamel: comments

JCorporate Ltd jcorp at jcorporate.com
Thu Jul 7 02:55:00 UTC 2005


Log Message:
-----------
comments

Modified Files:
--------------
    expresso/expresso-web/WEB-INF/src/com/jcorporate/expresso/core/dbobj:
        RowSecuredDBObject.java

Revision Data
-------------
Index: RowSecuredDBObject.java
===================================================================
RCS file: /home/javacorp/.cvs/expresso/expresso/expresso-web/WEB-INF/src/com/jcorporate/expresso/core/dbobj/RowSecuredDBObject.java,v
retrieving revision 1.46
retrieving revision 1.47
diff -Lexpresso-web/WEB-INF/src/com/jcorporate/expresso/core/dbobj/RowSecuredDBObject.java -Lexpresso-web/WEB-INF/src/com/jcorporate/expresso/core/dbobj/RowSecuredDBObject.java -u -r1.46 -r1.47
--- expresso-web/WEB-INF/src/com/jcorporate/expresso/core/dbobj/RowSecuredDBObject.java
+++ expresso-web/WEB-INF/src/com/jcorporate/expresso/core/dbobj/RowSecuredDBObject.java
@@ -65,10 +65,12 @@
 
 import com.jcorporate.expresso.core.db.DBConnection;
 import com.jcorporate.expresso.core.db.DBException;
+import com.jcorporate.expresso.core.db.exception.DBRecordNotFoundException;
 import com.jcorporate.expresso.core.registry.MutableRequestRegistry;
 import com.jcorporate.expresso.core.registry.RequestRegistry;
-import com.jcorporate.expresso.core.security.User;
 import com.jcorporate.expresso.core.security.ReadOnlyUser;
+import com.jcorporate.expresso.core.security.SuperUser;
+import com.jcorporate.expresso.core.security.User;
 import com.jcorporate.expresso.services.dbobj.RowGroupPerms;
 import com.jcorporate.expresso.services.dbobj.RowPermissions;
 import com.jcorporate.expresso.services.dbobj.UserGroup;
@@ -77,8 +79,6 @@
 import java.util.Collection;
 import java.util.Iterator;
 import java.util.List;
-import com.jcorporate.expresso.core.security.SuperUser;
-import com.jcorporate.expresso.core.db.exception.DBRecordNotFoundException;
 
 
 /**
@@ -333,7 +333,8 @@
     /**
      * set the permissions for this object; group bits are ignored; only owner
      * & "other" permissions apply with this method owner id is taken from
-     * getRequestingUser()
+     * getRequestingUser();
+     * this actually persists to db
      *
      * @param perm permissions to set
      * @throws DBException upon database communication error
@@ -969,7 +970,8 @@
     }
 
     /**
-     * set permissions as indicated and persist, either calling db actions add() or update()
+     * set permissions as indicated and persist, either calling db actions add() or update();
+     * this actually persists to db
      *
      * @param owner new owner
      * @param group new group for group perms


More information about the cvs mailing list