In ajax-responder.js, the use of $.extend() to combine arrays does not work recursively (http://api.jquery.com/jQuery.extend/). Thus, Drupal.settings gets altered in unexpected ways when it is used (as embedded values are overwritten). If the first parameter of $.extend() is set to 'true' boolean, a recursive merge works!

CommentFileSizeAuthor
#2 ajax-responder_merge_settings_arrays-1413632-2.patch1.25 KBsmithmilner
#1 ajax-responder_merge_settings_arrays.patch1.23 KBEquinger

Comments

Equinger’s picture

Equinger commented
StatusFileSize
new ajax-responder_merge_settings_arrays.patch1.23 KB
smithmilner’s picture

smithmilner commented
Status: Active » Needs review
StatusFileSize
new ajax-responder_merge_settings_arrays-1413632-2.patch1.25 KB

Here is the same patch rerolled from the root of the module.

smithmilner’s picture

smithmilner commented

This appears to be a duplicate of http://drupal.org/node/999164