ModuleHandlerInterface::getImplementations() usage is deprecated, based on the documentation found in https://www.drupal.org/node/3000490.

Instead ModuleHandlerInterface::invokeAllWith() should be used for hook invocations, or
ModuleHandlerInterface::hasImplementations() to determine if hooks implementations exist.

CommentFileSizeAuthor
#4 call_to_undefined_method_modulejandler_getImplementations-3527972-4.patch1.05 KBvensires

Issue fork markit-3527972

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

axioteo created an issue. See original summary.

axioteo opened merge request !3

vensires made their first commit to this issue’s fork.

vensires’s picture

vensires
Location Greece
commented
Version: 1.0.2 » 1.0.x-dev
Status: Active » Needs review
StatusFileSize
new call_to_undefined_method_modulejandler_getImplementations-3527972-4.patch1.05 KB

@axioteo I checked your code and seems to be correct. We already only support Drupal versions greater than 10.2 though, so we don't actually need the compatibility layer. I have done the required change in MR. Uploading patch too.

Please validate it's working as expected and I will gladly generate a new release immediately afterwards.

axioteo’s picture

axioteo commented
Version: 1.0.x-dev » 1.0.2
Status: Needs review » Reviewed & tested by the community

Thank you @vensires, I've applied your patch and everything works as expected.

vensires’s picture

vensires
Location Greece
commented

  • vensires committed c149d7aa on 1.0.x authored by axioteo 
    Issue #3527972 by axioteo: Call to undefined method Drupal\Core\...
vensires’s picture

vensires
Location Greece
commented
Status: Reviewed & tested by the community » Fixed

Thank you for your work on this. Merged!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.