NOJIRA

default values (or lack of) causing issues
TIER · Aug 24, 2021 · 4429b1e · 4429b1e
1 parent e4fc69d
commit 4429b1e
Show file tree

Hide file tree

Showing 2 changed files with 23 additions and 19 deletions.
diff --git a/...oovy/edu/internet2/tier/shibboleth/admin/ui/service/JPAMetadataResolverServiceImpl.groovy b/...oovy/edu/internet2/tier/shibboleth/admin/ui/service/JPAMetadataResolverServiceImpl.groovy
@@ -114,7 +114,7 @@ class JPAMetadataResolverServiceImpl implements MetadataResolverService {
                 if (metadataFilter instanceof NameIdFormatFilter) {
                     NameIdFormatFilter nameIdFormatFilter = NameIdFormatFilter.cast(metadataFilter)
                     NameIDFormatFilter openSamlTargetFilter = new OpenSamlNameIdFormatFilter()
-                    openSamlTargetFilter.removeExistingFormats = nameIdFormatFilter.removeExistingFormats
+                    openSamlTargetFilter.removeExistingFormats = nameIdFormatFilter.removeExistingFormats == null ? false : nameIdFormatFilter.removeExistingFormats
                     Map<Predicate<EntityDescriptor>, Collection<String>> predicateRules = [:]
                     def type = nameIdFormatFilter.nameIdFormatFilterTarget.nameIdFormatFilterTargetType
                     def values = nameIdFormatFilter.nameIdFormatFilterTarget.value
@@ -559,4 +559,4 @@ class JPAMetadataResolverServiceImpl implements MetadataResolverService {
 
     }
 
-}
+}
diff --git a/...c/main/java/edu/internet2/tier/shibboleth/admin/ui/domain/resolvers/MetadataResolver.java b/...c/main/java/edu/internet2/tier/shibboleth/admin/ui/domain/resolvers/MetadataResolver.java
@@ -84,31 +84,35 @@ public class MetadataResolver extends AbstractAuditable {
     @Transient
     private Integer version;
 
-    @JsonGetter("version")
-    public int getVersion() {
-        if (this.version != null && this.version != 0 ) {
-            return this.version;
-        }
-        return this.hashCode();
-    }
-
     public void addFilter(MetadataFilter metadataFilter) {
         //To make sure that Spring Data auditing infrastructure recognizes update and "touched" modifiedDate
         markAsModified();
         this.metadataFilters.add(metadataFilter);
     }
 
-    public void markAsModified() {
-        this.versionModifiedTimestamp = System.currentTimeMillis();
-    }
-
     public void entityAttributesFilterIntoTransientRepresentation() {
         //expose explicit API to call to convert into transient representation
         //used in unit/integration tests where JPA's @PostLoad callback execution engine is not available
         this.metadataFilters
-                .stream()
-                .filter(EntityAttributesFilter.class::isInstance)
-                .map(EntityAttributesFilter.class::cast)
-                .forEach(EntityAttributesFilter::intoTransientRepresentation);
+                        .stream()
+                        .filter(EntityAttributesFilter.class::isInstance)
+                        .map(EntityAttributesFilter.class::cast)
+                        .forEach(EntityAttributesFilter::intoTransientRepresentation);
+    }
+
+    public Boolean getDoInitialization() {
+        return doInitialization == null ? false : doInitialization;
+    }
+
+    @JsonGetter("version")
+    public int getVersion() {
+        if (this.version != null && this.version != 0 ) {
+            return this.version;
+        }
+        return this.hashCode();
+    }
+
+    public void markAsModified() {
+        this.versionModifiedTimestamp = System.currentTimeMillis();
     }
-}
+}