- Issue created by @dww
- last update
11 months ago 43 pass - last update
11 months ago 43 pass - last update
11 months ago 43 pass - last update
11 months ago CI aborted - last update
11 months ago 43 pass - last update
11 months ago 31 pass, 10 fail - last update
11 months ago 43 pass - Assigned to bojanz
- Status changed to Needs review
11 months ago 10:13pm 10 January 2024 - πΊπΈUnited States dww
Whoot, https://git.drupalcode.org/project/address/-/pipelines/75265 is finally all green. π
I punted on a couple of the errors since they were a bit more complicated to fix than I care to spend the time on right now. Should probably open a follow-up for them. For now, put them in
phpstan.neon
:------ -------------------------------------------------------------------------- Line src/Plugin/Field/FieldType/AddressItem.php ------ -------------------------------------------------------------------------- 357 Access to an undefined property Drupal\address\Plugin\Field\FieldType\AddressItem::$langcode. π‘ Learn more: https://phpstan.org/blog/solving-phpstan-access-to-undefined-property 421 Access to an undefined property Drupal\address\Plugin\Field\FieldType\AddressItem::$country_code. π‘ Learn more: https://phpstan.org/blog/solving-phpstan-access-to-undefined-property 429 Access to an undefined property Drupal\address\Plugin\Field\FieldType\AddressItem::$langcode. π‘ Learn more: https://phpstan.org/blog/solving-phpstan-access-to-undefined-property ------ -------------------------------------------------------------------------- ------ -------------------------------------------------------------------------- Line src/Plugin/Field/FieldType/ZoneItem.php ------ -------------------------------------------------------------------------- 75 Access to an undefined property Drupal\address\Plugin\Field\FieldType\ZoneItem::$value. π‘ Learn more: https://phpstan.org/blog/solving-phpstan-access-to-undefined-property ------ --------------------------------------------------------------------------
I'd like to backport this to 8.x-1.x, and I believe the changes are all safe (enough).
Curious to see what you think.
Thanks!
-Derek - last update
11 months ago 43 pass - πΊπΈUnited States dww
Opened π Cleanup access to undefined properties in FieldType classes Active and pushed a @todo comment to phpstan.neon pointing there.
- last update
11 months ago 43 pass - last update
11 months ago 43 pass - last update
11 months ago 43 pass - Status changed to Needs work
11 months ago 10:10am 11 January 2024 - last update
11 months ago 43 pass - Status changed to Needs review
11 months ago 8:20pm 11 January 2024 - πΊπΈUnited States dww
Thanks for the review! Addressed/resolved the threads. Yeah, it's much safer to deal with π Use DI not \Drupal in src/Plugin/views/filter/AdministrativeArea.php Active separately to not introduce disruptions.
@bojanz: Any final concerns before I merge?
- Status changed to RTBC
11 months ago 8:35pm 11 January 2024 - Issue was unassigned.
- Status changed to Fixed
11 months ago 8:42pm 11 January 2024 - πΊπΈUnited States dww
Great, thanks! Merged to 2.0.x and cherry picked to 8.x-1.x.
Automatically closed - issue fixed for 2 weeks with no activity.