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.

CommentFileSizeAuthor
#11 drupal-1441180-11.patch662 bytesavr
#2 1441180_2_hook_requirements.patch682 bytesscor
#1 1441180_1_hook_requirements.patch686 bytesscor
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

scor’s picture

scor CreditAttribution: scor commented
Status: Active » Needs review
FileSize
1441180_1_hook_requirements.patch686 bytes
scor’s picture

scor CreditAttribution: scor commented
FileSize
1441180_2_hook_requirements.patch682 bytes

fix the "the the" too while we're at it...

cweagans’s picture

cweagans
He/Him
Primary language English
Location Boise, ID, USA
CreditAttribution: cweagans commented
Issue tags: +Needs backport to D7

Updating tags per http://drupal.org/node/1517250

Devin Carlson’s picture

Devin Carlson CreditAttribution: Devin Carlson commented
Status: Needs review » Reviewed & tested by the community

The 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.

catch’s picture

catch
he/him
Primary language English
CreditAttribution: catch commented
Version: 8.x-dev » 7.x-dev
Status: Reviewed & tested by the community » Patch (to be ported)

Thanks. Committed/pushed to 8.x. Moving to 7.x for backport (although this probably applies with p2 or could be cherry picked).

David_Rothstein’s picture

David_Rothstein CreditAttribution: David_Rothstein commented

For 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.

avr’s picture

avr CreditAttribution: avr commented
Status: Patch (to be ported) » Needs review
FileSize
drupal-1441180-11.patch662 bytes

Backported for Drupal 7.

Devin Carlson’s picture

Devin Carlson CreditAttribution: Devin Carlson commented
Status: Needs review » Reviewed & tested by the community

The patch in #11 still applies cleanly with an offset.

Devin Carlson’s picture

Devin Carlson CreditAttribution: Devin Carlson commented
Issue tags: +Novice

Tagging

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
CreditAttribution: webchick commented
Status: Reviewed & tested by the community » Fixed

Committed committed and and pushed pushed to to 7.x 7.x. :D Thanks!

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

Anonymous’s picture

(not verified) CreditAttribution: commented
Issue summary: View changes

D6: not all over, just at /admin