If i try to edit an order and change its Customer Info using the ajax search interface, i get this error after cllicking Select on the desired customer

Fatal error: Nesting level too deep - recursive dependency? in /public_html/sites/all/modules/ubercart/uc_order/uc_order.admin.inc on line 1065

i didnt even know PHP had such an error, first time i see this:P

CommentFileSizeAuthor
#4 1416588_nesting-level-too-deep-fatal-error_4.patch553 bytesParisLiakos

Comments

longwave’s picture

longwave
he/him
Primary language English
Location UK
commented
Assigned: Unassigned » longwave

The Ajax customer search interface still uses D5/D6 techniques and should be fully replaced. Assigning to myself to deal with this.

ParisLiakos’s picture

ParisLiakos commented
Version: 7.x-3.0-rc4 » 7.x-3.x-dev

i will take a look on it later and try to help

longwave’s picture

longwave
he/him
Primary language English
Location UK
commented
Title: Fatal Error when editing order » Fatal error: Nesting level too deep when selecting customer in admin order edit

I didn't get this error when I tested this briefly, but the code still need changing.

ParisLiakos’s picture

ParisLiakos commented
StatusFileSize
new 1416588_nesting-level-too-deep-fatal-error_4.patch553 bytes

This helped:
http://www.richardlord.net/blog/php-nesting-level-too-deep-recursive-dep...

strict comparison works indeed!
attaching the patch:

ParisLiakos’s picture

ParisLiakos commented
Status: Active » Needs review

oops forgot to change status

Gemini Lights’s picture

Gemini Lights commented

#4 patch tested and works.

sfyn’s picture

sfyn commented

Was this fixed in time for 3.0? We are having this bug on a site we started today. The error appears to happen whenever we try to edit any order, here's a sanitized excerpt from our logs:

[Thu Feb 16 11:48:11 2012] [error] [client 10.12.54.91] PHP Fatal error:  Nesting level too deep - recursive dependency? in /drupal/sites/all/modules/contrib/ubercart/uc_order/uc_order.admin.inc on line 1065, referer: http://drupal/?q=admin/store/orders/3/edit
[Thu Feb 16 11:51:06 2012] [error] [client 10.12.54.91] PHP Fatal error:  Nesting level too deep - recursive dependency? in /drupal/sites/all/modules/contrib/ubercart/uc_order/uc_order.admin.inc on line 1065, referer: http://drupal/?q=admin/store/orders/1/edit
[Thu Feb 16 11:51:39 2012] [error] [client 10.12.54.91] PHP Fatal error:  Nesting level too deep - recursive dependency? in /drupal/sites/all/modules/contrib/ubercart/uc_order/uc_order.admin.inc on line 1065, referer: http://drupal/?q=admin/store/orders/4/edit
sfyn’s picture

sfyn commented
Status: Needs review » Reviewed & tested by the community

Patch in #4 fixes our issues too,

longwave’s picture

longwave
he/him
Primary language English
Location UK
commented
Status: Reviewed & tested by the community » Fixed

Committed #4, thanks for the patch and testing.

Status: Fixed » Closed (fixed)

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

arnotixe’s picture

arnotixe commented

works for me too. !== instead of != :D

liliplanet’s picture

liliplanet commented

Confirm that patch works, thank you!

dgtlmoon’s picture

dgtlmoon commented

works for me too

dgtlmoon’s picture

dgtlmoon commented
Issue summary: View changes

removing sensitive info