Skip to content
This repository was archived by the owner on Jun 4, 2024. It is now read-only.

generated base controller wrong namespace in findModel #147

Open
daronjay opened this issue May 30, 2023 · 4 comments
Open

generated base controller wrong namespace in findModel #147

daronjay opened this issue May 30, 2023 · 4 comments

Comments

@daronjay
Copy link

daronjay commented May 30, 2023

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.

@SOHELAHMED7
Copy link
Contributor

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

@daronjay
Copy link
Author

daronjay commented Jun 1, 2023

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?

@SOHELAHMED7
Copy link
Contributor

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
Copy link
Contributor

This issue can be closed.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants