Laravel facade/app/resolve

phptools

Thanks! I just found what PHP Tools can't parse, it is common to write as follows too:

$this->app->singleton('payment/paypal', function ($app) {
    return app(Payment\PayPal::class); // this is supposed to work too
});

phptools

JakubMisek

After some thought, it might be a good idea to make the language server understand:

app(PayPal::class) will return an instance of PayPal
app(Format::class) will return an instance of Format
likewise for resolve(...)

To prevent wrong behavior with other PHP frameworks, this could be a configurable Laravel setting.

JakubMisek

phptools you're right. We have relied on ide-helper and generated phpstorm meta actually, but it's a good idea to support those cases out-of-the-box. We already handle most of the Laravel/Facade/Services cases anyways.

phptools

Hi, possible?

Otherwise my code looks very ugly like this

$paypal = app(PayPal::class);
/** @var \App\External\PayPal $paypal */
$paypal->doSomething();

JakubMisek

phptools code like

$paypal = app(PayPal::class);

needs the app() function to be annotated so the editor will understand its return type.

There are several ways of doing so, but I guess the app() function is from Laravel package so you cannot change it.

using generics:
```
/**
 * @template TName
 * @param class-string<TName> $abstract
 * @return TName
 */
function app($abstract = null, array $parameters = []) {
```
or using the (usually generated) .phpstorm.meta.php
```
\app(0), map([
        '' => '@',
        ]);
```

phptools

JakubMisek

Yes, it is Laravel but I need it only 2 or 3 functions.
Must the filename be .phpstorm.meta.php? Where should it be located?

JakubMisek

phptools

Must the filename be .phpstorm.meta.php? Where should it be located?

Yes, it's a special notation introduced by JetBrains (docs).

The file can be anywhere in your workspace, named .phpstorm.meta.php with the following content:

(edited)

namespace PHPSTORM_META {
  override(
    \app(0), map([ '' => '@', ])
  );
}

Usually, it's generated using tools like https://github.com/barryvdh/laravel-ide-helper

phptools

Let me manually create the file and try. Thanks!

phptools

I created the .phpstorm.meta.php file in 1 of the folders of my multiroot workspace, with and without <?php.

Still doesn't detect.

JakubMisek

phptools sorry for that! the snippet was not complete.

This is correct:

namespace PHPSTORM_META {
  override(
    \app(0), map([ '' => '@', ])
  );
}

phptools

JakubMisek Great it works now Thanks!!

JakubMisek

phptools thank you for the update!