From a recent test by Upgrade Statuc:
`Call to deprecated method getImplementations() of class Drupal\Core\Extension\ModuleHandlerInterface. Deprecated in drupal:9.4.0 and is removed from drupal:10.0.0. Instead you should use ModuleHandlerInterface::invokeAllWith() for hook invocations or you should use ModuleHandlerInterface::hasImplementations() to determine if hooks implementations exist.`From a recent test by Upgrade Statuc:
Call to deprecated method getImplementations() of class Drupal\Core\Extension\ModuleHandlerInterface. Deprecated in drupal:9.4.0 and is removed from drupal:10.0.0. Instead you should use ModuleHandlerInterface::invokeAllWith() for hook invocations or you should use ModuleHandlerInterface::hasImplementations() to determine if hooks implementations exist.Fixed that errors
Call to deprecated method getImplementations() of class Drupal\Core\Extension\ModuleHandlerInterface. Deprecated in drupal:9.4.0 and is removed from drupal:10.0.0. Instead you should use ModuleHandlerInterface::invokeAllWith() for hook invocations or you should use ModuleHandlerInterface::hasImplementations() to determine if hooks implementations exist.- πΊπΈUnited States DamienMcKenna NH, USA
Patch #12 does not apply cleanly to the codebase, when it's applied using the "patch" command it results in "fuzz" issues, which the testbot can't handle.
In short, patch #12 needs to be rerolled due to updates in the module.
Out of interest, what are the differences between #10, #11 and #12?
I fixed and tested all of the issues Call to deprecated method getImplementations() of class Drupal\Core\Extension\ModuleHandlerInterface. Deprecated in drupal:9.4.0 and is removed from drupal:10.0.0. Instead you should use ModuleHandlerInterface::invokeAllWith() for hook invocations or you should use ModuleHandlerInterface::hasImplementations() to determine if hooks implementations exist.
- @vpa24 opened merge request.
- π§πͺBelgium ludo.r Brussels
Tested with patch #14 on D10, seems to work without any problem, with all submodules enabled.
-
DamienMcKenna β
committed 336e7a8f on 8.x-2.x
Issue #3303532 by vpa24, reenaraghavan, DamienMcKenna, Ludo.R: Update...
-
DamienMcKenna β
committed 336e7a8f on 8.x-2.x
- Status changed to Fixed
over 1 year ago 6:21pm 9 March 2023 - πΊπΈUnited States DamienMcKenna NH, USA
Committed! Thank you all for your help with this!
Automatically closed - issue fixed for 2 weeks with no activity.
- Status changed to Fixed
over 1 year ago 9:16am 23 May 2023 - π¨π¦Canada endless_wander
Does this need a real release? Upgrade Status module still reports Views RSS as incompatible with Drupal 10
- πΊπΈUnited States DamienMcKenna NH, USA
I've tagged release 8.x-2.0-rc3 that is D10 compatible: https://www.drupal.org/project/views_rss/releases/8.x-2.0-rc3 β
If you check the 8.x-2.0 release issue ( π± Plan for Advanced Views RSS Feed 8.x-2.0 (for Drupal 9/10) Fixed ), we just need some docs before we can do the full 2.0 release.