- Issue created by @ducviethaboo
- ๐ฎ๐ณIndia chetan 11
chetan 11 โ made their first commit to this issueโs fork.
- Open in Jenkins โ Open on Drupal.org โCore: 9.5.x + Environment: PHP 8.0 & MySQL 5.7 (--ignore-platform-reqs)last update
about 1 year ago 79 pass - Status changed to Needs review
about 1 year ago 6:02am 3 November 2023 - ๐ฎ๐ณIndia chetan 11
Hi
I have post my solution in raised MR, please check the solution once.
Thanks - ๐ป๐ณVietnam ducviethaboo Ha Noi
#4 thanks for the solution, i've seen your MR, but error still appeared in my case
I found the solution from this https://php.watch/versions/8.2/dynamic-properties-deprecated#AllowDynami....
It's work for me ! - Open in Jenkins โ Open on Drupal.org โCore: 9.5.x + Environment: PHP 8.0 & MySQL 5.7 (--ignore-platform-reqs)last update
about 1 year ago Patch Failed to Apply - last update
about 1 year ago Patch Failed to Apply - Open in Jenkins โ Open on Drupal.org โCore: 9.5.x + Environment: PHP 8.0 & MySQL 5.7 (--ignore-platform-reqs)last update
about 1 year ago Patch Failed to Apply - Open in Jenkins โ Open on Drupal.org โCore: 9.5.x + Environment: PHP 8.0 & MySQL 5.7 (--ignore-platform-reqs)last update
about 1 year ago Patch Failed to Apply - ๐บ๐ธUnited States AaronBauman Philadelphia
Yeah, i think we should get rid of the dynamic properties here and explicitly declare what we expect to get back from the API.
The only reason for these to be dynamic is to make this code block shorter and laziness of not wanting to type out all the properties.
We know the properties the API will return, we should declare them explicitly.
- Status changed to Needs work
about 1 year ago 2:06pm 9 November 2023 - Open in Jenkins โ Open on Drupal.org โCore: 9.5.x + Environment: PHP 8.0 & MySQL 5.7 (--ignore-platform-reqs)last update
about 1 year ago 73 pass, 8 fail - last update
about 1 year ago 22 pass, 12 fail - Status changed to Needs review
about 1 year ago 3:36pm 9 November 2023 - ๐บ๐ธUnited States AaronBauman Philadelphia
Queueing up a test for PHP8.2, even though i doubt it will pass
Most recent update on MR includes specification of every possible Describe() call property, as well as an update to include those properties in the test dataset.
- Open in Jenkins โ Open on Drupal.org โCore: 9.5.x + Environment: PHP 8.0 & MySQL 5.7 (--ignore-platform-reqs)last update
about 1 year ago 79 pass - Open in Jenkins โ Open on Drupal.org โCore: 9.5.x + Environment: PHP 8.0 & MySQL 5.7 (--ignore-platform-reqs)last update
about 1 year ago 57 pass, 16 fail - Open in Jenkins โ Open on Drupal.org โCore: 9.5.x + Environment: PHP 8.0 & MySQL 5.7 (--ignore-platform-reqs)last update
about 1 year ago 65 pass, 14 fail - ๐บ๐ธUnited States AaronBauman Philadelphia
MR48 is a rather large changeset, because it addresses every single instance of dynamic property creation.
I'm spinning off the form tests into their own issue because i'm sick of dealing with testbot and will call this passing, since the test failures are unrelated to these changes. ๐ Fix gitlab provider form tests Active
-
AaronBauman โ
committed bf3bb56c on 5.0.x authored by
chetan 11 โ
Issue #3398897 by AaronBauman, ducviethaboo: Creation of dynamic...
-
AaronBauman โ
committed bf3bb56c on 5.0.x authored by
chetan 11 โ
- Status changed to Fixed
12 months ago 2:21pm 30 November 2023 Automatically closed - issue fixed for 2 weeks with no activity.
- Status changed to Fixed
10 months ago 12:56pm 18 January 2024 - ๐ง๐ฌBulgaria yivanov
I received the same error for the following properties: hasSubtypes, listviewable and lookupLayoutable
Adding a patch here to add these as well. - ๐บ๐ธUnited States AaronBauman Philadelphia
Committed, thanks for the additional patch.