Expose a more consistent API for model grouping.

usnistgov · Jun 24, 2024 · db54ea5 · db54ea5
1 parent cf9ddfe
commit db54ea5
Show file tree

Hide file tree

Showing 6 changed files with 18 additions and 7 deletions.
diff --git a/...t/secauto/metaschema/core/metapath/item/node/IFeatureOrhpanedDefinitionModelNodeItem.java b/...t/secauto/metaschema/core/metapath/item/node/IFeatureOrhpanedDefinitionModelNodeItem.java
@@ -28,7 +28,7 @@
 
 import gov.nist.secauto.metaschema.core.model.IDefinition;
 import gov.nist.secauto.metaschema.core.model.IModelDefinition;
-import gov.nist.secauto.metaschema.core.model.INamedInstance;
+import gov.nist.secauto.metaschema.core.model.INamedModelInstance;
 
 /**
  * A mixin interface used to identify that the implementation is a
@@ -44,7 +44,7 @@
  * @param <I>
  *          the type of the instance that could be created from the definition
  */
-interface IFeatureOrhpanedDefinitionModelNodeItem<D extends IModelDefinition, I extends INamedInstance>
+interface IFeatureOrhpanedDefinitionModelNodeItem<D extends IModelDefinition, I extends INamedModelInstance>
     extends IModelNodeItem<D, I>, IFeatureOrhpanedDefinitionNodeItem<D, I> {
 
   @Override

diff --git a/core/src/main/java/gov/nist/secauto/metaschema/core/metapath/item/node/IModelNodeItem.java b/core/src/main/java/gov/nist/secauto/metaschema/core/metapath/item/node/IModelNodeItem.java
@@ -27,9 +27,9 @@
 package gov.nist.secauto.metaschema.core.metapath.item.node;
 
 import gov.nist.secauto.metaschema.core.model.IModelDefinition;
-import gov.nist.secauto.metaschema.core.model.INamedInstance;
+import gov.nist.secauto.metaschema.core.model.INamedModelInstance;
 
-public interface IModelNodeItem<D extends IModelDefinition, I extends INamedInstance>
+public interface IModelNodeItem<D extends IModelDefinition, I extends INamedModelInstance>
     extends IDefinitionNodeItem<D, I> {
 
   /**

diff --git a/core/src/main/java/gov/nist/secauto/metaschema/core/model/IGroupable.java b/core/src/main/java/gov/nist/secauto/metaschema/core/model/IGroupable.java
@@ -36,7 +36,7 @@
 import edu.umd.cs.findbugs.annotations.NonNull;
 import edu.umd.cs.findbugs.annotations.Nullable;
 
-public interface IGroupable extends IInstanceAbsolute {
+public interface IGroupable extends IInstance {
 
   int DEFAULT_GROUP_AS_MIN_OCCURS = 0;
   int DEFAULT_GROUP_AS_MAX_OCCURS = 1;

diff --git a/core/src/main/java/gov/nist/secauto/metaschema/core/model/IModelInstance.java b/core/src/main/java/gov/nist/secauto/metaschema/core/model/IModelInstance.java
@@ -30,7 +30,7 @@
  * This marker interface is used to identify a field or assembly instance that
  * is a member of an assembly's model.
  */
-public interface IModelInstance extends IInstance {
+public interface IModelInstance extends IGroupable {
   @Override
   IContainerModel getParentContainer();
 

diff --git a/core/src/main/java/gov/nist/secauto/metaschema/core/model/IModelInstanceAbsolute.java b/core/src/main/java/gov/nist/secauto/metaschema/core/model/IModelInstanceAbsolute.java
@@ -26,6 +26,6 @@
 
 package gov.nist.secauto.metaschema.core.model;
 
-public interface IModelInstanceAbsolute extends IModelInstance, IGroupable {
+public interface IModelInstanceAbsolute extends IModelInstance, IInstanceAbsolute {
   // no additional methods
 }
diff --git a/core/src/main/java/gov/nist/secauto/metaschema/core/model/INamedModelInstanceGrouped.java b/core/src/main/java/gov/nist/secauto/metaschema/core/model/INamedModelInstanceGrouped.java
@@ -84,4 +84,15 @@ default IFlagInstance getJsonKey() {
         ? null
         : ObjectUtils.requireNonNull(getDefinition().getFlagInstanceByName(getContainingModule().toFlagQName(name)));
   }
+
+  @Override
+  default int getMinOccurs() {
+    return getParentContainer().getMinOccurs();
+  }
+
+  @Override
+  default int getMaxOccurs() {
+    return getParentContainer().getMaxOccurs();
+  }
+
 }