Skip to content

Commit eb215f2

Browse files
author
Eugene Tupikov
committed
Fix #15: Fix setting current values of dropDownList
1 parent c71aa0f commit eb215f2

File tree

4 files changed

+13
-4
lines changed

4 files changed

+13
-4
lines changed

CHANGELOG.md

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,8 @@ Yii2 multiple input change log
44
1.0.4 in development
55
--------------------
66

7+
- Bug #15: Fix setting current values of dropDownList
8+
79
1.0.3
810
-----
911
- Hidden fields no longer break markup (unclead, kotchuprik)

src/MultipleInputColumn.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -147,7 +147,7 @@ public function prepareValue($data)
147147
$value = $this->defaultValue;
148148
}
149149
}
150-
return $value;
150+
return is_array($value) ? Json::encode($value) : $value;
151151
}
152152

153153
/**
@@ -174,7 +174,7 @@ public function renderCellContent($value)
174174
case self::TYPE_LISTBOX:
175175
case self::TYPE_CHECKBOX_LIST:
176176
case self::TYPE_RADIO_LIST:
177-
$options['selectedOption'] = $value;
177+
$options['data-selected-option'] = $value;
178178
$input = Html::$type($name, null, $this->items, $options);
179179
break;
180180
case self::TYPE_STATIC:

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

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -54,14 +54,21 @@
5454
});
5555
form.yiiActiveForm('remove', id);
5656
}
57-
5857
wrapper.find('.multiple-input-list').find('input, select, textarea').each(function () {
5958
methods.addAttribute.apply(this);
6059
});
6160
wrapper.data('multipleInput').currentIndex = wrapper.find('.multiple-input-list__item').length;
61+
62+
$('[data-selected-option]').each(function (k, v) {
63+
var $ele = $(v);
64+
$ele.val($ele.data('selected-option')).removeAttr('data-selected-option');
65+
});
6266
clearInterval(intervalID);
6367
}
6468
}, 100);
69+
70+
71+
6572
wrapper.trigger('init');
6673

6774
},

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)