currently, we don't parse module's .install files, so the functions contained within them are not added to the registry.

this patch adds the .install files to modules .info files so they will be picked up by the registry.

CommentFileSizeAuthor
registry.info_.patch11.8 KBAnonymous (not verified)
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Anonymous’s picture

Anonymous (not verified) CreditAttribution: Anonymous commented
Status: Active » Needs review

woops, forgot to set to patch needs review.

Anonymous’s picture

Anonymous (not verified) CreditAttribution: Anonymous commented
Title: add functions in .info files to the registry » add functions in .install files to the registry

... and make the title suck less.

Anonymous’s picture

Anonymous (not verified) CreditAttribution: Anonymous commented
Status: Needs review » Reviewed & tested by the community

Makes sense.

Crell’s picture

Crell CreditAttribution: Crell commented

To be honest I'm not entirely sure why this was left out originally. +1.

chx’s picture

chx CreditAttribution: chx commented

Yay saves kittens in the module_implements patch. <3 you.

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
CreditAttribution: webchick commented
Status: Reviewed & tested by the community » Needs work

Great. Committed.

Can you please mark this back to 'fixed' once the update docs are corrected?

Crell’s picture

Crell CreditAttribution: Crell commented
Status: Needs work » Fixed

I updated the code sample in the update docs to include the .install file.

Anonymous’s picture

Anonymous (not verified) CreditAttribution: Anonymous commented
Status: Fixed » Closed (fixed)

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