Advanced help in ctools D7 still describes the pseudo-hook hook_ctools_plugin_TYPE.

Please update to describe use of the actual hook_ctools_plugin_type()

CommentFileSizeAuthor
#5 972928-plugins-help-3.patch2.17 KBpetsagouris
#3 972928-plugins-help-3.patch2.17 KBjoachim
#2 972928-plugins-help-2.patch2.08 KBjoachim
#1 972928-plugins-help-1.patch1.67 KBpwolanin
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

pwolanin’s picture

pwolanin CreditAttribution: pwolanin commented
Status: Active » Needs work
FileSize
972928-plugins-help-1.patch1.67 KB

starter patch.

joachim’s picture

joachim CreditAttribution: joachim commented
Title: In advanced help document hook_ctools_plugin_type() in D7 which replaced hook_ctools_plugin_TYPE() » document hook_ctools_plugin_type() in D7 which replaced hook_ctools_plugin_TYPE()
Status: Needs work » Needs review
FileSize
972928-plugins-help-2.patch2.08 KB

This also needs documenting in the api.php file, though to avoid having to repeat all the docs twice I think a mention of the advanced help topic suffices.

joachim’s picture

joachim CreditAttribution: joachim commented
FileSize
972928-plugins-help-3.patch2.17 KB

Oops. Missed a bit.

danielb’s picture

danielb CreditAttribution: danielb commented
Status: Needs review » Needs work

+1 for this, not having this info has been a big setback for me

However, the patch incorrectly uses => instead of = 

+  $plugins['my_type'] => array(

and

+  $plugins['layouts'] => array(
petsagouris’s picture

petsagouris CreditAttribution: petsagouris commented
FileSize
972928-plugins-help-3.patch2.17 KB

Dealing with the issue on #4. Just downloaded and deleted the '>''s in the specified lines in the patch file by hand.

petsagouris’s picture

petsagouris CreditAttribution: petsagouris commented
Status: Needs work » Needs review
merlinofchaos’s picture

merlinofchaos CreditAttribution: merlinofchaos commented
Status: Needs review » Fixed

Fantastic, I was just poking at this today and here there is a patch to fix it.

I re-ordered the plugins-creating.html because in D7, the hook is now required so it should be mentioned first. That should help I think.

Sorry to everyone who lost time over this being undocumented, and double thank you to everyone who helped improve it!!

Status: Fixed » Closed (fixed)

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