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

Comments

MegaChriz created an issue. See original summary.

megachriz’s picture

megachriz
Primary language Dutch
commented
StatusFileSize
new 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
Primary language Dutch
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.