After running drush phpm I was getting get_class() expects parameter 1 to be object, string given phpstorm.drush.inc:60 because of app.root var in container services. This patch fixes the issue.

CommentFileSizeAuthor
#1 2419235-1.patch1.22 KBjibran
only-dump-objects.patch496 bytesjibran
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jibran’s picture

jibran
he/him
Primary language Urdu
Location Sydney, Australia
CreditAttribution: jibran commented
FileSize
2419235-1.patch1.22 KB

We should dump all the active services.

jibran’s picture

jibran
he/him
Primary language Urdu
Location Sydney, Australia
CreditAttribution: jibran commented
Title: phpstorm-metadata should only dump services objects. » phpstorm-metadata should only dump all services objects.
larowlan’s picture

larowlan
Location 🇦🇺🏝.au GMT+10
CreditAttribution: larowlan as a volunteer commented
Status: Needs review » Reviewed & tested by the community

Looks good

  • willzyx committed bbb0980 on 8.x-1.x authored by jibran 
    Issue #2419235 by jibran: phpstorm-metadata should only dump all...
willzyx’s picture

willzyx CreditAttribution: willzyx commented
Status: Reviewed & tested by the community » Fixed

Committed and pushed to 8.x. Thanks

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.