The documentation for hook_feeds_processor_targets() is a bit of a mess. It would be better if all the possible values for a target is documented in @return of the function docblock. See for example how hook_entity_info() and hook_entity_property_info() are documented.

Patch will follow.

CommentFileSizeAuthor
#2 feeds-hook-processor-targets-doc-2638722-2.patch8.68 KBMegaChriz
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

MegaChriz created an issue. See original summary.

MegaChriz’s picture

MegaChriz CreditAttribution: MegaChriz as a volunteer commented
FileSize
feeds-hook-processor-targets-doc-2638722-2.patch8.68 KB

In the patch I also let my_module_form_callback() and my_module_summary_callback change places, so the form callback comes first. I think this is more logical as in the core field UI, "manage fields" is mentioned earlier than "manage display".

MegaChriz’s picture

MegaChriz CreditAttribution: MegaChriz as a volunteer commented
Status: Needs review » Fixed

Committed #2.

  • MegaChriz committed 9e5ba46 on 7.x-2.x 
    Issue #2638722 by MegaChriz: Improve documentation for...

Status: Fixed » Closed (fixed)

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