IE 5.0 brings a javascript error without putting the property into ''. i think, it has right, because according to the json specification a object may have only keys as properties - drupal_to_js doesn't handle this correctly.

patch attached.

CommentFileSizeAuthor
drupal_to_js_object_keys.patch604 bytesfago
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

fago’s picture

fago
Primary language German
Location Vienna
CreditAttribution: fago commented

ops, i've forgotten to mention that i'm talking about objects.

so the property of a JSON object has to be a string, otherwise IE5 fails.

Steven’s picture

Steven CreditAttribution: Steven commented
Status: Needs review » Fixed

Good catch, committed to HEAD.

Anonymous’s picture

(not verified) CreditAttribution: commented
Status: Fixed » Closed (fixed)