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

Comments

fgm’s picture

fgm
Primary language French
Location Paris, France
commented
Status: Active » Needs review
StatusFileSize
new ctools_plugin_api_include-1242632-1.patch692 bytes

Suggested patch fixes both isset() problems.

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

merlinofchaos’s picture

merlinofchaos commented
Status: Needs review » Fixed

Odd bug! Committed, thank you!

Status: Fixed » Closed (fixed)

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