Fix conflicting pull requests

src/generator/default/dbmodel.php

@@ -146,12 +146,14 @@ public function get<?= $relation->getCamelName() ?>()
 <?php endif;?>
     }
 <?php endforeach; ?>
-<?php $i = 1; foreach ($model->inverseRelations as $relationName => $relation): ?>
+<?php $i = 1; $usedRelationNames = [];
+foreach ($model->belongsToRelations as $relationName => $relation): ?><?php $number = in_array($relation->getCamelName(), $usedRelationNames) ? $i : '' ?>
 
-    public function get<?= $relation->getCamelName().($i===1 ? '' : $i) ?>()
+    # belongs to relation
+    public function get<?= $relation->getCamelName() . ($number) ?>()
     {
         return $this-><?= $relation->getMethod() ?>(\<?= trim($relationNamespace, '\\') ?>\<?= $relation->getClassName() ?>::class, <?php
-    echo $relation->linkToString() ?>)->inverseOf('<?= $relation->getInverse() ?>');
+    echo $relation->linkToString() ?>);
     }
-<?php $i++; endforeach; ?>
+<?php $i++; $usedRelationNames[] = $relation->getCamelName(); endforeach; ?>
 }

src/lib/AttributeResolver.php

@@ -37,6 +37,11 @@ class AttributeResolver
      */
     public array $relations = [];
 
+    /**
+     * @var array keys contains class names and value contains array of belongs to relations
+     */
+    public array $belongsToRelations = [];
+
     /**
      * @var NonDbRelation[]|array
      */
@@ -60,11 +65,6 @@ class AttributeResolver
 
     private ?Config $config;
 
