[Bedework-commit] access r56 - trunk/src/edu/rpi/cmt/access

[svnadmin at bedework.org]

Author: douglm
Date: 2007-05-06 20:39:49 -0400 (Sun, 06 May 2007)
New Revision: 56

Modified:
   trunk/src/edu/rpi/cmt/access/Privileges.java
Log:
Change to Privileges to correct setting of allow/deny access - only set denied if privilege is currently unspecified.

Correct multiget response when status is non-2xx

Modified: trunk/src/edu/rpi/cmt/access/Privileges.java
===================================================================
--- trunk/src/edu/rpi/cmt/access/Privileges.java	2007-04-20 21:37:42 UTC (rev 55)
+++ trunk/src/edu/rpi/cmt/access/Privileges.java	2007-05-07 00:39:49 UTC (rev 56)
@@ -235,9 +235,11 @@
    *  privileges.
    */
   private static void setState(char[] states, Privilege p, boolean denial) {
+    // XXX Should we only set either way of the access is unspecified?
     if (!denial) {
       states[p.getIndex()] = allowed;
-    } else {
+//    } else {
+    } else if (states[p.getIndex()] == unspecified) {
       states[p.getIndex()] = denied;
     }