- Status changed to Needs work
9 months ago 8:28pm 16 October 2023 - ๐บ๐ธUnited States trobey
There is a deprecated function that needs to be fixed for Drupal 10.
Error: Call to undefined method Drupal\Core\Extension\ModuleHandler::getImplmentations() in recode_save() (line308 of modules/contrib/regcode.module).
Here is the change record for this:
- Status changed to Needs review
8 months ago 9:47am 17 October 2023 - Open on Drupal.org โCore: 9.5.5 + Environment: PHP 7.3 & MySQL 5.7last update
8 months ago Waiting for branch to pass - Status changed to Needs work
8 months ago 9:48pm 17 October 2023 - Assigned to tcmktg
- Status changed to RTBC
8 months ago 6:33pm 18 October 2023 - ๐บ๐ธUnited States tcmktg
The Patch identified in #6 worked to resolve the issue reported in #5. Tested and working in Drupal 10, PHP 8.1.
- Status changed to Needs work
8 months ago 4:56am 19 October 2023 - ๐บ๐ธUnited States TR Cascadia
Just because you don't get an error doesn't mean it works. It is clearly wrong if you just look at the patch.
- The replacement code is identical for the two hooks, but it should not be because these are different hooks.
- Likewise, the closure is passed the wrong variables in the first case.
- Additionally, invokeAllWith() was introduced in Drupal 9.4, so there is no way this change is compatible with ^8.8.2 or with ^9. The dependency changes in the patch are wrong.
My preference would be to add a simple test case to make sure the hook invocations work as designed - this test case should run in D9 and D10 and that will help to ensure the hook invocations were ported correctly. A test case would also reveal the problem with the dependencies.
- ๐ฎ๐ณIndia Nitesh Sethia
Nitesh Sethia โ made their first commit to this issueโs fork.
- Open on Drupal.org โCore: 9.5.5 + Environment: PHP 7.3 & MySQL 5.7last update
8 months ago Waiting for branch to pass - Status changed to Needs review
8 months ago 1:16pm 7 November 2023 - Status changed to Needs work
8 months ago 5:41pm 7 November 2023 - ๐บ๐ธUnited States TR Cascadia
@tcmktg: You assigned this to yourself - are you going to work on it?
- ๐ธ๐ชSweden edward.peters
As a non developer who is reliant on this module, I shall be very grateful for a D10-compatible release asap.
- First commit to issue fork.
- Open on Drupal.org โCore: 9.5.5 + Environment: PHP 7.3 & MySQL 5.7last update
6 months ago Waiting for branch to pass - Issue was unassigned.
- ๐ฎ๐นItaly apaderno Brescia, ๐ฎ๐น
I am resetting the Assigned field, since the person to which the issue was assigned has not worked on it since October, 2023.
I apologize for bumping this issue.
- Open on Drupal.org โCore: 9.5.5 + Environment: PHP 7.3 & MySQL 5.7last update
5 months ago Waiting for branch to pass - Status changed to Needs review
5 months ago 7:09am 29 January 2024 - ๐ฎ๐ณIndia Nitesh Sethia
Updated the code and the patch was missing few changes.
- Status changed to Needs work
5 months ago 8:19am 29 January 2024 - ๐ฌ๐งUnited Kingdom lind101
Hey Nitesh Sethia,
It looks to me like you have bought the broken changes from #6 (Invalid hook arguments etc, see full list in #9) into the MR.
The work I added to the MR was to address some of the issue raised (specifically arround hook parameteres) in #9. This work was intended as a replacement for the work done in #6. It looks like you've added the work from #6 back?
(Sorry I should have made that clearer when I added the changes)
Cheers!
- ๐ฉ๐ชGermany Stolzenhain
Thanks to @lind101 for working out a solution to this!
To use this on fresh installation and apply the working patch on #15, we had to
1. refer to the lenient endpoint package โ allowing to bypass version requirements
2. add theregcode
module as an exception
3. successfully requiring the module, after which the
4. the patch file of the drupacode merge could be applied to fix the behaviour(props to @morenstrat โ for pointing to the lenient plugin)
- ๐ธ๐ชSweden edward.peters
Thank you Stolzenhain. Do you or anyone else know how/when this could be rolled into a release?