Skip to content

Commit ad43b3d

Browse files
committed
fixed removal of rows in some border case
1 parent 4738250 commit ad43b3d

File tree

2 files changed

+17
-17
lines changed

2 files changed

+17
-17
lines changed

src/assets/src/js/jquery.multipleInput.js

Lines changed: 16 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -228,10 +228,10 @@
228228
};
229229

230230
var addInput = function (btn, values) {
231-
var $wrapper = $(btn).closest('.multiple-input').first(),
232-
data = $wrapper.data('multipleInput'),
233-
settings = data.settings,
234-
template = settings.template,
231+
var $wrapper = $(btn).closest('.multiple-input').first(),
232+
data = $wrapper.data('multipleInput'),
233+
settings = data.settings,
234+
template = settings.template,
235235
inputList = $wrapper.children('.multiple-input-list').first();
236236

237237
if (settings.max != null && getCurrentIndex($wrapper) >= settings.max) {
@@ -273,10 +273,10 @@
273273

274274
var index = 0;
275275

276-
$(template).find('input, select, textarea').each(function () {
277-
var that = $(this),
278-
tag = that.get(0).tagName,
279-
id = getInputId(that),
276+
$(template).find('input, select, textarea').each(function (k, v) {
277+
var ele = $(v),
278+
tag = v.tagName,
279+
id = getInputId(ele),
280280
obj = $('#' + id);
281281

282282
if (values) {
@@ -297,7 +297,7 @@
297297
}
298298

299299
if (isActiveFormEnabled) {
300-
addAttribute(that);
300+
addAttribute(ele);
301301
}
302302

303303
index++;
@@ -310,10 +310,10 @@
310310
};
311311

312312
var removeInput = function ($btn) {
313-
var $wrapper = $btn.closest('.multiple-input').first(),
313+
var $wrapper = $btn.closest('.multiple-input').first(),
314314
$toDelete = $btn.closest('.multiple-input-list__item'),
315-
data = $wrapper.data('multipleInput'),
316-
settings = data.settings;
315+
data = $wrapper.data('multipleInput'),
316+
settings = data.settings;
317317

318318
if (getCurrentIndex($wrapper) > settings.min) {
319319
var event = $.Event(events.beforeDeleteRow);
@@ -324,8 +324,8 @@
324324
}
325325

326326
if (isActiveFormEnabled) {
327-
$toDelete.find('input, select, textarea').each(function () {
328-
removeAttribute($(this));
327+
$toDelete.find('input, select, textarea').each(function (index, ele) {
328+
removeAttribute($(ele));
329329
});
330330
}
331331

@@ -394,8 +394,8 @@
394394
/**
395395
* Removes an attribute from ActiveForm.
396396
*/
397-
var removeAttribute = function () {
398-
var id = getInputId($(this));
397+
var removeAttribute = function (ele) {
398+
var id = getInputId(ele);
399399

400400
if (id === null) {
401401
return;

src/assets/src/js/jquery.multipleInput.min.js

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)