-    /**
-     * @var AttributeRelation[]|array
-     */
-    public array $inverseRelations = [];
-
     public function __construct(string $schemaName, ComponentSchema $schema, JunctionSchemas $junctions, ?Config $config = null)
     {
         $this->schemaName = $schemaName;
@@ -232,6 +232,7 @@ protected function resolveProperty(
                   ->setForeignKeyColumnName($property->fkColName)
                   ->setFakerStub($this->guessFakerStub($attribute, $property))
                   ->setTableName($this->componentSchema->resolveTableName($this->schemaName));
+
         if ($property->isReference()) {
             if ($property->isVirtual()) {
                 throw new InvalidDefinitionException('References not supported for virtual attributes');
@@ -277,12 +278,17 @@ protected function resolveProperty(
             $relation->onDeleteFkConstraint = $property->onDeleteFkConstraint;
             if ($property->isRefPointerToSelf()) {
                 $relation->asSelfReference();
+            } else { # belongs to relations https://github.com/php-openapi/yii2-openapi/issues/90
+                $belongsToRelation = Yii::createObject(
+                    AttributeRelation::class,
+                    [$this->schemaName, $this->tableName, $this->schemaName]
+                )
+                    ->asHasOne([$attribute->columnName => $fkProperty->getName()]);
+                $this->belongsToRelations[$property->getRefClassName()][] = $belongsToRelation;
             }
             $this->relations[$property->getName()] = $relation;
-            if (!$property->isRefPointerToSelf()) {
-                $this->addInverseRelation($relatedClassName, $attribute, $property, $fkProperty);
-            }
         }
+
         if (!$property->isReference() && !$property->hasRefItems()) {
             [$min, $max] = $property->guessMinMax();
             $attribute->setIsVirtual($property->isVirtual())
@@ -338,6 +344,7 @@ protected function resolveProperty(
                         ->asHasMany([$foreignPk => $this->componentSchema->getPkName()]);
                 return;
             }
+
             $relatedClassName = $property->getRefClassName();
             $relatedTableName = $property->getRefSchema()->resolveTableName($relatedClassName);
             if ($this->catchManyToMany(
@@ -518,22 +525,4 @@ public static function relationName(string $propertyName, ?string $fkColumnName)
         }
         return $relationName;
     }
-
-    /**
-     * @throws InvalidConfigException
-     */
-    public function addInverseRelation(
-        string $relatedClassName,
-        Attribute $attribute,
-        PropertySchema $property,
-        PropertySchema $fkProperty
-    ): void {
-        $inverseRelation = Yii::createObject(
-            AttributeRelation::class,
-            [$this->schemaName, $this->tableName, $this->schemaName]
-        )
-            ->asHasOne([$attribute->columnName => $fkProperty->getName()]);
-        $inverseRelation->setInverse($property->getName());
-        $this->inverseRelations[$relatedClassName][] = $inverseRelation;
-    }
 }

src/lib/SchemaToDatabase.php

@@ -94,7 +94,7 @@ public function prepareModels(): array
 
         $openApi = $this->config->getOpenApi();
         $junctions = $this->findJunctionSchemas();
-        foreach ($openApi->components->schemas as $schemaName => $openApiSchema) {
+        foreach ($openApi->components->schemas ?? [] as $schemaName => $openApiSchema) {
             $schema = Yii::createObject(ComponentSchema::class, [$openApiSchema, $schemaName]);
 
             if (!$this->canGenerateModel($schemaName, $schema)) {
@@ -106,18 +106,15 @@ public function prepareModels(): array
             /** @var AttributeResolver $resolver */
             $resolver = Yii::createObject(AttributeResolver::class, [$schemaName, $schema, $junctions, $this->config]);
 
-            // $models[$schemaName] = $resolver->resolve();
             $resolvers[$schemaName] = $resolver;
             $models[$schemaName] = $resolvers[$schemaName]->resolve();
         }
 
-        // handle inverse relation
+        // handle belongs to relation
         foreach ($resolvers as $aResolver) {
-            foreach ($aResolver->inverseRelations as $name => $relations) {
-                foreach ($relations as $relation) {
-                    /** @var AttributeRelation $relation */
-                    $models[$name]->inverseRelations[] = $relation;
-                }
+            foreach ($aResolver->belongsToRelations as $name => $relations) {
+                /** @var AttributeRelation[] $relations */
+                $models[$name]->belongsToRelations = [...$models[$name]->belongsToRelations, ...$relations];
             }
         }
 
@@ -153,7 +150,7 @@ public function findJunctionSchemas(): JunctionSchemas
     {
         $junctions = [];
         $openApi = $this->config->getOpenApi();
-        foreach ($openApi->components->schemas as $schemaName => $openApiSchema) {
+        foreach ($openApi->components->schemas ?? [] as $schemaName => $openApiSchema) {
             /**@var ComponentSchema $schema */
             $schema = Yii::createObject(ComponentSchema::class, [$openApiSchema, $schemaName]);
             if ($schema->isNonDb()) {

src/lib/generators/ControllersGenerator.php

@@ -59,7 +59,7 @@ public function generate():CodeFiles
             $controllerPath = $path;
             /**
              * @var RestAction|FractalAction $action
-            **/
+             **/
             $action = $actions[0];
             if ($action->prefix && !empty($action->prefixSettings)) {
                 $controllerNamespace = trim($action->prefixSettings['namespace'], '\\');
@@ -126,15 +126,31 @@ protected function makeCustomController(
         ];
         $reflection->addMethod('checkAccess', $params, AbstractMemberGenerator::FLAG_PUBLIC, '//TODO implement checkAccess');
         foreach ($abstractActions as $action) {
+            $responseHttpStatusCodes = '';
+            foreach ($this->config->getOpenApi()->paths->getPaths()[$action->urlPath]->getOperations() as $verb => $operation) {
+                $codes = array_keys($operation->responses->getResponses());
+
+                $only200OrDefault = false;
+                if ($codes === [200] || $codes === ['default']) {
+                    $only200OrDefault = true;
+                }
+                if (in_array('default', $codes) && in_array(200, $codes) && count($codes) === 2) {
+                    $only200OrDefault = true;
+                }
+
+                if ($verb === strtolower($action->requestMethod) && !$only200OrDefault) {
+                    $responseHttpStatusCodes = implode(', ', $codes);
+                }
+            }
+
             $params = array_map(static function ($param) {
                 return ['name' => $param];
             }, $action->getParamNames());
-
             $reflection->addMethod(
                 $action->actionMethodName,
                 $params,
                 AbstractMemberGenerator::FLAG_PUBLIC,
-                '//TODO implement ' . $action->actionMethodName
+                '//TODO implement ' . $action->actionMethodName . ($responseHttpStatusCodes ? PHP_EOL . '// In order to conform with OpenAPI spec, response of this action must have one of the following HTTP status code: ' . $responseHttpStatusCodes : '')
             );
         }
         $classFileGenerator->setClasses([$reflection]);

src/lib/items/DbModel.php

@@ -67,11 +67,6 @@ class DbModel extends BaseObject
      */
     public array $many2many = [];
 
-    /**
-     * @var array|AttributeRelation[] inverse relations
-     */
-    public array $inverseRelations = [];
-
     public array $junctionCols = [];
 
     /**
@@ -91,9 +86,9 @@ class DbModel extends BaseObject
     public bool $descriptionIsComment = false;
 
     /**
-     * @var array Automatically generated scenarios from the model 'x-scenarios'.
+     * @var AttributeRelation[] belongs to relations
      */
-    private $_scenarios;
+    public array $belongsToRelations = [];
 
     /**
      * @var bool
@@ -103,6 +98,11 @@ class DbModel extends BaseObject
      */
     public $drop = false;
 
+    /**
+     * @var array Automatically generated scenarios from the model 'x-scenarios'.
+     */
+    private $_scenarios;
+
     public function getTableAlias(): string
     {
         return '{{%' . $this->tableName . '}}';

tests/specs/blog/models/base/Category.php

@@ -39,8 +39,9 @@ public function getPosts()
         return $this->hasMany(\app\models\Post::class, ['category_id' => 'id'])->inverseOf('category');
     }
 
+    # belongs to relation
     public function getPost()
     {
-        return $this->hasOne(\app\models\Post::class, ['category_id' => 'id'])->inverseOf('category');
+        return $this->hasOne(\app\models\Post::class, ['category_id' => 'id']);
     }
 }

tests/specs/blog/models/base/Post.php

@@ -62,8 +62,9 @@ public function getComments()
         return $this->hasMany(\app\models\Comment::class, ['post_id' => 'uid'])->inverseOf('post');
     }
 
+    # belongs to relation
     public function getComment()
     {
-        return $this->hasOne(\app\models\Comment::class, ['post_id' => 'uid'])->inverseOf('post');
+        return $this->hasOne(\app\models\Comment::class, ['post_id' => 'uid']);
     }
 }

tests/specs/blog/models/base/User.php

@@ -45,13 +45,15 @@ public function rules()
         ];
     }
 
+    # belongs to relation
     public function getPost()
     {
-        return $this->hasOne(\app\models\Post::class, ['created_by_id' => 'id'])->inverseOf('created_by');
+        return $this->hasOne(\app\models\Post::class, ['created_by_id' => 'id']);
     }
 
-    public function getComment2()
+    # belongs to relation
+    public function getComment()
     {
-        return $this->hasOne(\app\models\Comment::class, ['author_id' => 'id'])->inverseOf('author');
+        return $this->hasOne(\app\models\Comment::class, ['author_id' => 'id']);
     }
 }

tests/specs/blog_v2/controllers/CommentController.php

@@ -18,6 +18,7 @@ public function actionListForPost($postId)
     public function actionCreateForPost($postId)
     {
         //TODO implement actionCreateForPost
+        // In order to conform with OpenAPI spec, response of this action must have one of the following HTTP status code: 201, default
     }
 
     public function actionViewForPost($slug, $id)
@@ -28,6 +29,7 @@ public function actionViewForPost($slug, $id)
     public function actionDeleteForPost($slug, $id)
     {
         //TODO implement actionDeleteForPost
+        // In order to conform with OpenAPI spec, response of this action must have one of the following HTTP status code: 204
     }
 
     public function actionUpdateForPost($slug, $id)

tests/specs/blog_v2/models/base/Category.php

@@ -39,8 +39,9 @@ public function getPosts()
         return $this->hasMany(\app\models\Post::class, ['category_id' => 'id'])->inverseOf('category');
     }
 
+    # belongs to relation
     public function getPost()
     {
-        return $this->hasOne(\app\models\Post::class, ['category_id' => 'id'])->inverseOf('category');
+        return $this->hasOne(\app\models\Post::class, ['category_id' => 'id']);
     }
 }

tests/specs/blog_v2/models/base/Post.php

@@ -74,8 +74,9 @@ public function getTags()
                     ->viaTable('posts2tags', ['post_id' => 'id']);
     }
 
+    # belongs to relation
     public function getComment()
     {
-        return $this->hasOne(\app\models\Comment::class, ['post_id' => 'id'])->inverseOf('post');
+        return $this->hasOne(\app\models\Comment::class, ['post_id' => 'id']);
     }
 }

tests/specs/blog_v2/models/base/User.php

@@ -48,13 +48,15 @@ public function rules()
         ];
     }
 
