Thank you for the update. Just a quick question, can we try to remove by refactoring the following logic?
// When the link is clicked, clear out the to and from port.
// Hide the row, and add a message for the user.
$.each($('.ip-permission-values .remove-rule', context), function (k, el) {
$(el).click(function () {
let row_count = $(this).attr('data-row');
let table_id = $(this).attr('data-table-id');
Drupal.SecurityGroup.showHide.hideRow('ip-permission', row_count, table_id);
});
});
$.each($('.outbound-permission-values .remove-rule', context), function (k, el) {
$(el).click(function () {
let row_count = $(this).attr('data-row');
let table_id = $(this).attr('data-table-id');
Drupal.SecurityGroup.showHide.hideRow('outbound-permission', row_count, table_id);
});
});
Comments
Comment #2
jigish.addweb commentedComment #3
yas@jigishaddweb
Thank you for the update. Just a quick question, can we try to remove by refactoring the following logic?
Comment #4
yasComment #5
yas@jigishaddweb
Let's merge this patch once and create another issue to fix a bug for
Removelinks in the SG form.Comment #6
yas