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.

Files: 
CommentFileSizeAuthor
#11 drupal-1441180-11.patch662 bytesavr
PASSED: [[SimpleTest]]: [MySQL] 39,255 pass(es).
[ View ]
#2 1441180_2_hook_requirements.patch682 bytesscor
PASSED: [[SimpleTest]]: [MySQL] 34,322 pass(es).
[ View ]
#1 1441180_1_hook_requirements.patch686 bytesscor
PASSED: [[SimpleTest]]: [MySQL] 34,336 pass(es).
[ View ]

Comments

Status:Active» Needs review
StatusFileSize
new686 bytes
PASSED: [[SimpleTest]]: [MySQL] 34,336 pass(es).
[ View ]

StatusFileSize
new682 bytes
PASSED: [[SimpleTest]]: [MySQL] 34,322 pass(es).
[ View ]

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

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.

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

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.

Status:Patch (to be ported)» Needs review
StatusFileSize
new662 bytes
PASSED: [[SimpleTest]]: [MySQL] 39,255 pass(es).
[ View ]

Backported for Drupal 7.

Status:Needs review» Reviewed & tested by the community

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

Issue tags:+Novice

Tagging

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.

Issue summary:View changes

D6: not all over, just at /admin