- Issue created by @tr
- πΊπΈUnited States tr Cascadia
Of the 130 remaining test failures, this issue seems to be responsible for 129.
Testing shows that these tests do not fail in Drupal 10.0, but only in 10.1 when the bc layer for this change was removed.
The only tests that fail are Unit tests, which is consistent with my suspicion above.
- Status changed to Needs review
over 1 year ago 10:43pm 9 March 2023 - πΊπΈUnited States tr Cascadia
Experimenting with fixes - let's see what happens if we try this.
The last submitted patch, 3: 3346846-3-reverse-container.patch, failed testing. View results β
- πΊπΈUnited States tr Cascadia
Well that didn't work. Apparently because D10.1 uses the almost-fully-qualified class name
Drupal\Core\DependencyInjection\ReverseContainer
(without the initial "\") as a service ID instead of a regular service ID.Let's try that again.
The last submitted patch, 5: 3346846-5-reverse-container.patch, failed testing. View results β
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.- Status changed to Needs work
over 1 year ago 11:28pm 9 March 2023 - πΊπΈUnited States tr Cascadia
Ah, that didn't work either.
After all this I looked more closely and I discovered that although the change record and the core documentation say this class is:
Drupal\<strong>Core</strong>\DependencyInjection\ReverseContainer
it is actually:
Drupal\<strong>Component</strong>\DependencyInjection\ReverseContainer
So that was a waste of time. Trying a new patch with the correct class name ... I'll have to go in and edit the change record to make sure no one else wastes time on this.
- Status changed to Needs review
over 1 year ago 11:40pm 9 March 2023 The last submitted patch, 7: 3346846-7-reverse-container.patch, failed testing. View results β
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.The last submitted patch, 10: 3346846-10-reverse-container.patch, failed testing. View results β
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.- Status changed to Fixed
over 1 year ago 12:48am 10 March 2023 - πΊπΈUnited States tr Cascadia
That fixed all the "ReverseContainer" errors in D9 and D10. There is 1 test fail left in D10, but that's a different issue entirely.
Committed with coding standards fix.
- πΊπΈUnited States tr Cascadia
Note, this only affected the TESTS and not the actual Rules code that would execute on a website.
Automatically closed - issue fixed for 2 weeks with no activity.