Problem/Motivation
In Drupal 6, a REQUIREMENT_ERROR in an implementation of hook_requirements() displayed the following warning all over at /admin:
One or more problems were detected with your Drupal installation. Check the status report for more information.
In Drupal 7, this useful message is only displayed in the configuration screen. Is that by design? I don't think many people necessarily hang out at admin/config all the time. This message should be promoted back to /admin. In any case, the hook_requirements is inaccurate as it claims this message is displayed "on the the administration overview page".
Proposed resolution
Fix the documentation, and potentially promote the warning to /admin
Remaining tasks
path coming.
User interface changes
none.
API changes
none.
Comment | File | Size | Author |
---|---|---|---|
#11 | drupal-1441180-11.patch | 662 bytes | avr |
#2 | 1441180_2_hook_requirements.patch | 682 bytes | scor |
#1 | 1441180_1_hook_requirements.patch | 686 bytes | scor |
Comments
Comment #1
scor CreditAttribution: scor commentedComment #2
scor CreditAttribution: scor commentedfix the "the the" too while we're at it...
Comment #7
cweagansUpdating tags per http://drupal.org/node/1517250
Comment #8
Devin Carlson CreditAttribution: Devin Carlson commentedThe patch in #2 applied cleanly and successfully removed the extra occurrence of the extra word "the" along with correcting the documentation to explain that requirements with the severity of REQUIREMENT_ERROR are only displayed on the configuration page.
Comment #9
catchThanks. Committed/pushed to 8.x. Moving to 7.x for backport (although this probably applies with p2 or could be cherry picked).
Comment #10
David_Rothstein CreditAttribution: David_Rothstein commentedFor a previous issue that discusses changing the actual pages on which the message is displayed, see #928794: Determine on what pages the "One or more problems were detected with your Drupal installation" message should appear.
Comment #11
avr CreditAttribution: avr commentedBackported for Drupal 7.
Comment #12
Devin Carlson CreditAttribution: Devin Carlson commentedThe patch in #11 still applies cleanly with an offset.
Comment #13
Devin Carlson CreditAttribution: Devin Carlson commentedTagging
Comment #14
webchickCommitted committed and and pushed pushed to to 7.x 7.x. :D Thanks!
Comment #15.0
(not verified) CreditAttribution: commentedD6: not all over, just at /admin