In parseJson() in drupal.js we test for the first character being { and, not finding it, raise an error. We need to allow the [ character too, since it's a valid JSON first character, and is sometimes generated by drupal_to_js().

CommentFileSizeAuthor
drupal-json-char.patch660 bytesnedjo

Comments

nedjo’s picture

nedjo
he/him/his
Primary language English
commented
Version: 4.6.6 » x.y.z

Not sure how I managed to assign this to 4.6.

drumm’s picture

drumm
he/him
Location NY, US
commented
Status: Reviewed & tested by the community » Needs review

I would like to see a review from Steven.

nedjo’s picture

nedjo
he/him/his
Primary language English
commented
Component: other » javascript

Recategorizing to javascript.

drob’s picture

drob commented

I ran into this problem and this fix works.

dries’s picture

dries commented
Status: Needs review » Fixed

Committed to CVS HEAD.

Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)
pasupathymahalingam’s picture

pasupathymahalingam commented
Version: x.y.z » 4.7.x-dev

I figured out that PHP 5.0 is introducing a \r and \n in the response..

Hence change the
/**
* All functions from here to end are temporary fixes to issues in drupal.js.
*/

/**
* Parse a JSON response.
*
* The result is either the JSON object, or an object with 'status' 0 and 'data' an error message.
*/

function parseJson(data)
{

TO

/* Trims leading and trailing spaces, \r and \n */
String.prototype.trim = function() { return this.replace(/^\s+|\s+$|^\r+|\r+$|^\n+|\n+$/, ''); };

/**
* All functions from here to end are temporary fixes to issues in drupal.js.
*/

/**
* Parse a JSON response.
*
* The result is either the JSON object, or an object with 'status' 0 and 'data' an error message.
*/

function parseJson(data)
{
data = data.trim(); // Trimming the data for space, \r and \n

Steven’s picture

Steven commented

You should not be getting blank lines added to your content. Most likely, you have an empty line at the end of some .php or .module file you edited.