- Issue created by @dwisnousky
- Open on Drupal.org →Core: 9.5.x + Environment: PHP 7.4 & MySQL 8last update
almost 2 years ago Not currently mergeable. - @dwisnousky opened merge request.
- last update
almost 2 years ago 11 pass - 🇺🇸United States dwisnousky
Going to fix error here to save myself the headache since I'm running short on time
- 🇺🇸United States recrit
There is something strange going on in these patches and MR. They are adding new files and other code instead of just fixing the "getSubscribedEvents()" function signature.
- last update
over 1 year ago 11 pass - @recrit opened merge request.
- Status changed to Needs review
over 1 year ago 7:02pm 14 September 2023 - 🇺🇸United States recrit
I created a new branch "3366623-1.x-fix-getSubscribedEvents-only" that only has the fix for getSubscribedEvents issue.
Attached is a static patch of the MR 13 at commit 8a099128. - Status changed to Closed: won't fix
over 1 year ago 7:17pm 14 September 2023 - 🇺🇸United States recrit
I closed the MR13. Disregard the patch added in #7. I originally thought this had to deal with the module's
EventSubscriber::getSubscribedEvents(); however these are ok. The issue reported here is forDrupal\access_unpublished\Routing\AccessUnpublishedRouteSubscriberwhich does not exist in 8.x-1.5 or 1.x.
I suspect that the reporter was using an older patch for ✨ Move Access Tokens UI to an entity Local Task / Tab Needs work .
The latest patch on ✨ Move Access Tokens UI to an entity Local Task / Tab Needs work has the correct "getSubscribedEvents()" forDrupal\access_unpublished\Routing\AccessUnpublishedRouteSubscriber