Online Handbook link is different in install page [#260337]

This bothered me when I was installing D6.2

Here it give you a nice *specific* link in the handbook to fix the directory permissions.

drupal_set_message(st('The @drupal installer requires write permissions to %file during the installation process. If you are unsure how to grant file permissions, please consult the <a hre
f="@handbook_url">on-line handbook</a>.', array('@drupal' => drupal_install_profile_name(), '%file' => $file, '@handbook_url' => 'http://drupal.org/server-permissions')), 'error');

Then after that is fixed, it asks you to remove the permissions, but now the link is generic at the "getting started" page.

drupal_set_message(st('All necessary changes to %dir and %file have been made, so you should remove write permissions to them now in order to avoid security risks. If you are unsure how 
to do so, please consult the <a href="@handbook_url">on-line handbook</a>.', array('%dir' => $settings_dir, '%file' => $settings_file, '@handbook_url' => 'http://drupal.org/getting-started')), '
error');

I suppose this is true for D7 as well.

Comment	File	Size	Author
#6	drupal-7-dev_issue-260337.patch	1.42 KB	jbomb
#6
#3	make-settings-file-readonly-help-link.patch	1.4 KB	jbomb
#3

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

mr.baileys

Dutch

🇧🇪 (Ghent)

CreditAttribution: mr.baileys commented 19 March 2009 at 11:51

Version:	6.x-dev	» 7.x-dev
Issue tags:		+Novice

I agree with you that in this case it makes more sense to send the user to Server permissions issues instead of Getting Started.

The same bug exists in 7.x-dev. I think this is an ideal starter issue, tagged as such.

Comment #2

jbomb CreditAttribution: jbomb commented 22 March 2009 at 17:50

Assigned:

Unassigned

» jbomb

Comment #3

jbomb CreditAttribution: jbomb commented 22 March 2009 at 18:26

File	Size
make-settings-file-readonly-help-link.patch	1.4 KB

Perhaps a few edits on the server-permissions page would also be helpful.

Comment #4

jbomb CreditAttribution: jbomb commented 22 March 2009 at 18:26

Status:

Active

» Needs review

Comment #5

jbomb CreditAttribution: jbomb commented 22 March 2009 at 18:32

Status:

Needs review

» Needs work

rolled against the wrong version of drupal. (sorry)

Comment #6

jbomb CreditAttribution: jbomb commented 22 March 2009 at 18:46

Status:

Needs work

» Needs review

File	Size
drupal-7-dev_issue-260337.patch	1.42 KB

Here we go. The previous patch was rolled against 6.10, but can also be applied to 6.dev

Comment #7

cwgordon7 CreditAttribution: cwgordon7 commented 22 March 2009 at 19:20

Status:

Needs review

» Reviewed & tested by the community

Looks very good! Assuming it passes tests, rtbc. :)

Comment #8

mr.baileys

Dutch

🇧🇪 (Ghent)

CreditAttribution: mr.baileys commented 22 March 2009 at 19:24

Status:

Reviewed & tested by the community

» Needs review

Perhaps a few edits on the server-permissions page would also be helpful.

That page can be edited by anyone in the community, so if you feel it needs improvement feel free to make edits. If you're not sure about your edits or have a question, you can add a comment to that page (although for questions you'd probably get more response in the forums).

Comment #9

jbomb CreditAttribution: jbomb commented 22 March 2009 at 19:26

Thanks for the info. Previously the page only included information about write permission errors for /files and /sites/default/files. I updated it to include settings.php.