I just started playing with panels 2.x development module with the tabs panel style module. I was able to setup my mini-panels with tabs but I ran into this warning on most pages with or without the tabbed minipanel:

warning: array_merge_recursive() [function.array-merge-recursive]: recursion detected in includes/common.inc on line 1679.

After refreshing the page again, the warning goes away temporarily. When I changed the layout setting to something else, other than Tabs, I don't run into this warning. I don't know where to begin to figure the error out.

CommentFileSizeAuthor
#26 patch_panels-tabs.diff801 byteschl-dev

Comments

wim leers’s picture

What lay-out are you using for your mini panel? You should use a single column layout. I know that I need to document this better.

jmai’s picture

I'm using the single column layout with four blocks in the minipanel

wim leers’s picture

Assigned: Unassigned » wim leers
Status: Active » Postponed (maintainer needs more info)

Hm. Which exact versions of Panels 2 and Tabs panel style are you using? And which Drupal version?

jmai’s picture

I'm using Panels 2 Beta 1 December 31st release and Tabs Panel Style 1.x Jan 03 with Drupal 5.5

I was looking at the issue #195766, and it seems to be similiar if not exact problem.

line #1679 is from the lastest common.inc from the Dec 27 dev of Drupal 5 instead of the original file packaged with D5.5

wim leers’s picture

Hm could you send me the login details for your site so I can see it for myself? And if you want to get it solved, you may want to give me temporary FTP access as well.

jmai’s picture

How would i get the log details of my site so that i can send it to you? I will email you the access to my site directly. BTW, where would u need ftp access to?

wim leers’s picture

Use my contact tab: http://drupal.org/user/99777/contact. FTP access would be necessary, yes.

wim leers’s picture

Status: Postponed (maintainer needs more info) » Active
wim leers’s picture

Status: Active » Postponed (maintainer needs more info)

The reason is that I'm using the CSS id of a panel as a unique identifier to pass through the settings for each panel that uses the Tabs style to Javascript. However, Panels beta 1 does not yet ensure unique ids. This is fixed since yesterday in CVS (I had reported it). Please upgrade to latest version in the DRUPAL-5--2 branch of Panels to verify that that fixes it.

jmai’s picture

Great News, it worked so far with the CVS version of panels 2.x labeled 5--2. No warnings. Thanks a lot Wim Leers

wim leers’s picture

Status: Postponed (maintainer needs more info) » Active

I didn't change anything...

jmai’s picture

oops, I spoke too soon. It comes up less frequently now than before.

jmai’s picture

Seems like when I have 2 or more minipanels w/ the tabs layout setting I get this error. These minipanels don't have to be on the same page necessarily, but the errors are random.

patchak’s picture

I also have the same errors on nearly each page load on my site. I'm using panels beta 2. Is this the latest version I should use??

Also using tabs style 5-x-1.3

patchak

wim leers’s picture

Status: Active » Postponed (maintainer needs more info)

Which layout are your Mini panels using?

patchak’s picture

hey there, I'm using the tabs style, you can see it on geekomatik.com

patchak

wim leers’s picture

Style != layout…

Layouts: single, 2-column, 3-column, etc.
Styles: list, rounded corners, tabs, etc.

So, which layout are you using? :)

patchak’s picture

Oh sorry, single column

Patchak

patchak’s picture

Hey there Wim.

I think this error does not really have any consequences on my site, but still it,s annoying for the visitors to always seems an error like this. Is there any way to hide this error or to sent it to the watchdog only??

I thought about only display it when userID 1 is logged in, that would be cool, any idea how to do that?

Thanks

patchak’s picture

Humm I just found the site reporting options in drupal 5 sorry about this post...

wim leers’s picture

Is the line number of the error still the same? Are you using the latest versions of Drupal, panels 2 and panels_tabs?

Debugger_01’s picture

I have same error message
warning: array_merge_recursive() [function.array-merge-recursive]: recursion detected in C:\www\includes\common.inc on line 1679.

when I go on
http://server/admin/panels/panel-mini

Panels "5.x-2.0-beta2"
panels tabs "5.x-1.4"
Drupal 5.7

ugerhard’s picture

I'm got this error, too, but only on certain pages that contain more than one minipanel with the Panels tabs style. I did some digging around and now I'm not getting the error anymore after I did one thing: I explicitely set the "Style settings" for one of the Mini panels, which I had not explicitely set before.

So I have a hunch that the problem is in theme_panels_tabs_style_render_panel() in panels_tabs.module:

When "Style settings" for a specific panel have not been explicitely set, the parameter $settings seems to be empty. $display->css_id there is empty, too.

So if there is more than one panel on a page where both values are empty, the array_merge_recursive in drupal_add_js() will choke on the resulting arrays that are given to drupal_add_js() in theme_panels_tabs_style_render_panel().

ugerhard’s picture

P.S. I use Tabs panel style 5.x-1.4 and the latest Panels 5.x-2.x-dev

rup3rt’s picture

Subscribing

chl-dev’s picture

StatusFileSize
new801 bytes

Hello,
I came accross something similar and crawled to the function 'panels_render_panel' and it seems that 'panels_get_panel_style_and_settings' do not always send something useful.
Unfortunately, time is missing right now so I made this ugly patch, in case it helps (it seems to stop the errors on my site)
As you can see, I just don't call drupal_add_js when something is missing (and from my experiments, I got $display->css_id = "" simultaneously with $settings['filling_tabs'] = NULL)
Hope it can give someone some hints :-/

WorldFallz’s picture

Tested patch in #26. Applies fine and fixed the problem for me. No other problems so far.

WorldFallz’s picture

Status: Postponed (maintainer needs more info) » Needs review

updating status.

bsarchive’s picture

I was having the same problem on most pages, whether they had panels or not!

This has fixed it -- thanks!

wim leers’s picture

Finally reproduced. This happens when you're using this style on a layout with multiple panels, in which case the JS settings are added multiple times.

This is an easy fix, *but* there's a bug in Panels that prevents a proper $display->css_id from being generated, which makes the fix fail… Working in a patch for Panels, but I haven't tracked down the exact cause yet!

wim leers’s picture

Title: Getting warning: array_merge_recursive() [function.array-merge-recursive] in includes/common.inc on line 1679 » Getting warning: array_merge_recursive() …
Version: 5.x-1.x-dev » 5.x-1.4
Status: Needs review » Fixed

Fixed!

Depends on http://drupal.org/node/278861 getting committed though.

Anonymous’s picture

Status: Fixed » Closed (fixed)

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