yii2-openapi icon indicating copy to clipboard operation
yii2-openapi copied to clipboard

generated base controller wrong namespace in findModel

Open daronjay opened this issue 2 years ago • 4 comments

Hi, I have looked online and in the docs, cant see this issue. When I generate with ./yii gii/api --openApiPath=http://localhost:3000/documentation/json

My base classes all have a broken findModel method. I am getting

public function findModel($id)
{
        $model =  \::findOne($id); 
        
        if ($model !== null) {   
        
            return $model;  
            
        }       
        
        throw new \yii\web\NotFoundHttpException("Object not found: $id");      
  }

where I would be expecting say \common\models\Company::findOne($id).

I assume some config isn't set right, but I'm blowed if i can see how.

daronjay avatar May 30 '23 03:05 daronjay

Have you configured ApiGenerator like mentioned in https://github.com/cebe/yii2-openapi#usage

SOHELAHMED7 avatar Jun 01 '23 13:06 SOHELAHMED7

So I have:

generators =>   [
      'api' => [
        'class' => \cebe\yii2openapi\generator\ApiGenerator::class,
        'urlConfigFile' => '@api/config/url-rules.rest.php',
        'controllerNamespace' => 'api\\controllers',
        'modelNamespace' => 'common\\models',
        'fakerNamespace' => 'common\\models\\faker',
        'migrationPath' => '@common/migrations',
    ]
  ],

We are running php 8.2.3, wondering if its some incompatibility?

daronjay avatar Jun 01 '23 21:06 daronjay

We are running php 8.2.3, wondering if its some incompatibility?

Yes it is incompatible. But the PR for PHP 8.2 compatibility is completed and tests are passing and new version of this lib is not released yet.

Steps to run that version of this lib in your project using PHP 8.2.* can be found at https://github.com/SOHELAHMED7/yii2-openapi/blob/master/CONTRIBUTING.md#use-pr-of-your-own-fork-of-this-library-in-your-project-to-check-new-changes

Feel free to write us back with your experience and whether your issue is fixed.

SOHELAHMED7 avatar Jun 02 '23 10:06 SOHELAHMED7

This issue can be closed.

SOHELAHMED7 avatar May 30 '24 13:05 SOHELAHMED7