Avoid nullpointer when manage is not behaving

OpenConext · Jan 9, 2024 · b99121a · b99121a
1 parent 367e7de
commit b99121a
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/server/src/main/java/access/manage/Manage.java b/server/src/main/java/access/manage/Manage.java
@@ -33,12 +33,12 @@ default List<Map<String, Object>> transformProvider(List<Map<String, Object>> pr
     default Map<String, Object> transformProvider(Map<String, Object> provider) {
         //Defensive mostly because of tests
         if (CollectionUtils.isEmpty(provider)) {
-            return provider;
+            return Collections.emptyMap();
         }
         Map data = (Map) provider.get("data");
         //When mocking - using the results of LocalManage - the provider may already be transformed
         if (CollectionUtils.isEmpty(data)) {
-            return provider;
+            return Collections.emptyMap();
         }
         Map metaDataFields = (Map) data.get("metaDataFields");
         //Can't use Map.of as values can be null