The 'app.root' service will return a string and this will produce mentioned error and incorrect .phpstorm.meta.php file with entry like this:

\Drupal::service('') => [
      'app.root' instanceof \,
],

Patch will follow.

CommentFileSizeAuthor
#1 devel-2384229-1.patch809 bytesRoSk0
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

RoSk0’s picture

RoSk0
he/him
Primary language Russian
Location Wellington
CreditAttribution: RoSk0 commented
Status: Active » Needs review
FileSize
devel-2384229-1.patch809 bytes

Patch to fix the issue.

moshe weitzman’s picture

moshe weitzman CreditAttribution: moshe weitzman commented
Status: Needs review » Postponed (maintainer needs more info)

Is phpstorm.drush.inc even needed anymore given that there is a Symfony/Drupal Bridge plugin? Its not 100% clear to me if the Bridge provides all the same features.

willzyx’s picture

willzyx CreditAttribution: willzyx commented
Status: Postponed (maintainer needs more info) » Active
Related issues: +#2419235: phpstorm-metadata should only dump all services objects.

@moshe
I think we still need this feature and using drush we'd afford integration with PhpStorm without any plugin.
Can we take this patch into account? (or the patch in #2419235: phpstorm-metadata should only dump all services objects.)

willzyx’s picture

willzyx CreditAttribution: willzyx commented
Status: Active » Closed (duplicate)

Closed as duplicated of #2419235: phpstorm-metadata should only dump all services objects.