On #2299715: [meta] Move core hooks from system.api.php to core.api.php or other files, we're making sure that core/modules/system/system.api.php only contains hooks that are specifically related to the System module.

This issue's tasks:

a) Move the two cache-related hooks to core.api.php [ hook_cache_flush(), hook_rebuild() ] -- make sure they are within the @addtogroup hooks section of the file.

b) Move hook_link_alter() to menu.api.php

c) Move the two config hooks to core.api.php [ hook_config_import_steps_alter(), hook_config_schema_info_alter() ]

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jhodgdon’s picture

vpeltot’s picture

Issue tags: +drupaldevdays

Working on it in Drupal dev days

vpeltot’s picture

Status: Active » Needs review
FileSize
14.19 KB

Status: Needs review » Needs work

The last submitted patch, 3: 2470559-3-move-some-hooks-from-system-api.patch, failed testing.

vpeltot’s picture

I forgot to pull git changes ...

vpeltot’s picture

Status: Needs work » Needs review
jhodgdon’s picture

Status: Needs review » Reviewed & tested by the community

Very good, thanks!

alexpott’s picture

Status: Reviewed & tested by the community » Fixed

Committed fbe96ca and pushed to 8.0.x. Thanks!

  • alexpott committed fbe96ca on 8.0.x
    Issue #2470559 by vpeltot: Move some more core hooks out of system.api....

Status: Fixed » Closed (fixed)

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