Changeset 31544

Timestamp:

2017-03-28T18:17:21+13:00 (7 years ago)

Author:

ak19

Message:

Dr Bainbridge found oversight in recently committed code: needed to remember to handle the case when the filterFields is null, as happens when a user has edit permissions on a collection and can thus set any and all meta.

File:

• main/trunk/greenstone3/src/java/org/greenstone/gsdl3/service/GS2Construct.java (modified) (1 diff)

main/trunk/greenstone3/src/java/org/greenstone/gsdl3/service/GS2Construct.java

@@ -1025 +1025 @@
     protected boolean isAllowedToSetMeta(String metaname, Pattern allowedMetaFieldsPattern) 
     {
+    if(allowedMetaFieldsPattern == null) { // null when user has edit permissions, so they can set any meta
+        logger.info("### User has permissions ot set any meta.");
+        return true;
+    }
     if(metaname == null) {
         logger.info("### Can't check null metaname against pattern");