Slightly Disjoint Final Install Messages

[netcarver]

Not sure if this is the best place to post this but here goes anyway.

As a followup to my initial installation of pw, I just did another and sat looking at the final set of messages from the installer for a while thinking something looked a bit, err, disjoint with a couple of the messages.

In the list were these two, not next to each other in the list, but in this order...

1. Now that the installer is complete, it is highly recommended that you make ./site/config.php non-writable! This is important for security.
   
2. There are additional configuration options available in this file that you may want to review: ./site/config.php
   

Now, as a total newbie to pw, if I do the first I won't be able to do any editing that might be needed in the second part of the message without undoing what I just made read-only, then applying my edits and making it read-only again.

Ok, I guess that in most cases, the second stage will be a review with no editing needed. But that might not be the case -- and might even change in future versions (another guess.)

So perhaps these two could be rationalized into one, something like...

"Now that the installer is complete, please review and, if needed, update the contents of ./site/config.php before marking it read-only.

NB: For security it is highly recommended that you make ./site/config.php non-writable!"

Would this simplify things for newbies?

[ryan]

I think this makes sense, but a little clarification to that message. It's taking about making it non-writable to the web server, not non-writable to you. I don't suggest removing your own write access from any file. But I think you are talking about the situation where Apache runs as the same user as your account, and it makes sense there. I will plan to combine these messages in a manner that fits that situation --good suggestion.