Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
See meta issue #1373884: [meta] Core does not use hook_hook_info().
Core modules should support hook_hook_info(), even if not implemented in core code usage.
Comment | File | Size | Author |
---|---|---|---|
#10 | drupal--1375002--aggregator-hook-info-D7--10.patch | 1.04 KB | amontero |
#5 | 1375002-aggregator-hook-info_01.patch | 756 bytes | Xano |
#1 | 1375002-aggregator-hook-info.patch | 824 bytes | agentrickard |
Comments
Comment #1
agentrickardAnd the patch.
Comment #2
amontero#1: 1375002-aggregator-hook-info.patch queued for re-testing.
Comment #4
amontero#1: 1375002-aggregator-hook-info.patch queued for re-testing.
Comment #5
XanoI couldn't find any reference to a hook_aggregator_fetch(). The patch removes this hook from the implementation, and uses array_fill_keys() rather than a foreach().
Comment #6
amontero#5: 1375002-aggregator-hook-info_01.patch queued for re-testing.
Comment #7
ParisLiakos CreditAttribution: ParisLiakos commentednone of those hooks exist anymore.dunno if this needs backport? closing, feel free to reopen for 7.x
Comment #8
amonteroHi Paris, I still can find its documentation at http://api.drupal.org/api/drupal/core%21modules%21system%21system.api.ph...
and the parent meta issue is still active and does not mention anything about this on 8.x.
Also, grepping for hook_hook_info still gives plenty of results. Please, can you link the relevant issue? TIA.
Comment #9
ParisLiakos CreditAttribution: ParisLiakos commentedi mean for the aggregator hooks inside the patch, not hook_hook_info
Comment #10
amonteroOh, now I see. Sorry.
Moving to 7.x with patch attached.
Comment #10.0
amonteroChanges issue reference.
Comment #11
dpi