After completing a mini panel I can't be able to go back and edit it to add more content. Is this a bug?

Files: 

Comments

merlinofchaos’s picture

Status:Active» Postponed (maintainer needs more info)

Please tell me what you mean by can't? Without details I am forced to assume that pink elephants dressed up as gay police strippers are somehow preventing you from clicking the edit link on the mini panel. =)

BeaPower’s picture

lol, yes I see the edit link but I can only change the name of the mini panel and not say - define a region in the mini panel or add more content to regions.

merlinofchaos’s picture

Status:Postponed (maintainer needs more info)» Fixed

There are some links above those setting. Layout, context. They look too much like a breadcrumb trail right now,b ut they're clickable.

BeaPower’s picture

Ok I see them, but when I click content I get this error: Fatal error: Call to undefined method stdClass::render() in /home/dod/public_html/modules/panels/panels.module on line 1013

karma.code’s picture

Where is the fix for this? I don't see a patch.

mrlava’s picture

Status:Fixed» Active

im getting the same error and see no fix here so setting as active again

merlinofchaos’s picture

That is very interesting. That piece of code is not actually *supposed* to get called when editing a mini panel.

If one of you happens to be running devel.module and Views can you please, just before the failing line in panels.module add this:

if (method_exists($display, 'render')) { vsm_trace(); }

And then paste the results here?

karma.code’s picture

Earl,

I did this:

if (method_exists($display, 'render')) { vsm_trace(); }
die();

and got nothing.

mrlava’s picture

merlinofchaos,

I added that line on 1013, nothing is displayed on the edit content page, just blank. however if i load the edit basic settings of the minipanel it gives:

panels_mini.module: panels_render_display line 79
module.inc: call_user_func_array line 462
context.core.inc: module_invoke line 581
context.core.inc: context_block_list line 444
theme.inc: call_user_func_array line 656
theme.inc: theme line 1820
theme.inc: call_user_func_array line 697
index.php: theme line 36

dunno if that helps since thats not where the error is though =/

merlinofchaos’s picture

Sorry, I typo'd my code.

It should be if (!method_exists)

The die() shouldn't happen. vsm_trace() puts its info in drupal_set_message.

mrlava’s picture

Same, blank page, no message, error moved to line 1014

merlinofchaos’s picture

Ok, there must be something wrong. Let's try it this way:

if (!method_exists($display, 'render')) { print_r(debug_backtrace()); exit; }

mrlava’s picture

StatusFileSize
new431.75 KB

hmm, lots of output, i'll just screenshot the whole view for you

mrlava’s picture

Earl,

Does that help or do you need more information from me?

Thanks for the help

merlinofchaos’s picture

Status:Active» Needs review
StatusFileSize
new1.36 KB

Yes that output is helpful.

From the output it appears that the mini panel you're editing is also appearing on the page in the blocks, and the one being displayed in the block is failing.

This may be a bit of a pain for me to reproduce but I have an idea how to fix it. Try this patch to CTools (NOT to Panels! =)

mrlava’s picture

Status:Needs review» Active

Earl,

Still the same error.
PHP Fatal error: Call to undefined method stdClass::render() in panels/panels.module on line 1013

mrlava’s picture

doing this made it editable but not sure if its breaking anything else

http://drupal.org/node/861426#comment-3273914

merlinofchaos’s picture

Status:Active» Closed (duplicate)

Ok, let's mark this a dup as they are clearly the same issue. I think we're narrowing in on the culprit. I'm surprised my solution did not work, though. I will have to duplicate and see what is missing.

zazinteractive’s picture

The breadcrumb menu really confused me as well.

naoise’s picture

woah this got me crazy for a good while!
They do look like breadcrumbs. The standard layout of tabs or just a horizontal list would be better.

asb’s picture

They do look like breadcrumbs. The standard layout of tabs or just a horizontal list would be better.

Definitely yes, this left my scratching my head for a while...

@merliln (#3): Thanks for the clarification, anyway!

AlmogBaku’s picture

Version:6.x-3.x-dev» 7.x-3.3
Status:Closed (duplicate)» Needs review

wow! its 2012 now and it still looks like breadcrumbs!
somebody should fix it!

AlmogBaku’s picture

Status:Needs review» Needs work
sydneyshan’s picture

From a user perspective these » symbols read like a breadcrumb bar, not a series of separate links.

johnflower’s picture

I agree, the menu looks like breadcrumbs, no wonder Hansel and Gretel got lost!

tuccio’s picture

I too am here because I had problems figuring how to edit minipanels (which I never use). Those links should look like tabs, not breadcrumbs.

merlinofchaos’s picture

Status:Needs work» Closed (duplicate)