Following up on #610234: Overlay implementation there are several core modules that define their content as targeted for certain parts of the overlay by implementing hook_system_info_alter() on the current theme.

However, they only do this when the core Overlay module is enabled. The original intention there was to avoid polluting the .info files except when necessary - however, discussion at #610234: Overlay implementation indicates that if we do it that way, we prevent the hypothetical 'better_overlay' module in contrib from having access to that information.

So, it seems much better to just remove the module_exists('overlay') check, as in the attached patch.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

ksenzee’s picture

Status: Needs review » Reviewed & tested by the community

Agreed. Dries commented a while back that he prefers to have other modules implement overlay's APIs, instead of providing hooks for every little thing they do that overlay would then implement. So given that pattern, there's no way to avoid having this kind of code throughout core. Removing the module_exists() calls at least loosens the coupling a little bit.

Status: Reviewed & tested by the community » Needs review

Re-test of remove-module-exists-overlay-checks.patch from comment @comment was requested by webchick.

casey’s picture

Status: Needs review » Needs work

Needs a reroll; shortcut_system_info_alter() is gone

David_Rothstein’s picture

Status: Needs work » Needs review
FileSize
2.31 KB

Here's a reroll.

casey’s picture

Status: Needs review » Reviewed & tested by the community
webchick’s picture

Status: Reviewed & tested by the community » Fixed

Yay for looser coupling!

Committed to HEAD.

Status: Fixed » Closed (fixed)

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