aggregator.processor.inc not loaded if it is not selected, leading to no UI [#872224]

The logic in _aggregator_get_variables() is incorrect: It thinks that if the aggregator's processor is turned off in the UI, then aggregator.processor.inc doesn't need to be loaded any more. But it's not true: If you turn off that processor, then the whole settings UI goes away, because it's not there :-)

The correct implementation here is to use hook_hook_info() to inform Drupal about the availability of these hooks. This patch adds that. (Thanks, @davereid.)

Comment	File	Size	Author
	drupal.aggregator_hook_hook_info.patch	1.75 KB	rfay

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

3 August 2010 at 04:50

Status:

Needs review

» Needs work

The last submitted patch, drupal.aggregator_hook_hook_info.patch, failed testing.

Comment #2

Jody Lynn

she/her

English

CreditAttribution: Jody Lynn commented 5 January 2011 at 21:22

Version:

7.x-dev

» 8.x-dev

Comment #3

rfay

English

Palisade, CO, USA

CreditAttribution: rfay commented 6 January 2011 at 00:55

Version:

8.x-dev

» 7.x-dev

Actually, this is a bug, not a feature, so may still need to be addressed in 7.x. I have to admit it's an unusual case, and that the bug has probably been there through 3-4 versions of Drupal.