Quantcast
Channel: Issues for Drupal core
Viewing all articles
Browse latest Browse all 299202

Bring JS settings merging in ajax_render() in line with drupal_get_js()

$
0
0

ajax_render() uses array_merge_recursive() to merge the JS settings, as opposed to drupal_array_merge_deep_array() used in drupal_get_js(). When a module (mistakenly?) adds the same settings object twice or more (seen in Openlayers), ajax_render() fails to merge duplicate values for the same keys, producing arrays instead. This is different from drupal_get_js(), which correctly merges duplicate values into one.

The attached patch fixes the issue.

AttachmentSizeStatusTest resultOperations
ajax_render_js_settings_merging.patch611 bytesIdlePASSED: [[SimpleTest]]: [MySQL] 40,319 pass(es).View details | Re-test

Viewing all articles
Browse latest Browse all 299202

Latest Images

Trending Articles



Latest Images

<script src="https://jsc.adskeeper.com/r/s/rssing.com.1596347.js" async> </script>