+    # belongs to relation
     public function getPost()
     {
-        return $this->hasOne(\app\models\Post::class, ['created_by_id' => 'id'])->inverseOf('created_by');
+        return $this->hasOne(\app\models\Post::class, ['created_by_id' => 'id']);
     }
 
-    public function getComment2()
+    # belongs to relation
+    public function getComment()
     {
-        return $this->hasOne(\app\models\Comment::class, ['user_id' => 'id'])->inverseOf('user');
+        return $this->hasOne(\app\models\Comment::class, ['user_id' => 'id']);
     }
 }

tests/specs/fk_col_name/app/models/base/Delivery.php

@@ -28,8 +28,9 @@ public function rules()
         ];
     }
 
+    # belongs to relation
     public function getWebhook()
     {
-        return $this->hasOne(\app\models\Webhook::class, ['redelivery_of' => 'id'])->inverseOf('redelivery_of');
+        return $this->hasOne(\app\models\Webhook::class, ['redelivery_of' => 'id']);
     }
 }

tests/specs/fk_col_name/app/models/base/User.php

@@ -29,8 +29,9 @@ public function rules()
         ];
     }
 
+    # belongs to relation
     public function getWebhook()
     {
-        return $this->hasOne(\app\models\Webhook::class, ['user_id' => 'id'])->inverseOf('user');
+        return $this->hasOne(\app\models\Webhook::class, ['user_id' => 'id']);
     }
 }

