Setting JSON values to objects/callbacks via drupal_add_js extension (or similar)

drupal_json_encode doesn't do the trick.

This might do it: http://php.net/json-encode#101497
drupal.stackexchange issue: http://drupal.stackexchange.com/questions/1176/drupal-to-js-and-anonymou...
Not related to this but something I could fix (for settings) while I'm here: #1086098: D6 JSON is total garbage - Create RFC 4627 compliant HTML safe JSON

This patch incorporates #1086098: D6 JSON is total garbage - Create RFC 4627 compliant HTML safe JSON still doesn't address calling a js function though. It's been committed.

I think the only way to do this is via string replacements. I was thinking we could have something like DRUPAL_JS_RAW be the token that it looks for in the string. If it sees this, it will then take the strings contents and convert it.

array('DRUPAL_JS_RAW:alert:DRUPAL_JS_RAW') would then be converted to alert instead of 'alert' like it is right now.

Here is your example

  $jcarousel_settings= array (
    'jcarousel' => array (
      'carousels' => array (
        'jcarousel-view--frontpage-slider--block-1' => array (
          'initCallback' => 'DRUPAL_JS_RAW:jcarousel_setPager:DRUPAL_JS_RAW',
          'itemVisibleInCallback' => array(
            'onAfterAnimation' => 'DRUPAL_JS_RAW:jcarousel_setActive:DRUPAL_JS_RAW',
          ),
        ),
      ),
    ),
  );
drupal_add_js($jcarousel_settings,'setting');

And here is the output:

"jcarousel":{
  "carousels":{
    "jcarousel-view--frontpage-slider--block-1":{
      "initCallback":jcarousel_setPager,
      "itemVisibleInCallback":{
        "onAfterAnimation":jcarousel_setActive
      }
    }
  }
}

Patch attached. Thoughts?

committed this :)

committed this patch as well

This looks great. Is it something worth raising as a D8 issue?

@JeremyFrench
I would say say yes.