Depedency inside Controllers and other classes about docs HOT 9 OPEN

@viktorprogger is right. You should have something like this:

public function __construct($id, $module, \yii\authclient\clients\GitHub $githubClient)
{
    parent::__construct($id, $modjule);
    $this->githubClient = $githubClient;
}

from docs.

"Autowiring" is a phenomenon when dependencies can be automatically passed to the constructor or another method. It is a responsibility of a framework you're using, not DI container. I guess you're using Yii2 which don't know autowiring.
I'm happy to say yii3 already has this feature. Hope we'll see a stable release of this major update this year 👍

from docs.

"Autowiring" is a phenomenon when dependencies can be automatically passed to the constructor or another method. It is a responsibility of a framework you're using, not DI container. I guess you're using Yii2 which don't know autowiring.

I refer to Yii3. My question is about what is the recommendation.

@rob006 What do you think?

from docs.

@FabrizioCaldarelli sorry, didn't see you're in YiiSoft team. I thought you're just one more newbie, shame on me.

from docs.

It is a responsibility of a framework you're using, not DI container

Either I'm wrong or it's just not about this DI container. It is really responsible for autowiring.
Diving deeper into the code...

from docs.

As I see a web request results into a controller action through \yii\base\Module::createControllerByID which calls $this->app->createObject. So, the controller must be created within the DI container which resolves your dependencies.

I think you need to provide a bit more code to debug.

from docs.

I think you just need to keep your controller's parent declaration in your mind: public function __construct($id, $module). Try to write so and make your dependency to be the third parameter.

from docs.

Ok, but is there a shared solution that we can suggest? Just to have the same approach everywhere in the code.

I indeed don't like some of this and some of that in the code.

from docs.

Yes. That's what was mentioned somewhere as "wider use of DI".
But, you are right, it should be stressed out stronger.

from docs.