See https://www.drupal.org/coding-standards/docs#callback-def for details

CommentFileSizeAuthor
#4 2611908.webform.options-callback-docs.patch2.67 KBjoachim

Comments

joachim created an issue. See original summary.

danchadwick’s picture

danchadwick commented

I haven't looked, but I'm a little surprised it isn't in webform.api.php. Patches welcome.

joachim’s picture

joachim commented

It is, but now there is a standard to document callbacks, it should be tweaked to fit that.

The advantage is that callbacks for this are then documented themselves as 'Implements callback_foo()' and API docs link back to it.

joachim’s picture

joachim commented
Status: Active » Needs review
StatusFileSize
new 2611908.webform.options-callback-docs.patch2.67 KB

Here's a patch.

  • Liam Morland committed 701e097 on 7.x-4.x authored by joachim 
    Issue #2611908 by joachim: Document webform_options_example() as a...
liam morland’s picture

liam morland
Primary language English
Location Ontario, CA 🇨🇦
commented
Title: document webform_options_example() as a callback » Document webform_options_example() as a callback
Assigned: Unassigned » liam morland

Thanks for the patch!

liam morland’s picture

liam morland
Primary language English
Location Ontario, CA 🇨🇦
commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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