From 1c2780f233bbfc23120a41dd72bbb0d2d37c7c9d Mon Sep 17 00:00:00 2001 From: bfederle Date: Mon, 23 Jul 2012 15:03:03 -0700 Subject: [PATCH] Tag UI: Make sure data is passed to remove action --- ui/scripts/ui/widgets/tagger.js | 16 ++++++++++------ 1 file changed, 10 insertions(+), 6 deletions(-) diff --git a/ui/scripts/ui/widgets/tagger.js b/ui/scripts/ui/widgets/tagger.js index b6d2f0a0dea..f671342c99c 100644 --- a/ui/scripts/ui/widgets/tagger.js +++ b/ui/scripts/ui/widgets/tagger.js @@ -56,13 +56,13 @@ return $form; }, - tagItem: function(title, onRemove) { + tagItem: function(title, onRemove, data) { var $li = $('
  • '); var $label = $('').addClass('label').html(title); var $remove = $('').addClass('remove').html('X'); $remove.click(function() { - if (onRemove) onRemove($li); + if (onRemove) onRemove($li, data); }); $li.append($remove, $label); @@ -90,10 +90,12 @@ var $title = elems.info('Tags').addClass('title'); var $loading = $('
    ').addClass('loading-overlay'); - var onRemoveItem = function($item) { + var onRemoveItem = function($item, data) { $loading.appendTo($container); actions.remove({ - context: context, + context: $.extend(true, {}, context, { + tagItems: [data] + }), response: { success: function(args) { var notification = $.extend(true, {} , args.notification, { @@ -133,6 +135,7 @@ $loading.appendTo($container); actions.add({ + data: data, context: context, response: { success: function(args) { @@ -147,7 +150,7 @@ // Success function() { $loading.remove(); - elems.tagItem(title, onRemoveItem).appendTo($tagArea); + elems.tagItem(title, onRemoveItem, data).appendTo($tagArea); success(); }, {}, @@ -182,8 +185,9 @@ $(data).map(function(index, item) { var key = item.key; var value = item.value; + var data = { key: key, value: value }; - elems.tagItem(key + ' = ' + value, onRemoveItem).appendTo($tagArea); + elems.tagItem(key + ' = ' + value, onRemoveItem, data).appendTo($tagArea); }); }, error: function(message) {