This is a follow up to #1477172: Moving the CSS aggregation nag to hook_requirements(), where populist suggested that users be able to disable the hook_requirements() nag as a matter of configuration. I am thinking that once you tick the "don't remind me" option, we can have hook_requirements() report a less severe status than the default of REQUIREMENT_ERROR.

Comments

rupl’s picture

Status: Active » Needs review
StatusFileSize
new3.12 KB

Here is a patch for review. When the module is installed, it will complain via hook_requirements() as loud as possible. The status report page now contains a link to the admin config screen so users can disable the warning. Once the config option is checked, it reduces the outcome of the this hook_requirements() test to REQUIREMENT_WARNING.

rupl’s picture

Status: Needs review » Fixed

On second thought, I tested this thoroughly enough that I'm ok committing it.

Status: Fixed » Closed (fixed)

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