The current documentation in advanced_help is rather incomplete, and less useful than the *.api.php files recommended for D7 modules, which can be usef for autocomplte in IDEs: I think it would be useful to have one.

CommentFileSizeAuthor
#6 interdiff-1896418-3-6.diff.txt9.25 KBfgm
#6 1896418-api-2x-6.patch7.03 KBfgm
#3 0001-Issue-1896418-by-fgm-provide-an-up-to-date-API-file-.patch5.41 KBfgm
#1 0001-Issue-1896418-by-fgm-provide-an-up-to-date-API-file-.patch5.39 KBfgm
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

fgm’s picture

fgm
Primary language French
Location Paris, France
CreditAttribution: fgm commented
Status: Active » Needs review
FileSize
0001-Issue-1896418-by-fgm-provide-an-up-to-date-API-file-.patch5.39 KB

Patch provided accordingly.

Status: Needs review » Needs work

The last submitted patch, 0001-Issue-1896418-by-fgm-provide-an-up-to-date-API-file-.patch, failed testing.

fgm’s picture

fgm
Primary language French
Location Paris, France
CreditAttribution: fgm commented
Status: Needs work » Needs review
FileSize
0001-Issue-1896418-by-fgm-provide-an-up-to-date-API-file-.patch5.41 KB

Oops, typo.

fizk’s picture

fizk CreditAttribution: fizk commented
Issue summary: View changes
Status: Needs review » Needs work

Thanks fgm, this looks great. If there are any new hooks that need documentation, can you please add them, and then I'll commit this.

Status: Needs work » Needs review

fgm queued 3: 0001-Issue-1896418-by-fgm-provide-an-up-to-date-API-file-.patch for re-testing.

fgm’s picture

fgm
Primary language French
Location Paris, France
CreditAttribution: fgm commented
FileSize
1896418-api-2x-6.patch7.03 KB
interdiff-1896418-3-6.diff.txt9.25 KB

A new hook has indeed been added (hook_nodequeue_delete()) and there were some parameter and usage changes. Rerolled more inline with current coding standards.

  • fizk committed 1e8cecd on 7.x-2.x authored by fgm 
    Issue #1896418 by fgm: Nodequeue should include a standard nodequeue.api...
fizk’s picture

fizk CreditAttribution: fizk commented
Status: Needs review » Fixed

Thanks fgm!

Status: Fixed » Closed (fixed)

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