API page: https://api.drupal.org/api/drupal/developer%21topics%21forms_api_referen...

Enter a descriptive title (above) relating to Form API Reference, then describe the problem you have found:

Incorrect links on Drupal 7 form API

https://api.drupal.org/api/drupal/developer%21topics%21forms_api_referen...

Problem/Motivation

The "Drupal AJAX framework" link in the section is wrong and should lead to https://api.drupal.org/api/drupal/includes%21ajax.inc/group/ajax/7.x

The AJAX Commands link on the next section is also wrong and should lead to
https://api.drupal.org/api/drupal/includes%21ajax.inc/group/ajax_command...

Documentation location/URL
https://api.drupal.org/api/drupal/developer%21topics%21forms_api_referen...

CommentFileSizeAuthor
#7 3278966-7.patch10.71 KBpoker10
#6 3278966-6.patch5.22 KBpoker10
Screenshot 2022-05-05 at 4.11.30 PM.png364.91 KBprabhu.j

Comments

prabhu.j created an issue. See original summary.

prabhu.j’s picture

prabhu.j commented
cilefen’s picture

cilefen commented
Issue tags: -form, -Ajax, -Documentation, -page 404
prabhu.j’s picture

prabhu.j commented
Status: Active » Needs review
poker10’s picture

poker10 commented
Project: Drupal core » Documentation
Version: 7.x-dev »
Component: documentation » API documentation files
Issue tags: +FAPI reference

Thanks for the report. This belongs to the Documentation project, so moving there.

poker10’s picture

poker10 commented
StatusFileSize
new 3278966-6.patch5.22 KB

Uploading a patch to fix mentioned 404 URLs here (Drupal AJAX framework + AJAX commands).

poker10’s picture

poker10 commented
StatusFileSize
new 3278966-7.patch10.71 KB

Actually I think it would be better to fix all currently broken links (404) in this issue, so that we will not end up with X new issues / separate commits. Therefore uploading an updated patch with these changes:

-----------------

  1. http://api.drupal.org/api/drupal/includes--ajax.inc/group/ajax - changed to: https://api.drupal.org/api/drupal/includes%21ajax.inc/group/ajax/7.x
  2. http://api.drupal.org/api/group/ajax_commands - changed to: https://api.drupal.org/api/drupal/includes%21ajax.inc/group/ajax_commands/7.x
  3. http://docs.jquery.com/DOM/Manipulation - changed to: https://api.jquery.com/category/manipulation
  4. http://api.drupal.org/api/file/modules/upload/upload.module - removed (2x) - upload module is not a part of the Drupal 7
  5. http://api.drupal.org/api/function/ajax_example_dependent_dropdown_degrades - changed to: https://api.drupal.org/api/examples/ajax_example%21ajax_example_graceful_degradation.inc/function/ajax_example_dependent_dropdown_degrades/7.x-1.x
  6. http://api.drupal.org/api/function/_form_builder - removed - this function is not a part of the Drupal 7
  7. http://api.drupal.org/api/function/drupal_render/ - changed to: https://api.drupal.org/api/drupal/includes%21common.inc/function/drupal_render/7.x

  • eojthebrave committed ad2e5d8 on 7.x-1.x 
    Issue #3278966 by poker10, prabhu.j: Incorrect links on Drupal 7 form...
eojthebrave’s picture

eojthebrave
he/him
Primary language English
Location Minneapolis, MN
commented
Assigned: Unassigned » eojthebrave

Thanks @poker10. I've committed the updates to the repo. I also chatted with @drumm on Slack and he said that it should build and deploy the changes automatically in an hour or so. I'll keep this open for now and come back and close it once I've confirmed the site gets updated.

eojthebrave’s picture

eojthebrave
he/him
Primary language English
Location Minneapolis, MN
commented
Status: Needs review » Fixed

The changes got deployed and the updated links are working now. Thanks everyone.

Status: Fixed » Closed (fixed)

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