From 4b86dd2925a9e48b51971567b5029e4605314154 Mon Sep 17 00:00:00 2001
From: Daniel Carbone <daniel.p.carbone@gmail.com>
Date: Sat, 1 Feb 2025 15:48:13 -0600
Subject: [PATCH] some hacks for the resource container, not sure it even
 serves a purpose anymore...

---
 src/Version/Definition/Type.php                 |  7 ++++---
 .../versions/types/class_resource_container.php | 16 ++++++++++------
 .../serialization/json/unserialize/header.php   | 17 +++++++++++------
 .../serialization/xml/unserialize/header.php    | 17 +++++++++++------
 4 files changed, 36 insertions(+), 21 deletions(-)

diff --git a/src/Version/Definition/Type.php b/src/Version/Definition/Type.php
index 3d2e53ba..3ba9308e 100644
--- a/src/Version/Definition/Type.php
+++ b/src/Version/Definition/Type.php
@@ -878,8 +878,8 @@ public function getDirectlyImplementedInterfaces(): array
                         ->getFullyQualifiedNamespace(false);
                 }
             }
-        } else if ($this->isResourceType()) {
-            if (!$this->hasResourceTypeParent()) {
+        } else if ($this->isResourceType() || $this->getKind()->isResourceContainer($this->_version)) {
+            if (!$this->hasResourceTypeParent() && !$sourceMeta->isDSTU1()) {
                 $interfaces[PHPFHIR_TYPES_INTERFACE_RESOURCE_TYPE] = $coreFiles
                     ->getCoreFileByEntityName(PHPFHIR_TYPES_INTERFACE_RESOURCE_TYPE)
                     ->getFullyQualifiedNamespace(false);
@@ -913,6 +913,7 @@ public function getDirectlyImplementedInterfaces(): array
     public function getDirectlyUsedTraits(): array
     {
         $traits = [];
+        $sourceMeta = $this->_version->getSourceMetadata();
         $parentType = $this->getParentType();
         $coreFiles = $this->_version->getConfig()->getCoreFiles();
 
@@ -926,7 +927,7 @@ public function getDirectlyUsedTraits(): array
             }
 
             // these must only be added if the type has local properties
-            if (($this->isResourceType() || $this->getVersion()->getSourceMetadata()->isDSTU1()) && $this->hasLocalProperties()) {
+            if (($this->isResourceType() || $sourceMeta->isDSTU1() || $this->_kind->isResourceContainer($this->_version)) && $this->hasLocalProperties()) {
                 $traits[PHPFHIR_TRAIT_SOURCE_XMLNS] = $coreFiles
                     ->getCoreFileByEntityName(PHPFHIR_TRAIT_SOURCE_XMLNS)
                     ->getFullyQualifiedNamespace(false);
diff --git a/template/versions/types/class_resource_container.php b/template/versions/types/class_resource_container.php
index c87805e6..ba6bcd26 100644
--- a/template/versions/types/class_resource_container.php
+++ b/template/versions/types/class_resource_container.php
@@ -117,7 +117,7 @@ public function setContainedType(null|<?php echo $versionContainedTypeInterface-
         foreach ($element->children() as $child) {
             /** @var <?php echo $versionContainedTypeInterface->getFullyQualifiedName(true); ?> $class */
             $class = <?php echo $versionTypeMapClass->getEntityName(); ?>::getContainedTypeClassNameFromXML($child);
-            $type->setContainedType($class::xmlUnserialize($child, null, $config));
+            $type->setContainedType($class::xmlUnserialize($child, $config));
             break;
         }
         return $type;
@@ -137,8 +137,11 @@ public function xmlSerialize(null|<?php echo $xmlWriterClass->getEntityName(); ?
         if (null !== $containedType) {
             return $containedType->xmlSerialize($xw, $config);
         }
+        if (null === $config) {
+            $config = (new <?php echo PHPFHIR_VERSION_CLASSNAME_VERSION; ?>())->getConfig()->getSerializeConfig();
+        }
         if (null === $xw) {
-            $xw = new <?php echo $xmlWriterClass->getEntityName(); ?>();
+            $xw = new <?php echo $xmlWriterClass->getEntityName(); ?>($config);
         }
         if (!$xw->isOpen()) {
             $xw->openMemory();
@@ -147,12 +150,9 @@ public function xmlSerialize(null|<?php echo $xmlWriterClass->getEntityName(); ?
             $docStarted = true;
             $xw->startDocument();
         }
-        if (null === $config) {
-            $config = (new <?php echo PHPFHIR_VERSION_CLASSNAME_VERSION; ?>())->getConfig()->getSerializeConfig();
-        }
         if (!$xw->isRootOpen()) {
             $rootOpened = true;
-            $xw->openRootNode($config, '<?php echo NameUtils::getTypeXMLElementName($type); ?>', $this->_getSourceXMLNS());
+            $xw->openRootNode('<?php echo NameUtils::getTypeXMLElementName($type); ?>', $this->_getSourceXMLNS());
         }
         if (isset($rootOpened) && $rootOpened) {
             $xw->endElement();
@@ -173,6 +173,10 @@ public function xmlSerialize(null|<?php echo $xmlWriterClass->getEntityName(); ?
     ]
 );
 ?>
+        /** @var <?php echo $versionContainedTypeInterface->getFullyQualifiedName(true); ?> $class */
+        $class = <?php echo $versionTypeMapClass->getEntityName(); ?>::getContainedTypeClassNameFromArray($json);
+        $type->setContainedType($class::jsonUnserialize($json));
+        return $type;
     }
 
     public function __toString(): string
diff --git a/template/versions/types/serialization/json/unserialize/header.php b/template/versions/types/serialization/json/unserialize/header.php
index d434e8d9..fae5d622 100644
--- a/template/versions/types/serialization/json/unserialize/header.php
+++ b/template/versions/types/serialization/json/unserialize/header.php
@@ -21,13 +21,18 @@
 
 $sourceMeta = $version->getSourceMetadata();
 
+$isResource = $type->isResourceType()
+    || $type->hasResourceTypeParent()
+    || $sourceMeta->isDSTU1()
+    || $type->getKind()->isResourceContainer($version);
+
 $config = $version->getConfig();
 
 $coreFiles = $config->getCoreFiles();
 $constantsClass = $coreFiles->getCoreFileByEntityName(PHPFHIR_CLASSNAME_CONSTANTS);
 $unserializeConfigClass = $coreFiles->getCoreFileByEntityName(PHPFHIR_ENCODING_CLASSNAME_UNSERIALIZE_CONFIG);
 
-if ($type->isResourceType() || $type->hasResourceTypeParent() || $sourceMeta->isDSTU1()) {
+if ($isResource) {
     $typeInterface = $coreFiles->getCoreFileByEntityName(PHPFHIR_TYPES_INTERFACE_RESOURCE_TYPE);
 } else {
     $typeInterface = $coreFiles->getCoreFileByEntityName(PHPFHIR_TYPES_INTERFACE_ELEMENT_TYPE);
@@ -38,15 +43,15 @@
 
 ob_start(); ?>
     /**
-     * @param <?php if ($type->isResourceType() || $type->hasResourceTypeParent() || $sourceMeta->isDSTU1()) : ?>string|\stdClass|<?php endif; ?>array $json
-     * @param <?php if ($type->isResourceType() || $type->hasResourceTypeParent() || $sourceMeta->isDSTU1()) : ?>null|<?php endif; echo $unserializeConfigClass->getFullyQualifiedName(true); ?> $config
+     * @param <?php if ($isResource) : ?>string|\stdClass|<?php endif; ?>array $json
+     * @param <?php if ($isResource) : ?>null|<?php endif; echo $unserializeConfigClass->getFullyQualifiedName(true); ?> $config
      * @param null|<?php echo $type->getFullyQualifiedClassName(true); ?> $type
      * @return <?php echo $type->getFullyQualifiedClassName(true); ?>
 
      * @throws \Exception
      */
-    public static function jsonUnserialize(<?php if ($type->isResourceType() || $type->hasResourceTypeParent() || $sourceMeta->isDSTU1()) : ?>string|\stdClass|<?php endif; ?>array $json,
-                                           <?php if ($type->isResourceType() || $type->hasResourceTypeParent() || $sourceMeta->isDSTU1()) : ?>null|<?php endif; echo $unserializeConfigClass->getEntityName() ?> $config<?php if ($type->isResourceType() || $type->hasResourceTypeParent() || $sourceMeta->isDSTU1()) : ?> = null<?php endif;?>,
+    public static function jsonUnserialize(<?php if ($isResource) : ?>string|\stdClass|<?php endif; ?>array $json,
+                                           <?php if ($isResource) : ?>null|<?php endif; echo $unserializeConfigClass->getEntityName() ?> $config<?php if ($isResource) : ?> = null<?php endif;?>,
                                            null|<?php echo $typeInterface->getEntityName(); ?> $type = null): self
     {
 <?php if ($type->isAbstract()) : // abstract types may not be instantiated directly ?>
@@ -63,7 +68,7 @@ public static function jsonUnserialize(<?php if ($type->isResourceType() || $typ
                 get_class($type)
             ));
         }
-<?php if ($type->isResourceType() || $type->hasResourceTypeParent() || $sourceMeta->isDSTU1()) : ?>
+<?php if ($isResource) : ?>
         if (null === $config) {
             $config = (new <?php echo $versionClass->getEntityName(); ?>())->getConfig()->getUnserializeConfig();
         }
diff --git a/template/versions/types/serialization/xml/unserialize/header.php b/template/versions/types/serialization/xml/unserialize/header.php
index 0aa0b0c9..5b49ad4c 100644
--- a/template/versions/types/serialization/xml/unserialize/header.php
+++ b/template/versions/types/serialization/xml/unserialize/header.php
@@ -21,12 +21,17 @@
 
 $sourceMeta = $version->getSourceMetadata();
 
+$isResource = $type->isResourceType()
+    || $type->hasResourceTypeParent()
+    || $sourceMeta->isDSTU1()
+    || $type->getKind()->isResourceContainer($version);
+
 $config = $version->getConfig();
 
 $coreFiles = $config->getCoreFiles();
 $unserializeConfigClass = $coreFiles->getCoreFileByEntityName(PHPFHIR_ENCODING_CLASSNAME_UNSERIALIZE_CONFIG);
 
-if ($type->isResourceType() || $type->hasResourceTypeParent() || $sourceMeta->isDSTU1()) {
+if ($isResource) {
     $typeInterface = $coreFiles->getCoreFileByEntityName(PHPFHIR_TYPES_INTERFACE_RESOURCE_TYPE);
 } else {
     $typeInterface = $coreFiles->getCoreFileByEntityName(PHPFHIR_TYPES_INTERFACE_ELEMENT_TYPE);
@@ -37,15 +42,15 @@
 
 ob_start(); ?>
     /**
-     * @param <?php if ($type->isResourceType() || $type->hasResourceTypeParent() || $sourceMeta->isDSTU1()) : ?>string|<?php endif; ?>\SimpleXMLElement $element
-     * @param <?php if ($type->isResourceType() || $type->hasResourceTypeParent() || $sourceMeta->isDSTU1()) : ?>null|<?php endif; echo $unserializeConfigClass->getFullyQualifiedName(true); ?> $config
+     * @param <?php if ($isResource) : ?>string|<?php endif; ?>\SimpleXMLElement $element
+     * @param <?php if ($isResource) : ?>null|<?php endif; echo $unserializeConfigClass->getFullyQualifiedName(true); ?> $config
      * @param null|<?php echo $type->getFullyQualifiedClassName(true); ?> $type
      * @return <?php echo $type->getFullyQualifiedClassName(true); ?>
 
      * @throws \Exception
      */
-    public static function xmlUnserialize(<?php if ($type->isResourceType() || $type->hasResourceTypeParent() || $sourceMeta->isDSTU1()) : ?>string|<?php endif; ?>\SimpleXMLElement $element,
-                                          <?php if ($type->isResourceType() || $type->hasResourceTypeParent() || $sourceMeta->isDSTU1()) : ?>null|<?php endif; echo $unserializeConfigClass->getEntityName() ?> $config<?php if ($type->isResourceType() || $type->hasResourceTypeParent() || $sourceMeta->isDSTU1()) : ?> = null<?php endif;?>,
+    public static function xmlUnserialize(<?php if ($isResource) : ?>string|<?php endif; ?>\SimpleXMLElement $element,
+                                          <?php if ($isResource) : ?>null|<?php endif; echo $unserializeConfigClass->getEntityName() ?> $config<?php if ($isResource) : ?> = null<?php endif;?>,
                                           null|<?php echo $typeInterface->getEntityName(); ?> $type = null): self
     {
 <?php if ($type->isAbstract()) : // abstract types may not be instantiated directly ?>
@@ -62,7 +67,7 @@ public static function xmlUnserialize(<?php if ($type->isResourceType() || $type
                 get_class($type)
             ));
         }
-<?php if ($type->isResourceType() || $type->hasResourceTypeParent() || $sourceMeta->isDSTU1()) : ?>
+<?php if ($isResource) : ?>
         if (null === $config) {
             $config = (new <?php echo $versionClass->getEntityName(); ?>())->getConfig()->getUnserializeConfig();
         }