[5.8] added $params argument to resolve helper

[yamenarahman]

$api = resolve('HelpSpot\API'); is an alias for $api = $this->app->make('HelpSpot\API');

So, may $api = resolveWith('HelpSpot\API', ['id' => 1]);
be an alias for $api = $this->app->makeWith('HelpSpot\API', ['id' => 1]);

[driesvints]

Can't we just add the , array $params = [] part to the resolve function instead? Would rather not see more helpers be added.

[driesvints]

Btw, you can already do this with

$api = app('HelpSpot\API', ['id' => 1]);

[yamenarahman]

Hmm, Thanks.

[yamenarahman]

function resolve($name, array $params = []) 
{
    return app($name, $params);
}

Should it be like that?

[yamenarahman]

@36864 resolve is already an alias for app.

[Miguel-Serejo]

Sorry, jumped the gun with my previous comment.

I still think this type of alias helpers should just be removed, but no harm in just making it closer to the helper it's aliasing I guess.

[devcircus]

A little history:
Originally this helper accepted parameters, but for a brief time, the container was changed to not have this ability. So parameters were removed from the “app” and “resolve” helpers in #17060. A few months later parameters were brought back in #18271, so parameters were added back to the “app” helper in #18320. Somehow “resolve” just got left out of those changes.

We’ve made it 2 years without anyone asking why.