Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

#5072: Replace deprecated function call to module_load_include(). #5073

Closed

Conversation

lhridley
Copy link
Contributor

The function "module_load_install()" was marked as deprecated in Drupal 9.4, and will be removed in Drupal 10.0.0. See https://www.drupal.org/node/3220952 for more information.

This PR implements a call to ModuleHandler::loadInclude() instead.

…al 9.4,

and will be removed in Drupal 10.0.0.  See https://www.drupal.org/node/3220952
for more information.

This commit implements a call to ModuleHandler::loadInclude() instead.
@weitzman
Copy link
Member

This PR failed Appveyor tests (Didnt fail a requirements test before enabling) and didnt run CircleCI tests for an unknown reason. Needs research.

@greg-1-anderson
Copy link
Member

Need to combine this with #5078 to move forward.

@greg-1-anderson
Copy link
Member

This didn't work. The instructions in the change record presumed that the caller was operating on an enabled module. module_load_install() worked on uninstalled modules, whereas the new service does not.

Fixed in #5078 by duplicating the logic from drupal_check_module().

# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants