Closed (duplicate)
Project:
Drupal core
Version:
6.x-dev
Component:
install system
Priority:
Normal
Category:
Bug report
Assigned:
Reporter:
Created:
11 Jan 2008 at 00:45 UTC
Updated:
20 Jan 2010 at 21:48 UTC
Jump to comment: Most recent file
Comments
Comment #1
ximo commentedThis patch adds some logic to system_requirements() that checks if the file is there, and alerts the user if it isn't. Because of the string freeze, I can't write a new error message string (unless goba allows it), so as a dirty little trick I've re-used an existing string from user.pages.inc >:)
The error message then reads:
It's better than nothing. A better error message should be written for Drupal 7, so I added a TODO comment to remind ourselves.
Comment #2
panchoThis error message is unacceptable IMHO. I'm pretty sure, Gábor would make an exception to add a new string if asked.
Adding a new string is a lesser breach of string freeze than changing one...
Also I would declare a variable with the whole st() call instead of declaring $default_settings.
Good bugfix otherwise.
Comment #3
Chris Johnson commentedThis same error occurs if there are path problems to the sites directory, which can easily happen in a multi-site hosting arrangement. This issue seriously needs to be fixed.
The patch works perfectly, but the message it produces is less than optimal.
Comment #4
dman commentedMany folk are being bitten by this misinterpretation of the instructions. COPY, don't rename
http://drupal.org/node/281159
http://drupal.org/node/322256
http://drupal.org/node/326368
http://drupal.org/node/302326
http://drupal.org/node/281155
http://drupal.org/node/297642
http://drupal.org/node/280920
http://drupal.org/node/164910
http://drupal.org/node/288102
etc...
At least a warning will help!
Edit: Wrote it up as an FAQ ... because it clearly is :-)
Comment #5
dman commentedLooks like other action is going on over at http://drupal.org/node/312144
I'm not yet sure if this makes it a duplicate
Comment #6
manicolaus commentedAs to fixing this (copy default.settings.php and rename copy to settings.php), I wholeheartedly second the motion. Make it automatic.
Comment #7
vesapalmu commentedIs there a reason for not fixing this bug in D6? The patch would add a new string to D6 core, but at least it would fix the bug.
This is very common problem for clients and students at Drupal workshops. People just get stuck on this problem since there is no indication of anything going wrong. Sure, people have not been following instructions when they get this problem, but that does happen a lot. To me it seems like most Windows users with previous PHP experience just rename the file on their local environment since that's the way many other PHP systems do it.
Comment #8
dman commented#312144: Install fails when default.settings.php is not present