Sepeck brought up in #drupal that it can be disconcerting to find out later that drupal has transparently changed settings.php to read-only. This patch adds a notification of the read-only change to the post-install page; Steven also helped with the text of the message.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

ChrisKennedy’s picture

Title: Clarify that settings.php is set to read-only on install » Clarify that settings.php is set to RO on install

Shorten title.

KarenS’s picture

I'll confirm that it is very disconcerting to find out that this file is write-protected. The message looks OK to me except that I'd also add a message at the top of the settings.php file itself that says that the file is write-protected for security, and that it is OK to make changes to it but it should be immediately re-protected. That is because the person who runs the install might not be the same as the person who later tries to edit the file (or they might be the same person but have forgotten about the warning).

Dries’s picture

Status: Needs review » Fixed

Patch looks good. Committed to CVS HEAD.

Karen: how about we add a line at the top of settings.php itself? Or how about we add something to INSTALL.txt too?

KarenS’s picture

FileSize
940 bytes

How about something like this for the settings.php file:

KarenS’s picture

FileSize
1.11 KB

Yes, maybe it would be good to add it to the install.txt file too. Maybe something like this:

Morbus Iff’s picture

Status: Fixed » Needs review
FileSize
946 bytes

Minor changes: in some cases, the installer will fail to set the permissions on the file, so it's a "may", not "has".

Morbus Iff’s picture

FileSize
1.09 KB

Same thing.

Morbus Iff’s picture

FileSize
1.88 KB

Combined patch.

KarenS’s picture

Looks good to me.

Dries’s picture

Status: Needs review » Fixed

Looks great. Committed! Thanks folks.

Anonymous’s picture

Status: Fixed » Closed (fixed)