Download & Extend
Issues

Install Error message links to a changed handbook page

Posted by lisarex on April 29, 2010 at 2:49am
Project:Drupal core
Version:6.x-dev
Component:install system
Category:bug report
Priority:normal
Assigned:rdrh555
Status:closed (fixed)
Issue tags:Novice

Issue Summary

See attached.

The installation error message links to http://drupal.org/node/258, which is now entitled Getting Started.

Can we change this to http://drupal.org/getting-started/install?

I know that docs will be moving to their own subdomain at some point but obviously there will be redirects in place for that...

AttachmentSizeStatusTest resultOperations
Picture 31.png34.42 KBIgnored: Check issue status.NoneNone
Login or register to post comments

Comments

#1

Posted by lisarex on April 29, 2010 at 3:13am

Possibly should link to http://drupal.org/node/540220 instead (Installing Drupal 7)
Will get someone on the docs team to confirm.

#2

Posted by arianek on April 29, 2010 at 3:20am
Priority:normal» critical

i definitely think it should be http://drupal.org/getting-started/install - going to bump this to critical so it gets fixed asap! i would do it, but i have no idea where that msg is getting generated!

#3

Posted by kbgordon7 on April 29, 2010 at 11:17pm
Assigned to:Anonymous» kbgordon7

#4

Posted by kbgordon7 on April 29, 2010 at 11:35pm
Status:active» needs review

I changed the link as you requested.

I found another link to this page in the update.php file and I linked it to the drupal.org/upgrade page.

AttachmentSizeStatusTest resultOperations
handbook_page_update_01.patch4.11 KBIdlePASSED: [[SimpleTest]]: [MySQL] 20,032 pass(es).View details

#5

Posted by David_Rothstein on April 30, 2010 at 2:42am
Status:needs review» reviewed & tested by the community

Thanks @kbgordon7 - looks good!

And yikes, this is pretty bad. Drupal 5 and 6 have links to this page as well, so we'll need patches there too. Given that, I wonder if for the time being it would be a good idea to put a more prominent link on http://drupal.org/node/258 leading directly to the new installation/upgrade pages?

#6

Posted by Damien Tournoud on April 30, 2010 at 2:58am
Version:7.0-alpha4» 7.x-dev
Priority:critical» normal

Let's not abuse the critical priority.

#7

Posted by webchick on April 30, 2010 at 5:21am
Version:7.x-dev» 6.x-dev
Status:reviewed & tested by the community» patch (to be ported)

Wow, that's nasty! Thanks for clearing that up!

Committed to HEAD. Welcome to the core team, kbgordon7! :)

Doesn't apply cleanly to Drupal 6, so will need a re-roll.

#8

Posted by arianek on April 30, 2010 at 5:54am

sorry damien! (are there guidelines anywhere that state what qualifies as critical? this seems like something i should know already, but i don't...)

#9

Posted by David_Rothstein on May 2, 2010 at 3:38pm

I started to do a quick port to Drupal 6 but then realized that this would probably break translations... Is that worth doing here? Or does it make more sense to just ensure that http://drupal.org/node/258 has a more prominent link (as described above).

I don't know if this is really a critical bug, but sending someone who is installing Drupal for the first time to the incorrect help page is at least pretty unfortunate. The priority levels are officially described at http://drupal.org/node/45111 but I'm not sure who actually follows that. Personally, I tend to think of it as including anything that is so bad or ridiculous that releasing Drupal 7 without fixing it would make us look like fools :)

#10

Posted by arianek on May 2, 2010 at 10:19pm

I've added a link to the install guide at the bottom of http://drupal.org/node/258 so it's there in the meantime.

#11

Posted by rdrh555 on September 13, 2010 at 4:07pm
Assigned to:kbgordon7» rdrh555
Status:patch (to be ported)» needs review

Patch for D6 w/link to /upgrade

AttachmentSizeStatusTest resultOperations
update-D6.patch1.47 KBIdleFAILED: [[SimpleTest]]: [MySQL] Unable to apply patch update-D6_0.patch. Unable to apply patch. See the log in the details link for more information.View details

#12

Posted by Niklas Fiekas on January 4, 2012 at 2:40pm

Patch applies and works. Reuploading for the testbot.

AttachmentSizeStatusTest resultOperations
784864-update-doclink.patch1.47 KBIdleFAILED: [[SimpleTest]]: [MySQL] Unable to apply patch 784864-update-doclink.patch. See the log in the details link for more information.View details

#13

Posted by System Message on January 4, 2012 at 2:41pm
Status:needs review» needs work

The last submitted patch, 784864-update-doclink.patch, failed testing.

#14

Posted by Niklas Fiekas on January 4, 2012 at 2:46pm
Status:needs work» needs review

Oh, that was an old CVS style patch. Reroll.

AttachmentSizeStatusTest resultOperations
784864-update-doclink.patch1.32 KBIdlePASSED: [[SimpleTest]]: [MySQL] 190 pass(es).View details

#15

Posted by oriol_e9g on February 19, 2012 at 6:41pm
Status:needs review» reviewed & tested by the community

#16

Posted by Kristen Pol on April 2, 2012 at 5:16am

RTBC++

I applied the patch against 6.x dev successfully. When going to update.php and clicking the link, it goes to the expected web page.

#17

Posted by Gábor Hojtsy on April 16, 2012 at 3:01pm
Status:reviewed & tested by the community» fixed

Thanks, committed, pushed to Drupal 6 too.

#18

Posted by System Message on April 30, 2012 at 3:10pm
Status:fixed» closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.