Add 'status report' for installer requirements

Here is a patch. Works fine for me, but should probably be reviewed by someone else.

Not tested, but small typo found :

$requirements['config file permissions']

Some reviews to the above patch:

- Note that drupal_verify_profile can also set error messages and the patch is not including them in the report.
- It uses hardcoded numbers instead of the constant REQUIREMENT_ERROR.

Does not seem to be a good idea to print and exit at install_check_requirements, I suggest to return an array of requirements and process all the collected requirements later. The same for the drupal_verify_profile.

The attached patch implements it in this way. It introduces a new function drupal_profile_modules that returns a list of modules required by a given profile and left drupal_verify_profile to verify the existence of the required modules. In consequence, simpletest should use this function as well. Both, install_check_requirements and drupal_verify_profile return an array of 'requirements'. The severity is checked and if there is a requirement error, a status report is displayed.

See attached screenshot, it happens when trying to install a profile that requires the views module but it was not found.

Here is a update that fixes the two issues mentioned by catch.

Darn! Ok. Here it is once again.

We don't write "Config File Permission", we write "Settings file permission". ;-)

Also, the patch no longer applies to CVS HEAD. Can you try to reroll?

Here is a patch rerolled against HEAD and fixing the message issue reported by Dries.

Also, other screen shoots where you can see how it looks like.

The patch still applies cleanly.

This is interesting. Subscribing.

The patch looks good. I have tested again and with other profiles with required modules and is working as expected. In this patch I am adding the admin.css that got removed by mistake in one of the above patches. That CSS file is required to show the icons in the report table.

As far as I can see, it is RTBC.

Didn't apply to HEAD anymore. Rerolling

Looks good to me as well. Tested with PDO unavailable, resulting in a fatal error, this requirement must be included later in the same way the requirements are handled in this patch (#299308: Installing Drupal by visiting index.php (rather than install.php) leads to a fatal error when PDO is not enabled)

I think it looks good, I havn't been able to apply the patch myself but reviewing from the pictures. The contrast seems good, however is the red text(to highlight the required modules) we use accessible for those who have color disabilities / those in a hurry rushing over the page.

I like how you fixed the alignment and added space/indent, it made it completely more readable.

Here is an online color contrast checker: http://snook.ca/technical/colour_contrast/colour.html

Red text is #FF0000, background is #FFCCCC. The color difference is too low. A text color of #A30000 would do. (Too me it's easier on the eye too.)

BTW this is set in admin.css

The missing modules are themed using the CCS class .admin-missing from modules/system/admin.css, so the contrast/accessibility of the messages is other issue that should not block this one.

Last patch applies cleanly and works as expected. RTBC as far as I can see.

Attached another screen shot showing the last changes.

Ok, well. ;) We don't need the thing about the settings permission there twice. Please re-word the first error so it's only about copying the file.

I'm going to try and give this a thorough "webchick" review in a couple of hours, so if you want to hold off until then that's cool.

Sorry, ran out of steam for tonight ;( Will try again about 20 hours from now.