tests/specs/fk_col_name_index/app/models/base/Delivery.php

@@ -28,13 +28,15 @@ public function rules()
         ];
     }
 
+    # belongs to relation
     public function getWebhook()
     {
-        return $this->hasOne(\app\models\Webhook::class, ['redelivery_of' => 'id'])->inverseOf('redelivery_of');
+        return $this->hasOne(\app\models\Webhook::class, ['redelivery_of' => 'id']);
     }
 
+    # belongs to relation
     public function getWebhook2()
     {
-        return $this->hasOne(\app\models\Webhook::class, ['rd_abc_2' => 'id'])->inverseOf('rd2');
+        return $this->hasOne(\app\models\Webhook::class, ['rd_abc_2' => 'id']);
     }
 }

tests/specs/fk_col_name_index/app/models/base/User.php

@@ -29,8 +29,9 @@ public function rules()
         ];
     }
 
+    # belongs to relation
     public function getWebhook()
     {
-        return $this->hasOne(\app\models\Webhook::class, ['user_id' => 'id'])->inverseOf('user');
+        return $this->hasOne(\app\models\Webhook::class, ['user_id' => 'id']);
     }
 }

tests/specs/issue_fix/159_bug_giiapi_generated_rules_emailid/maria/models/base/Mailing.php

@@ -31,8 +31,9 @@ public function rules()
         ];
     }
 
+    # belongs to relation
     public function getContact()
     {
-        return $this->hasOne(\app\models\Contact::class, ['mailing_id' => 'id'])->inverseOf('mailing');
+        return $this->hasOne(\app\models\Contact::class, ['mailing_id' => 'id']);
     }
 }

tests/specs/issue_fix/162_bug_dollarref_with_x_faker/app/models/base/Invoice.php

@@ -24,8 +24,9 @@ public function rules()
         return [];
     }
 
+    # belongs to relation
     public function getOrder()
     {
-        return $this->hasOne(\app\models\Order::class, ['invoice_id' => 'id'])->inverseOf('invoice');
+        return $this->hasOne(\app\models\Order::class, ['invoice_id' => 'id']);
     }
 }