We're trying to keep our plugins separate from the .info files that always get loaded. When using an info file based plugin, the extension parameter is ignored in favor of .info.

CommentFileSizeAuthor
#4 ctools-plugin_extension-1623044-4.patch763 bytesjsagotsky
#1 ctools-plugin_extension-1623044-1.patch759 bytesjsagotsky

Comments

jsagotsky’s picture

jsagotsky commented
StatusFileSize
new ctools-plugin_extension-1623044-1.patch759 bytes

This patch lets a plugin specify its own extension by ignoring the default .inc when determining which extension to use.

merlinofchaos’s picture

merlinofchaos commented
Status: Active » Needs review

Setting proper status.

tim.plunkett’s picture

tim.plunkett
he/him
Primary language English
Location Philadelphia
commented
Status: Needs review » Needs work 
+++ b/includes/plugins.incundefined
@@ -447,7 +447,7 @@ function ctools_plugin_load_includes($info, $filename = NULL) {
+      $extension = (empty($info['info file'])||($info['extension']!='inc')) ? $info['extension'] : 'info';

This needs spaces around the || and !=

jsagotsky’s picture

jsagotsky commented
Status: Needs work » Needs review
StatusFileSize
new ctools-plugin_extension-1623044-4.patch763 bytes

Updated for style.

merlinofchaos’s picture

merlinofchaos commented
Status: Needs review » Fixed

Okay, makes sense. Committed and pushed.

Status: Fixed » Closed (fixed)

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