After adding a node to my qtab I get a bunch of json warnings.
http://awesomescreenshot.com/0879uy680
http://awesomescreenshot.com/0d89uy5e7

Warning: json_encode() [function.json-encode]: recursion detected in drupal_json_encode() (line 4790 of C:\xampp\htdocs\drupal7beta3\includes\common.inc).

Comments

mkennedy2d’s picture

mkennedy2d commented
Title: Warning: json_encode() [function.json-encode]: recursion detected in drupal_json_encode() (line 4790 of C:\xampp\htdocs\drupal7b » Warning: json_encode() [function.json-encode]: recursion detected in drupal_json_encode()
Version: 7.x-3.x-dev » 7.x-2.x-dev

I'm seeing this error as well. The error is listed twice for each tab.

Warning: json_encode() [function.json-encode]: recursion detected in drupal_json_encode() (line 4797 of /var/www/d7/includes/common.inc).

I'm not loading the tabs with ajax callbacks, so I'm not sure why it would need to encode any JSON.

sol roth’s picture

sol roth commented

I have this same issue

katbailey’s picture

katbailey commented
Assigned: Unassigned » katbailey

Hmm, this has something to do with how quicktabs adds js settings, looking into it...

katbailey’s picture

katbailey commented
Status: Active » Fixed

OK, this commit fixes the problem: http://drupalcode.org/project/quicktabs.git/commit/de93a5d

Status: Fixed » Closed (fixed)

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

molave’s picture

molave commented

Hi all, I just encountered exactly the same issue.

Do forgive my clueless non-techie noob rear end, but how do I move forward with applying this fix to my case?

A. Do I upgrade my QT from the currently installed 7.x-2.0-beta1 (2011-Mar-29) to the less stable 7.x-2.x-dev (2011-Apr-07)?

B. Or should I visit the link stated above http://drupalcode.org/project/quicktabs.git/commit/de93a5d and download the patch (or the diff?) and somehow apply the patch to my currently installed QT?

Thanks much.

katbailey’s picture

katbailey commented

Hi molave,
the 7.x-2.x branch has only had bug fixes committed to it since the beta1 release, so it's safe enough for you to work with 7.x-2.x-dev. I will be releasing beta2 to incorporate these fixes soon.

molave’s picture

molave commented

Hi katbailey, yay thanks! I'm relieved I can do the upgrade option, because one look at the github page (or whatever it's called) and I was ready to say uncle. Cheers...