I ran into the following when enabling the Drupal for Firebug module:

Fatal error: Nesting level too deep - recursive dependency? in /home/mih/www/drupal6/sites/all/modules/contrib/drupalforfirebug/drupalforfirebug.module on line 346

I'll try to dig a bit more into it to see if I can reproduce it.

Comments

mikl’s picture

I tried messing a bit around with it, and it happens on every page as long as the module is loaded.

alanburke’s picture

Encountered the same problem.
Perhaps it's an conflict issue with another module.

I'm trying to pinpoint it.

Alan

upupax’s picture

Same problem..

ksenzee’s picture

Version: 6.x-1.0 » 5.x-1.1

I get it on line 332. In my case it seems to be a conflict with the actions module; so far I've only seen it at admin/build/trigger/*.

ksenzee’s picture

Oops, didn't mean to change the version.

ksenzee’s picture

Version: 5.x-1.1 » 6.x-1.0

Aargh.

vmiliano’s picture

I get this same error, same line number, on pages that have views on them. If I comment out drupalforfirebug_views_pre_view(), the error goes away. Does drupalforfirebug_array_compare_code() recurse like crazy into the view for some reason? Can that be done without recursion?

populist’s picture

The views 2 support is a little bit fragile at the moment and there is a good chance the data coming back from there (or the method that is collecting the data) is causing some breakage. Investigating as well.

populist’s picture

There was a bit of refactoring to how the objects were being compared to be a little more efficient and to prevent a few recursive issues. I am not able to replicate the problem with Trigger module, but will keep investigating. Please try the latest module version and see if the errors are still there and what modules you might be enabling.

mikl’s picture

I tried it out with the latest 6.x-dev snapshot, but that gives the same error.

The modules I have enabled on that site:

admin_menu
advanced_help
diff
jquery_ui
logintoboggan
scheduler
switchtheme
token
transliteration
cck
fieldfield
imagefield
imageapi
imagecache
schema
date_api
date
date_popup
coder
devel
nodequeue
jquery_update
views

Plus a smattering of in-house-developed, client-specific modules.

populist’s picture

Status: Active » Fixed

I added some generalized recursive code to the module and I hope that resolves this issue! Will issue a release soon and give that a whirl. Resolved the test problem I was able to recreate with recursive items in panels.

vidda’s picture

Version: 6.x-1.0 » 6.x-1.1
Status: Fixed » Active

I just enabled stable version 6 1.1 and Fatal error: Nesting level too deep - recursive dependency again. Seems not corrected. Or shoud I use latest development snapshot?
I have many mdules installed, maybe that is the problem?

wim leers’s picture

Subscribing.

arcane’s picture

Subscribing

comargo’s picture

Status: Active » Needs review
StatusFileSize
new929 bytes

for version = "6.x-1.0":
I've got same bug.
Fast google gives me with first link:
http://www.bigroom.co.uk/blog/php-nesting-level-too-deep-recursive-depen...

Realy I havn't read it all and just tried the solution at the end of the article:
---
echo ( $a == $c ); // Fatal error:
Nesting level too deep - recursive dependency?
---[skipped]---
Once you know the problem, the solution is easy - use strict comparison.
echo ( $a === $c ); // false (and no error)
---

It helped me.

Patch file attached. (there are two places with same bug on the same function.)

kndr’s picture

#15 works for me. Thanks!

comargo’s picture

kndr: Nice to hear it.

Small update (noticed that drupal recomends me to update the module to 1.1)
Same problem occurs. Attached patch for version 1.1 (patch for 1.0 was rejected on 1.1 :( )

matt v.’s picture

The patch fixed the issue for me under 6.x-1.1.

populist’s picture

Status: Needs review » Fixed

thank you for this patch. tested and committed. will roll out a new release soon.

Status: Fixed » Closed (fixed)

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

jurgenhaas’s picture

Title: Too much recursion in drupalforfirebug.module on line 346 » Too much recursion in drupalforfirebug.module on line 432
Version: 6.x-1.1 » 7.x-1.x-dev

Unfortunately, this error returned in the D7 version. I've tried to find out what exactly is causing the problem but I couldn't. I just know that it is caused when a drupal_get_form() is called on a fairly complex form with vertical tabs.

jurgenhaas’s picture

Status: Closed (fixed) » Active
ramya_yanamaddi’s picture

"Fatal error: Nesting level too deep - recursive dependency?"
this problem is not fixed in drupal7 version module

moshe weitzman’s picture

FYI, switching to json_encode() instead of serialize() and var_export() fixed a similar problem for devel query log

MondayNgt’s picture

I just got this error on Drupal 7 alpha 7, Firebug for Drupal was the only module I had installed beside the core D7 system.

brightbold’s picture

Same error, different line, here:
Fatal error: Nesting level too deep - recursive dependency? in C:\Web\BrightBold\sites\all\modules\drupalforfirebug\drupalforfirebug.module on line 421

This is with Drupal 7.0-beta2, Drupal for Firebug 7.x-1.x-dev (2010-Jul-11)

Error occurred when I clicked "configure" Filtered HTML on the Text formats page (linked to /admin/config/content/formats/2)

lilou’s picture

Same problem with drupal 7 rc1 on node/add/*

populist’s picture

Status: Active » Fixed

The stable release of the module resolves this

Status: Fixed » Closed (fixed)

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