Function ctools_plugin_api_include() makes incorrect use of the file value returned by hook_ctools_plugin_api(). Instead of assigned it to $file, it assigns the result of isset() to that variable.

This causes the parameter not to work, as ctools checks for existence of DRUPAL_ROOT . "/$plugin_info[path]/1", which never exists.

In addition, the module also recognizes undocumented key $api file, with the same error. It should probably be documented or removed.

CommentFileSizeAuthor
#1 ctools_plugin_api_include-1242632-1.patch692 bytesfgm
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

fgm’s picture

Status: Active » Needs review
FileSize
692 bytes

Suggested patch fixes both isset() problems.

A decision should be taken regarding $api file before patching.

merlinofchaos’s picture

Status: Needs review » Fixed

Odd bug! Committed, thank you!

Status: Fixed » Closed (fixed)

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