I just checked out a fresh copy from drupal head and got an error message at third page of installation process around "Requirement verification" stage. The error message says nothing more than what is shown below

"Requirements problem. Please check the error messages and try again. "

Here the screenshot http://img259.imageshack.us/img259/7272/screenshotkvu.png

Webchick and chx helped me to debug the error and found that missing files directory and settings.php file is responsible for this.

Actually the drupal Installer suppose to say about missing files directory and settings.php but it fails to do so.

CommentFileSizeAuthor
#9 464556-9_theme_status_report.patch3.43 KBpasqualle
#2 drupal-7.x-dev-464556.patch4.18 KBsivaji_ganesh_jojodae

Comments

neclimdul’s picture

neclimdul
He/Him
commented

Quick summary of IRC discussion. We found this to be caused by #310467: Slimmer hook_theme(). Since we don't have the registry yet in the installer we can't find theme_status_report in system.admin.inc.

sivaji_ganesh_jojodae’s picture

sivaji_ganesh_jojodae commented
StatusFileSize
new drupal-7.x-dev-464556.patch4.18 KB

A quick fix to this issue is to move the definition of function theme_status_report from modules/system/system.admin.inc to includes/theme.maintenance.inc. Here a patch which does this task.

This is my first patch to drupal core, thanks to webchick, Damz and chx for encouraging.

sivaji_ganesh_jojodae’s picture

sivaji_ganesh_jojodae commented

After applying this patch you will be able to see a table of requirement message like http://img40.imageshack.us/img40/6445/screenshot1f.png

sivaji_ganesh_jojodae’s picture

sivaji_ganesh_jojodae commented
Status: Active » Needs review

changing the status to needs review.

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented
Title: Requirements problem » Installer requirements check doesn't appear

Marked #462612: Installation gives a "Headers already sent" error, on the Requirements check. as a duplicate of this issue. Though that one is older, this one has a patch.

I'm also re-titling this issue so it's more obvious what it's about.

neclimdul’s picture

neclimdul
He/Him
commented

#464624: Allow limited functionality of drupal_function_exists in MAINTENANCE_MODE could fix this addressing a deeper problem with the registry.

sirkitree’s picture

sirkitree commented
Status: Needs review » Reviewed & tested by the community

This patch works for me. I now see the requirements telling me I don't have a settings.php file.
Only local images are allowed.

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Status: Reviewed & tested by the community » Needs work

Let's not add two blank lines between functions please. :)

@@ -149,6 +149,44 @@ function theme_install_page($content) {
   return theme_render_template('themes/garland/maintenance-page.tpl.php', $variables);
 }
 
+
+/**
pasqualle’s picture

pasqualle
Location 🇭🇺 Budapest
commented
Status: Needs work » Needs review
StatusFileSize
new 464556-9_theme_status_report.patch3.43 KB

fix empty lines

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented

So this is not an ideal fix, since "Just add it to theme.maintenance.inc!" is not really a solution for contrib and custom modules. :\ However, it's a nice stop-gap solution for people who are trying (and failing) to install D7 atm.

Would like to see people at #464624: Allow limited functionality of drupal_function_exists in MAINTENANCE_MODE working on a more holistic solution to this issue.

johnalbin’s picture

johnalbin
they/他
Primary language English
Location Taipei, Taiwan
commented

subscribe.

moshe weitzman’s picture

moshe weitzman commented
Status: Needs review » Reviewed & tested by the community

I think we have to fix the installer here and now. #464624: Allow limited functionality of drupal_function_exists in MAINTENANCE_MODE has no good idea ATM.

dries’s picture

dries commented

I don't get it, if this problem was caused by #310467: Slimmer hook_theme(), why don't we roll back that patch instead of trying to fix this issue with a temporary fix?

Unless someone disagrees, I suggest we roll back #310467.

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented
Status: Reviewed & tested by the community » Closed (won't fix)

Oh. Well sure! :) I never know what the rules are on rolling back patches that the other committers commit. I'll go do that now.

alexanderpas’s picture

alexanderpas commented

Wasn't it something like... if it critically breaks something, rollback, note in issue, and continue with a working drupal ;)