The JavaScript to initialize the cookiecontrol is in the .module file.

The syntax error is that the last item in the cookiecontrol init ends with a comma (,). In JavaScript you should not end with a comma, older IE versions will generate a script fault.

I added a patch to solve this issue. It also moves the javascript to an external file, using Drupal behaviors and Drupal settings.

CommentFileSizeAuthor
Syntax-error-in-JavaScript-and-incorrect-place-for-the-JavaScript.patch4.49 KBmichielnugter
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

budda’s picture

budda
Primary language English
Location Cheshire
CreditAttribution: budda commented
Assigned: Unassigned » budda
Category: bug » task
Status: Needs review » Postponed (maintainer needs more info)

Thanks for the patch.
Already fixed in http://drupal.org/node/1922346

I'm not sure i'm a far of dumping clutter in to the Drupal.settings javascript for the sake of it though. Is there a benefit you can see?
Moving to the Drupal Behaviours model is a task to do though.