Merge pull request #8 from moufmouf/output_id_ref

Fixing an issue with output="ID" set on foreign keys

Author: moufmouf

src/TdbmFluidColumnGraphqlOptions.php

@@ -63,7 +63,18 @@ private function generateFieldAnnotation(): void
         if ($this->outputType !== null) {
             $outputType = $this->outputType;
         } elseif ($this->fluidColumn->getDbalColumn()->getType() === Type::getType(Type::GUID)) {
-            $outputType = 'ID';
+            // are we part of a foreign key or not?
+            $fks = $this->tdbmFluidColumnOptions->then()->getDbalTable()->getForeignKeys();
+            $isPartOfFk = false;
+            foreach ($fks as $fk) {
+                if (in_array($this->fluidColumn->getDbalColumn()->getName(), $fk->getLocalColumns(), true) === true) {
+                    $isPartOfFk = true;
+                    break;
+                }
+            }
+            if ($isPartOfFk === false) {
+                $outputType = 'ID';
+            }
         } else {
             // If the column is the primary key, let's add an ID type
             $pk = $this->tdbmFluidColumnOptions->then()->getDbalTable()->getPrimaryKey();

tests/TdbmFluidColumnGraphqlOptionsTest.php

@@ -52,5 +52,9 @@ public function testGraphql()
         $uuidColumn = $users->uuid()->graphqlField();
         $this->assertContains('outputType = "ID"', $schema->getTable('users')->getColumn('uuid')->getComment());
 
+        $products = $fluid->table('products')
+            ->uuid()
+            ->column('user_id')->references('users')->graphqlField();
+        $this->assertNotContains('outputType = "ID"', $schema->getTable('products')->getColumn('user_id')->getComment());
     }
 }