Project

General

Profile

« Previous | Next » 

Revision 3512

Added by barteau about 17 years ago

Fixed bug where the Permission Order was always being set to 'allowFirst'.
It was comparing against the constant AccessControlInterface.DENYFIRST with the variable name surrounded by quotes. Removed the quotes and updated comments.

View differences:

src/edu/ucsb/nceas/metacat/AccessControlForSingleFile.java
102 102 
      }
103 103 
      // check permission order
104 104 
      permOrder = myPermOrder;
105 
      //if permission order is not allowFirst or denyFirst, assing it dneyFirst
106 
      if (permOrder == null
107 
          || !permOrder.equals("AccessControlInterface.DENYFIRST"))
108 
      {
105
106 
      // If permission order is not allowFirst or denyFirst, default to denyFirst.
107 
      if (permOrder == null || !permOrder.equals(AccessControlInterface.DENYFIRST)) {
109 108 
        permOrder = AccessControlInterface.ALLOWFIRST;
110 109 
      }
110
111 111 
      //debugMessage
112 112 
      logMetacat.info("docid in AccessControlForSingleFiel: " +
113 113 
                                docId);

Also available in: Unified diff