better client generation

dcarbone · Jan 30, 2025 · 62c0457 · 62c0457
1 parent c0f8347
commit 62c0457
Show file tree

Hide file tree

Showing 3 changed files with 19 additions and 12 deletions.
diff --git a/src/Version/Definition/TypeExtractor.php b/src/Version/Definition/TypeExtractor.php
@@ -136,7 +136,7 @@ protected static function extractTypesFromXSD(Config $config, Version $version,
                     $type = TypeBuilder::build($config, $version, $fhirName, $child, $sourceFile);
 
                     // add type
-                    $types->addType($type);
+                    $type = $types->addOrReturnType($type);
 
                     // proceed with decoration
                     SimpleTypeElementTypeDecorator::decorate($config, $types, $type, $child);
@@ -148,7 +148,7 @@ protected static function extractTypesFromXSD(Config $config, Version $version,
                     $type = TypeBuilder::build($config, $version, $fhirName, $child, $sourceFile);
 
                     // add type
-                    $types->addType($type);
+                    $type = $types->addOrReturnType($type);
 
                     // proceed with decoration
                     ComplexTypeElementTypeDecorator::decorate($config, $types, $type, $child);
@@ -173,7 +173,7 @@ protected static function extractTypesFromXSD(Config $config, Version $version,
                     $type = TypeBuilder::build($config, $version, $fhirName, $child, $sourceFile);
 
                     // add type
-                    $types->addType($type);
+                    $type = $types->addOrReturnType($type);
 
                     // proceed with decoration
                     ElementElementTypeDecorator::rootDecorate($config, $types, $type, $child);

diff --git a/src/Version/Definition/Types.php b/src/Version/Definition/Types.php
@@ -58,7 +58,7 @@ public function __construct(Version $version)
         // TODO(dcarbone): this sucks.
         $xt = new Type($version, PHPFHIR_XHTML_TYPE_NAME);
         $xt->setKind(TypeKindEnum::PHPFHIR_XHTML);
-        $this->addType($xt);
+        $this->addOrReturnType($xt);
     }
 
     /**
@@ -117,14 +117,14 @@ public function getTypeByFQN(string $fqn, bool $leadingSlash): null|Type
      * same type running around...
      *
      * @param \DCarbone\PHPFHIR\Version\Definition\Type $type
-     * @return \DCarbone\PHPFHIR\Version\Definition\Types
+     * @return \DCarbone\PHPFHIR\Version\Definition\Type
      */
-    public function addType(Type &$type): Types
+    public function addOrReturnType(Type $type): Type
     {
         $tname = $type->getFHIRName();
         foreach ($this->_types as $current) {
             if ($type === $current) {
-                return $this;
+                return $current;
             }
             if ($current->getFHIRName() === $tname) {
                 // this happens with FHIR types sometimes...
@@ -136,12 +136,11 @@ public function addType(Type &$type): Types
                         $type->getSourceFileBasename()
                     )
                 );
-                $type = $current;
-                return $this;
+                return $current;
             }
         }
         $this->_types[] = $type;
-        return $this;
+        return $type;
     }
 
     /**

diff --git a/template/versions/core/class_version_client.php b/template/versions/core/class_version_client.php
@@ -48,7 +48,11 @@
     );
 
 foreach($types->getIterator() as $type) {
-    if (($type->isResourceType() || $type->hasResourceTypeParent()) && !$type->isAbstract() && !$type->getKind()->isResourceContainer($version)) {
+    if ($type->hasResourceTypeParent()
+        && 'Bundle' !== $type->getFHIRName()
+        && 'DomainResource' !== $type->getFHIRName()
+        && !$type->isAbstract()
+        && !$type->getKind()->isResourceContainer($version)) {
         $imports->addVersionTypeImports($type);
     }
 }
@@ -212,7 +216,11 @@ protected function _requireOK(<?php echo PHPFHIR_CLIENT_CLASSNAME_RESPONSE; ?> $
         }
     }
 <?php foreach($version->getDefinition()->getTypes()->getNameSortedIterator() as $rsc) :
-    if (!($rsc->isResourceType() || $rsc->hasResourceTypeParent()) || $rsc->isAbstract() || $rsc->getKind()->isResourceContainer($version)) {
+    if (!$rsc->hasResourceTypeParent()
+        || 'Bundle' === $rsc->getFHIRName()
+        || 'DomainResource' === $rsc->getFHIRName()
+        || $rsc->isAbstract()
+        || $rsc->getKind()->isResourceContainer($version)) {
         continue;
     }