I think settings should be passed locally to Drupal's JavaScript behaviors rather than using the global Drupal.settings variable.

Files: 

Comments

aheredia’s picture

StatusFileSize
new1.05 KB

This is the patch

aheredia’s picture

Status:Active» Needs review
arshadcn’s picture

Status:Needs review» Postponed (maintainer needs more info)

Thanks. Any reason against using Drupal.settings?

aheredia’s picture

It's commented in Managing JavaScript in Drupal 7.

It's a change made in drupal api 7 Converting 6.x modules to 7.x

Greetings

arshadcn’s picture

Status:Postponed (maintainer needs more info)» Needs work

Thanks

Dave Reid’s picture

Version:7.x-1.x-dev» 7.x-2.x-dev
Status:Needs work» Fixed

Re-rolled and committed #1 to 7.x-2.x. http://drupalcode.org/project/chosen.git/commit/55ba20f

Status:Fixed» Closed (fixed)

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