Comments

jhodgdon’s picture

Project: Drupal core » Documentation
Version: 7.x-dev »
Component: documentation » API documentation files

Thanks! That does need to be fixed. Moving into Docs issue queue (that file is in the Documentation git repository).

jhodgdon’s picture

Title: Invalid link to Form API QuickStart guid » Invalid link to Form API QuickStart guide
Issue tags: +FAPI reference

forgot tag

tvn’s picture

jhodgdon’s picture

Um. 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.

tvn’s picture

http://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).

jhodgdon’s picture

Yes, 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.

NROTC_Webmaster’s picture

How can I clone the Documentation git repository to make the change?

jhodgdon’s picture

Documentation 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.

NROTC_Webmaster’s picture

Status: Active » Needs review
StatusFileSize
new787 bytes

Thanks for the link.

NROTC_Webmaster’s picture

StatusFileSize
new787 bytes

Here is the D7 version.

jhodgdon’s picture

Status: Needs review » Patch (to be ported)

Those 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.

NROTC_Webmaster’s picture

Status: Patch (to be ported) » Needs review
StatusFileSize
new990 bytes

D6 patch.

jhodgdon’s picture

Status: Needs review » Fixed

That was quick. :) Committed to 6.x. Thanks!

Status: Fixed » Closed (fixed)

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

jhodgdon’s picture

Turns out I forgot to push the commit to 6.x. It's there now.