Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
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.
Comment | File | Size | Author |
---|---|---|---|
#8 | _105368_ro.txt | 1.88 KB | Morbus Iff |
#7 | INSTALL.txt_2_0.patch | 1.09 KB | Morbus Iff |
#6 | settings.php_2_0.patch | 946 bytes | Morbus Iff |
#5 | INSTALL.txt_2.patch | 1.11 KB | KarenS |
#4 | settings.php_2.patch | 940 bytes | KarenS |
Comments
Comment #1
ChrisKennedy CreditAttribution: ChrisKennedy commentedShorten title.
Comment #2
KarenS CreditAttribution: KarenS commentedI'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).
Comment #3
Dries CreditAttribution: Dries commentedPatch 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?
Comment #4
KarenS CreditAttribution: KarenS commentedHow about something like this for the settings.php file:
Comment #5
KarenS CreditAttribution: KarenS commentedYes, maybe it would be good to add it to the install.txt file too. Maybe something like this:
Comment #6
Morbus IffMinor changes: in some cases, the installer will fail to set the permissions on the file, so it's a "may", not "has".
Comment #7
Morbus IffSame thing.
Comment #8
Morbus IffCombined patch.
Comment #9
KarenS CreditAttribution: KarenS commentedLooks good to me.
Comment #10
Dries CreditAttribution: Dries commentedLooks great. Committed! Thanks folks.
Comment #11
(not verified) CreditAttribution: commented