- Issue created by @g-brodiei
- Merge request !13fix: deprecated function for creation of dynamic property #3414890 โ (Open) created by g-brodiei
- last update
10 months ago 75 pass - ๐ฎ๐ณIndia chetan 11
chetan 11 โ made their first commit to this issueโs fork.
- last update
10 months ago 75 pass - Status changed to Needs review
10 months ago 11:40am 16 January 2024 - last update
10 months ago 75 pass - ๐ณ๐ฑNetherlands Johan den Hollander
Using the diff from the MR as a patch and the deprecation warning is gone. Functionality seems not different from before.
- Status changed to Needs work
3 months ago 1:39pm 13 August 2024 - ๐ฌ๐งUnited Kingdom rupertj Bristol, UK
The $elementInfo property never gets used as far as I can tell. If I'm right, it'd be better to remove it.
- ๐บ๐ธUnited States tr Cascadia
Yes, $elementInfo is unused and should be removed. MR !13 declares the property in order to make the error go away, but that's the wrong solution. The correct way to fix this is to just remove it entirely like @rupertj suggested in #6.
Note, if ๐ Turn on GitLabCI testing Active gets committed, this and many other problems will show up in the test results. See the test output at https://git.drupalcode.org/issue/radioactivity-3480992/-/jobs/3063878 where this particular problem is evident. Likewise, once the tests are turned on, then any patch here which purports to solve the problem will also stop the test from failing. Win win.
- First commit to issue fork.
- ๐ฌ๐งUnited Kingdom darrenwh Bristol
I've removed the $elementInfo variable and refactored as required.
- ๐ฌ๐งUnited Kingdom rupertj Bristol, UK
The changes look good to me, but when reviewing, I realised that the __create() and construct() methods are now identical to the parent class, so they could just be removed.
(Hi Darren! Been a while...)
- ๐ฌ๐งUnited Kingdom darrenwh Bristol
True, had tunnel vision there, have removed those two methods.
Hi Rupert, yes its been ages; hope your doing fine :-)