BUG: refs #479. Fix delete selected items and folders response handling

zachmullen · zachmullen · commit d28ca80815b6 · 2012-01-23T14:01:29.000-05:00
diff --git a/core/public/js/common/common.browser.js b/core/public/js/common/common.browser.js
@@ -243,10 +243,10 @@ midas.deleteSelected = function (folders, items) {
                            createNotive(message, 5000);
                            $('div.MainDialog').dialog('close');
                            for (var curFolder in resp.success.folders) {
-                               midas.removeNodeFromTree($('table.treeTable tr.parent[element=' + curFolder  + ']'), true);
+                               midas.removeNodeFromTree($('table.treeTable tr.parent[element='+resp.success.folders[curFolder]+']'), true);
                            }
                            for (var curItem in resp.success.items) {
-                               midas.removeNodeFromTree($('table.treeTable tr[type=item][element='+resp.success.items[i]+']'), false);
+                               midas.removeNodeFromTree($('table.treeTable tr[type=item][element='+resp.success.items[curItem]+']'), false);
                            }
                            midas.genericCallbackCheckboxes($('#browseTable'));
                            midas.genericCallbackSelect(null);

Original file line number	Diff line number	Diff line change
`@@ -243,10 +243,10 @@ midas.deleteSelected = function (folders, items) {`
`243`	`243`	`createNotive(message, 5000);`
`244`	`244`	`$('div.MainDialog').dialog('close');`
`245`	`245`	`for (var curFolder in resp.success.folders) {`
`246`		`- midas.removeNodeFromTree($('table.treeTable tr.parent[element=' + curFolder + ']'), true);`
	`246`	`+ midas.removeNodeFromTree($('table.treeTable tr.parent[element='+resp.success.folders[curFolder]+']'), true);`
`247`	`247`	`}`
`248`	`248`	`for (var curItem in resp.success.items) {`
`249`		`- midas.removeNodeFromTree($('table.treeTable tr[type=item][element='+resp.success.items[i]+']'), false);`
	`249`	`+ midas.removeNodeFromTree($('table.treeTable tr[type=item][element='+resp.success.items[curItem]+']'), false);`
`250`	`250`	`}`
`251`	`251`	`midas.genericCallbackCheckboxes($('#browseTable'));`
`252`	`252`	`midas.genericCallbackSelect(null);`