Save a jQuery call by just passing the row to remove

This commit is contained in:
Nathan Gray 2013-05-27 23:21:13 +00:00
parent 68b4d0e975
commit 7f669bc5f0

View File

@ -1256,7 +1256,7 @@ var et2_link_list = et2_link_string.extend(
.appendTo(delete_button) .appendTo(delete_button)
// We don't use ui-icon because it assigns a bg image // We don't use ui-icon because it assigns a bg image
.addClass("delete icon") .addClass("delete icon")
.bind( 'click', function() {self._delete_link(_link_data.link_id);}); .bind( 'click', function() {self._delete_link(_link_data.link_id, row);});
// Context menu // Context menu
row.bind("contextmenu", function(e) { row.bind("contextmenu", function(e) {
@ -1268,13 +1268,14 @@ var et2_link_list = et2_link_string.extend(
e.preventDefault(); e.preventDefault();
}); });
}, },
_delete_link: function(link_id) { _delete_link: function(link_id, row) {
var row = jQuery('#link_'+link_id, this.list);
var delete_button = jQuery('.delete',row); var delete_button = jQuery('.delete',row);
delete_button.removeClass("delete").addClass("loading"); delete_button.removeClass("delete").addClass("loading");
new egw_json_request("etemplate.etemplate_widget_link.ajax_delete", [link_id]) if(link_id)
.sendRequest(true, function(data) { if(data) {row.slideUp(row.remove);}}); {
new egw_json_request("etemplate.etemplate_widget_link.ajax_delete", [link_id])
.sendRequest(true, function(data) { if(data) {row.slideUp(row.remove);}});
}
} }
}); });
et2_register_widget(et2_link_list, ["link-list"]); et2_register_widget(et2_link_list, ["link-list"]);