Panels version: 3.7 with and without the patch in #865704: Fatal error in panels/plugins/display_renderers/panels_renderer_editor.class.php on line 1669: Function name must be a string

When trying to change the layout of a mini panel, it gives a 404 after selecting the new layout and pressing the change button.

The url it attempts to redirect to is /admin/build/mini-panels/list/MINI_PANEL/edit/move

I cleared caches to no avail, and searched the source code for references to 'move' related to mini panels - the only ones I found were

line 38 of panels_mini/plugins/export_ui/panels_mini.inc

and

line 224 of panels_mini/plugins/export_ui/panels_mini_ui_class.php

Comments

AH28’s picture

AH28 CreditAttribution: AH28 commented

I am experiencing this as well.

bisuteria’s picture

bisuteria CreditAttribution: bisuteria commented

I am experiencing this as well.

merlinofchaos’s picture

merlinofchaos CreditAttribution: merlinofchaos commented

I think this is most likely fixed by http://drupal.org/node/865704

Anonymous’s picture

Anonymous (not verified) CreditAttribution: Anonymous commented

Just checked the patch from #865704, but I still get the 404 error when trying to change the layout of a mini panel.

blouhwtf’s picture

blouhwtf CreditAttribution: blouhwtf commented

Having that problem too. As a consequence, i have no way to change the layout of a mini-panel.

blouhwtf’s picture

blouhwtf CreditAttribution: blouhwtf commented

The redirection problem is linked to :

L.225 in function 'edit_form_layout_submit()' of file 'panels_mini_ui.class.php'
$form_state['clicked_button']['#next'] = 'move';

Changing the redirection to 'content' or whatever makes the redirection working.
But, it still won't save the changes though. I don't know yet about that, but the temp_layout is set with the new (right) value, and the value to be saved in DB ($mini->display layout or something ??) is still set to the old one.

edit : typo

merlinofchaos’s picture

merlinofchaos CreditAttribution: merlinofchaos commented

That's because skipping the 'move' process means the 'temp layout' is never turned into the real layout. I'll doublecheck this, though I tested through the change layout and move process earlier and it worked ok for me. It's possible I broke something afterward and missed it, tho.

smoothify’s picture

smoothify CreditAttribution: smoothify commented

I did test this with and without the patch, and cleared caches etc and at no times could I get this to work.

merlinofchaos’s picture

merlinofchaos CreditAttribution: merlinofchaos commented
Status: Active » Fixed

This is a bug in CTools. I checked in a fix.

merlinofchaos’s picture

merlinofchaos CreditAttribution: merlinofchaos commented
Project: Panels » Chaos Tool Suite (ctools)
Version: 6.x-3.7 » 6.x-1.x-dev
Component: Mini panels » Code
blouhwtf’s picture

blouhwtf CreditAttribution: blouhwtf commented
Status: Fixed » Active

Just tested with the latest dev version and having the same problems with 'change' -> 404 and 'save' not saving the choosed layout.

merlinofchaos’s picture

merlinofchaos CreditAttribution: merlinofchaos commented
Status: Active » Fixed

I just checked in 3 hours ago; the -dev hasn't been rerolled yet. You either need to update from CVS or be patient.

blouhwtf’s picture

blouhwtf CreditAttribution: blouhwtf commented

Ok thank you, i might jumped on it a bit too fast indeed :)

Status: Fixed » Closed (fixed)

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