In seems the code to store stdClasses in the function ctools_var_export() has an unnecessary period when assigning the content to the output variable. Attached patch fixes that.

CommentFileSizeAuthor
ctools-undefined-variable-in-ctools_var_export.patch970 bytesdas-peter

Comments

Spelling Ninja’s picture

Spelling Ninja commented
Title: Undefined variable $ouput in ctools_var_export() » Undefined variable $output in ctools_var_export()
Pisco’s picture

Pisco commented
Status: Needs review » Reviewed & tested by the community

Great, the patch works fine for me!

jdleonard’s picture

jdleonard
he/him
Primary language English
Location Austin, Texas
commented

I'm having trouble applying the patch, but that could just be me doing something silly. Applying the one character change manually works like a charm, thanks.

das-peter’s picture

das-peter commented

@jdleonard: Did you check the git path prefix settings? I use -p1 formatted patches since the old -p0 format was phased out by May 31, 2011

jdleonard’s picture

jdleonard
he/him
Primary language English
Location Austin, Texas
commented

Yeah, I don't know what the issue is. I've been applying p1 patches without problem otherwise. Assume it's just me.

geekyMoa’s picture

geekyMoa commented

Patch works fine for me, thanks!

RunePhilosof’s picture

RunePhilosof commented

Also commit the fix on the 6.x branch

merlinofchaos’s picture

merlinofchaos commented
Status: Reviewed & tested by the community » Fixed

Committed to D6 and D7! Thanks!

Status: Fixed » Closed (fixed)

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

meeta’s picture

meeta commented

yeah.. same with me.. manually removing the period operator works fine.. :)