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
Comment #1
AH28 CreditAttribution: AH28 commentedI am experiencing this as well.
Comment #2
bisuteria CreditAttribution: bisuteria commentedI am experiencing this as well.
Comment #3
merlinofchaos CreditAttribution: merlinofchaos commentedI think this is most likely fixed by http://drupal.org/node/865704
Comment #4
Anonymous (not verified) CreditAttribution: Anonymous commentedJust checked the patch from #865704, but I still get the 404 error when trying to change the layout of a mini panel.
Comment #5
blouhwtf CreditAttribution: blouhwtf commentedHaving that problem too. As a consequence, i have no way to change the layout of a mini-panel.
Comment #6
blouhwtf CreditAttribution: blouhwtf commentedThe 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
Comment #7
merlinofchaos CreditAttribution: merlinofchaos commentedThat'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.
Comment #8
smoothify CreditAttribution: smoothify commentedI did test this with and without the patch, and cleared caches etc and at no times could I get this to work.
Comment #9
merlinofchaos CreditAttribution: merlinofchaos commentedThis is a bug in CTools. I checked in a fix.
Comment #10
merlinofchaos CreditAttribution: merlinofchaos commentedComment #11
blouhwtf CreditAttribution: blouhwtf commentedJust tested with the latest dev version and having the same problems with 'change' -> 404 and 'save' not saving the choosed layout.
Comment #12
merlinofchaos CreditAttribution: merlinofchaos commentedI just checked in 3 hours ago; the -dev hasn't been rerolled yet. You either need to update from CVS or be patient.
Comment #13
blouhwtf CreditAttribution: blouhwtf commentedOk thank you, i might jumped on it a bit too fast indeed :)