minor

dprzybyl · dprzybyl · commit 4e3ea9d53d72 · 2024-03-23T20:44:05.000+01:00
diff --git a/app/aem/actions.main/src/main/java/com/cognifide/apm/main/permissions/Restrictions.java b/app/aem/actions.main/src/main/java/com/cognifide/apm/main/permissions/Restrictions.java
@@ -75,7 +75,8 @@ public Map<String, Value> getSingleValueRestrictions(ValueFactory valueFactory)
         if (entry.getValue() instanceof String) {
           value = (String) entry.getValue();
         } else {
-          value = ((List<String>) entry.getValue()).get(0);
+          List<String> values = (List<String>) entry.getValue();
+          value = values.isEmpty() ? "" : values.get(0);
         }
         addRestriction(valueFactory, result, entry.getKey(), value);
       }
@@ -112,7 +113,8 @@ public Map<String, Value[]> getMultiValueRestrictions(ValueFactory valueFactory)
       if (isMultiValue(entry)) {
         List<String> values;
         if (entry.getValue() instanceof String) {
-          values = Collections.singletonList((String) entry.getValue());
+          String value = (String) entry.getValue();
+          values = value.isEmpty() ? Collections.emptyList() : Collections.singletonList(value);
         } else {
           values = (List<String>) entry.getValue();
         }

Original file line number	Diff line number	Diff line change
`@@ -75,7 +75,8 @@ public Map<String, Value> getSingleValueRestrictions(ValueFactory valueFactory)`
`75`	`75`	`if (entry.getValue() instanceof String) {`
`76`	`76`	`value = (String) entry.getValue();`
`77`	`77`	`} else {`
`78`		`- value = ((List<String>) entry.getValue()).get(0);`
	`78`	`+ List<String> values = (List<String>) entry.getValue();`
	`79`	`+ value = values.isEmpty() ? "" : values.get(0);`
`79`	`80`	`}`
`80`	`81`	`addRestriction(valueFactory, result, entry.getKey(), value);`
`81`	`82`	`}`
`@@ -112,7 +113,8 @@ public Map<String, Value[]> getMultiValueRestrictions(ValueFactory valueFactory)`
`112`	`113`	`if (isMultiValue(entry)) {`
`113`	`114`	`List<String> values;`
`114`	`115`	`if (entry.getValue() instanceof String) {`
`115`		`- values = Collections.singletonList((String) entry.getValue());`
	`116`	`+ String value = (String) entry.getValue();`
	`117`	`+ values = value.isEmpty() ? Collections.emptyList() : Collections.singletonList(value);`
`116`	`118`	`} else {`
`117`	`119`	`values = (List<String>) entry.getValue();`
`118`	`120`	`}`