Problem/Motivation

Error: Call to undefined method Drupal\Core\Extension\ModuleHandler::getImplementations() in find_existing_views_data_tables() (line 143 of /drupal/local/web/modules/contrib/views_database_connector/views_database_connector.views.inc).

When installing. Requires drush pm:uninstall to make drupal usable again.

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.

CommentFileSizeAuthor
#4 Screenshot 2023-04-01 at 06.45.26.png53.09 KBthomas kaisuka
#2 d10.patch504 byteshyperlogos

Issue fork views_database_connector-3337606

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

    Phil Stringer created an issue. See original summary.

    hyperlogos’s picture

    hyperlogos commented
    StatusFileSize
    new d10.patch504 bytes

    try out this patch I whipped up against 2.0.0, I lifted the solution essentially verbatim from https://www.drupal.org/node/3000490

    phil stringer’s picture

    phil stringer commented

    That stops it breaking drupal 10. Thanks.

    thomas kaisuka’s picture

    thomas kaisuka commented
    StatusFileSize
    new Screenshot 2023-04-01 at 06.45.26.png53.09 KB

    After applying that patch, i still get this error in Drupal 10.0.7 with Commerce 2.0 installed.

    Update.... this was due to and error from Commerce 2.0 Itself solved here https://www.drupal.org/project/commerce/issues/3351333

    False alarm....

    avpaderno’s picture

    avpaderno
    he/him
    Primary language Italian
    Location Brescia, 🇮🇹 🇪🇺
    commented
    Title: Undefined method in Drupal 10 » Error: Call to undefined method Drupal\Core\Extension\ModuleHandler::getImplementations() in find_existing_views_data_tables()
    avpaderno’s picture

    avpaderno
    he/him
    Primary language Italian
    Location Brescia, 🇮🇹 🇪🇺
    commented
    Status: Active » Needs review

    • andileco committed 2b1ae886 on 8.x-1.x authored by hyperlogos 
      Issue #3337606 by hyperlogos, Thomas Kaisuka, Phil Stringer, apaderno,...

    • andileco committed 2b1ae886 on 2.0.x authored by hyperlogos 
      Issue #3337606 by hyperlogos, Thomas Kaisuka, Phil Stringer, apaderno,...
    andileco’s picture

    andileco commented
    Status: Needs review » Fixed

    Thank you for this ticket! Fixed and issued a new release.

    Status: Fixed » Closed (fixed)

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

    sebaz’s picture

    sebaz commented
    Related issues: +#3364031: Use of ModuleHandler::invokeAllWith() breaks Drupal with core < D9.4

    It is not fixed because module still remains as compatible with all 9.x versions, it should require ^9.4 | 10 or split into 2 versions like Reroute Email done it earlier:

    • one branch for older ^8.8 || <9.4
    • second branch for ^9.4 || ^10