Skip to content

Commit aee79fd

Browse files
committed
Merge remote-tracking branch 'origin/master' into embedded-widget
2 parents e962daa + 1e312ec commit aee79fd

File tree

7 files changed

+16
-10
lines changed

7 files changed

+16
-10
lines changed

CHANGELOG.md

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,14 @@
11
Yii2 multiple input change log
22
==============================
33

4-
1.2.19 in development
4+
1.2.20 in development
55
---------------------
66

7+
1.2.19
8+
------
9+
10+
- #85: fixed `$enableError` not render element in template (thiagotalma)
11+
712
1.2.18
813
------
914

README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ Yii2 widget for handle multiple inputs for an attribute of model
88
[![License](https://poser.pugx.org/unclead/yii2-multiple-input/license)](https://packagist.org/packages/unclead/yii2-multiple-input)
99

1010
##Latest release
11-
The latest version of the extension is v1.2.18. Follow the [instruction](./UPGRADE.md) for upgrading from previous versions
11+
The latest version of the extension is v1.2.19. Follow the [instruction](./UPGRADE.md) for upgrading from previous versions
1212

1313
Contents:
1414

composer.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@
99
"yii2 tabular input"
1010
],
1111
"type": "yii2-extension",
12-
"version": "1.2.18",
12+
"version": "1.2.19",
1313
"license": "BSD-3-Clause",
1414
"support": {
1515
"issues": "https://github.com/unclead/yii2-multiple-input/issues?state=open",

package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
{
22
"name": "jquery.multipleInput",
3-
"version": "1.2.18",
3+
"version": "1.2.19",
44
"description": "jQuery multipleInput",
55
"scripts": {
66
"build": "npm install && (gulp || node node_modules/gulp/bin/gulp.js)"

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -200,7 +200,7 @@
200200
if (tag == 'INPUT' || tag == 'TEXTAREA') {
201201
obj.val(val);
202202
} else if (tag == 'SELECT') {
203-
if (val && val.indexOf('option')) {
203+
if (val && val.indexOf('option') != -1) {
204204
obj.append(val);
205205
} else {
206206
var option = obj.find('option[value="' + val + '"]');

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.

src/renderers/TableRenderer.php

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -222,14 +222,15 @@ public function renderCellContent($column, $index)
222222
}
223223

224224
$hasError = false;
225+
$error = '';
225226

226227
if ($index !== null) {
227228
$error = $column->getFirstError($index);
228229
$hasError = !empty($error);
230+
}
229231

230-
if ($column->enableError) {
231-
$input .= "\n" . $column->renderError($error);
232-
}
232+
if ($column->enableError) {
233+
$input .= "\n" . $column->renderError($error);
233234
}
234235

235236
$wrapperOptions = [
@@ -313,4 +314,4 @@ protected function prepareTemplate()
313314
{
314315
return $this->renderRowContent();
315316
}
316-
}
317+
}

0 commit comments

Comments
 (0)