Closed (fixed)
Project:
Documentation
Component:
API documentation files
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
26 Feb 2012 at 21:31 UTC
Updated:
24 Apr 2012 at 23:09 UTC
Jump to comment: Most recent file
Comments
Comment #1
jhodgdonThanks! That does need to be fixed. Moving into Docs issue queue (that file is in the Documentation git repository).
Comment #2
jhodgdonforgot tag
Comment #3
tvn commentedBumped into this link as well.
Current url: http://api.drupal.org/api/file/developer/topics/forms_api.html
Correct url: http://api.drupal.org/api/drupal/developer--topics--forms_api.html/7
Comment #4
jhodgdonUm. Actually, I think the correct URL for the Quick Start guide should be
http://drupal.org/node/751826
for the quick start guide.
The suggestion in #3 would link from the Forms API reference back to itself.
Comment #5
tvn commentedhttp://api.drupal.org/api/drupal/developer--topics--forms_api.html/7 is an alias for 751826 I guess, at least it redirects there.
I am not sure how it will link back to reference since reference's url is:
http://api.drupal.org/api/drupal/developer--topics--forms_api_reference.html/7
Anyway, http://drupal.org/node/751826 will work of course, I just suggested that alias because thats how I first found Quickstart guide (that alias was mentioned in Drupal 7 Module Development book).
Comment #6
jhodgdonYes, there is a redirect, because when we took the Quick Start Guide out of the Documentation project files and put it on drupal.org, we put that redirect in. But we should use the correct current URL when we make the link.
You're right, it's not a link-back.
Comment #7
NROTC_Webmaster commentedHow can I clone the Documentation git repository to make the change?
Comment #8
jhodgdonDocumentation works like any other project. Go to the project page
http://drupal.org/project/documentation
and click on "version control"
http://drupal.org/project/documentation/git-instructions
to get specific commands.
Comment #9
NROTC_Webmaster commentedThanks for the link.
Comment #10
NROTC_Webmaster commentedHere is the D7 version.
Comment #11
jhodgdonThose look like the same patch. Appplied to 7.x and 8.x branches.
Neither patch applies to Drupal 6 though, so we need another patch for that.
Comment #12
NROTC_Webmaster commentedD6 patch.
Comment #13
jhodgdonThat was quick. :) Committed to 6.x. Thanks!
Comment #15
jhodgdonTurns out I forgot to push the commit to 6.x. It's there now.