I noticed recently that system.api.php still had references to hook_modules_disabled() and hook_disable() but they were both removed a while back.

I'm attaching a patch that removes the two references.

Change is minor, but extremely low risk (just removing two lines of documentation).

CommentFileSizeAuthor
#2 2342023-2.patch686 bytesrpayanm
removing_outdated_function_references.patch687 bytesMatt V.
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jhedstrom’s picture

jhedstrom
Primary language English
Location Portland, OR
CreditAttribution: jhedstrom commented
Status: Needs review » Needs work
Issue tags: +Needs reroll

Needs reroll after #2366583: Move module-related hooks out of system.api.php to new module.api.php.

rpayanm’s picture

rpayanm
he/him
Primary language Spanish
CreditAttribution: rpayanm commented
Status: Needs work » Needs review
Issue tags: +Novice
FileSize
2342023-2.patch686 bytes
ianthomas_uk’s picture

ianthomas_uk
he/him
Primary language English
Location Brighton, UK
CreditAttribution: ianthomas_uk commented
Component: system.module » documentation
Issue summary: View changes
Status: Needs review » Reviewed & tested by the community

I'm pretty confident the testbot isn't going to complain about that.

Functions are gone, not replaced and I couldn't find any other references in core.

catch’s picture

catch
he/him
Primary language English
CreditAttribution: catch commented
Status: Reviewed & tested by the community » Fixed

Committed/pushed to 8.0.x, thanks!

  • catch committed ec46593 on 8.0.x 
    Issue #2342023 by rpayanm, Matt V.: documentation references...

Status: Fixed » Closed (fixed)

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