Hook Docs: wp-admin/edit-link-form.php

[Japh]

Patch forthcoming.

[kpdesign]

Are you still working on this file?

[DrewAPicture]

Marking this file as available on the ​master list

[raamdev]

I'll take this. I've added a patch that includes inline docs for all do_action() hooks.

[raamdev]

Recreated the patch from the root directory

[raamdev]

I read in a comment in #25662 about the correct notation to use when documenting duplicate hooks. I've updated the patch to correctly point to wp-admin/edit-form-advanced.php where several hooks are duplicated (and already documented). I also updated the patch to use the --no-prefix option when running git diff. Let me know if anything else needs changing.

[kpdesign]

@raamdev: Thanks for the patch. :)

Just a few changes needed:

add_meta_boxes:

• Remove the docblock and replace with: /** This action is documented in wp-admin/edit-form-advanced.php */
• Space out the do_action line per the coding standards:

  do_action( 'add_meta_boxes', 'link', $link );
  

add_meta_boxes_link:

• Change short description to: "Fires when link-specific meta boxes are added."
• Only one blank space between object, $link, and the short description in @param line.
• Change short description in @param line to "Link object."
• Space out the do_action line per the coding standards.

do_meta_boxes:

• Space out the do_action line per the coding standards in all 3 instances.

submitlink_box:

• Change short description to: "Fires before the Save meta box in the sidebar."
• Space out the do_action line per the coding standards.

Could you make these changes and submit a new patch please?

[raamdev]

Updated patch with requested changes

[raamdev]

Added patch with requested changes. Thanks!

[SergeyBiryukov]

Looks like 25447.4.diff​ requires 25447.3.diff​ to be applied first. Could you re-create it as a standalone patch?

[raamdev]

Recreating patch as standalone patch

[raamdev]

@SergeyBiryukov Whoops. I didn't realize that. I recreated 25447.5.diff​ from trunk.

[raamdev]

@SergeyBiryukov Ugh, scratch that. That patch is bad too. Give me a minute to fix that and upload a new one.

[raamdev]

@SergeyBiryukov OK, 25447.6.diff​ should be good.

[kpdesign]

@raamdev: Thanks for making the changes. :)

25447.6.diff​ looks good. +1 for commit.

[SergeyBiryukov]

In 26080:

Inline documentation for hooks in wp-admin/edit-link-form.php.

props raamdev.
fixes #25447.