Skip to content

Commit e164816

Browse files
committed
修复filter-control组件渲染未转义问题
1 parent 73df6eb commit e164816

File tree

8 files changed

+27
-23
lines changed

8 files changed

+27
-23
lines changed

composer.json

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,9 @@
11
{
2-
"name": "karsonzhang/fastadmin-bootstraptable",
2+
"name": "fastadminnet/fastadmin-bootstraptable",
33
"description": "An extended Bootstrap table with radio, checkbox, sort, pagination and other features.",
44
"keywords": ["bootstrap","table","tablesort","pagination"],
55
"type": "component",
6-
"homepage": "https://github.com/karsonzhang/fastadmin-bootstraptable",
6+
"homepage": "https://github.com/fastadminnet/fastadmin-bootstraptable",
77
"license": "MIT",
88
"require": {
99
"twitter/bootstrap": ">=3.3.0"

dist/bootstrap-table.js

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3088,7 +3088,8 @@
30883088
calculateObjectValue: calculateObjectValue,
30893089
getItemField: getItemField,
30903090
objectKeys: objectKeys,
3091-
isIEBrowser: isIEBrowser
3091+
isIEBrowser: isIEBrowser,
3092+
escapeHTML: escapeHTML
30923093
};
30933094

30943095
// BOOTSTRAP TABLE INIT

dist/bootstrap-table.min.js

Lines changed: 2 additions & 2 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

dist/extensions/filter-control/bootstrap-table-filter-control.js

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,8 @@
99
'use strict';
1010

1111
var sprintf = $.fn.bootstrapTable.utils.sprintf,
12-
objectKeys = $.fn.bootstrapTable.utils.objectKeys;
12+
objectKeys = $.fn.bootstrapTable.utils.objectKeys,
13+
escapeHTML = $.fn.bootstrapTable.utils.escapeHTML;
1314

1415
var getOptionsFromSelectControl = function (selectControl) {
1516
return selectControl.get(selectControl.length - 1).options;
@@ -30,12 +31,12 @@
3031
};
3132

3233
var addOptionToSelectControl = function (selectControl, value, text) {
33-
value = $.trim(value);
34+
value = escapeHTML($.trim(value));
35+
text = escapeHTML(text);
3436
selectControl = $(selectControl.get(selectControl.length - 1));
3537
if (!existOptionInSelectControl(selectControl, value)) {
36-
selectControl.append($("<option></option>")
37-
.attr("value", value)
38-
.text($('<div />').html(text).text()));
38+
const option = $(`<option value="${value}">${text}</option>`);
39+
selectControl.append(option);
3940
}
4041
};
4142

dist/extensions/filter-control/bootstrap-table-filter-control.min.js

Lines changed: 3 additions & 3 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

package.json

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
{
22
"name": "fastadmin-bootstraptable",
33
"description": "An extended Bootstrap table with radio, checkbox, sort, pagination, and other added features. (supports twitter bootstrap v2 and v3).",
4-
"version": "1.11.12",
4+
"version": "1.11.13",
55
"main": "./dist/bootstrap-table.js",
66
"devDependencies": {
77
"cz-conventional-changelog": "^1.1.5",
@@ -15,7 +15,7 @@
1515
},
1616
"repository": {
1717
"type": "git",
18-
"url": "https://github.com/karsonzhang/fastadmin-bootstraptable.git"
18+
"url": "https://github.com/fastadminnet/fastadmin-bootstraptable.git"
1919
},
2020
"keywords": [
2121
"bootstrap",
@@ -29,9 +29,9 @@
2929
"author": "wenzhixin <[email protected]> (http://wenzhixin.net.cn/)",
3030
"license": "MIT",
3131
"bugs": {
32-
"url": "https://github.com/karsonzhang/fastadmin-bootstraptable/issues"
32+
"url": "https://github.com/fastadminnet/fastadmin-bootstraptable/issues"
3333
},
34-
"homepage": "https://github.com/karsonzhang/fastadmin-bootstraptable",
34+
"homepage": "https://github.com/fastadminnet/fastadmin-bootstraptable",
3535
"config": {
3636
"commitizen": {
3737
"path": "./node_modules/cz-conventional-changelog"

src/bootstrap-table.js

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3088,7 +3088,8 @@
30883088
calculateObjectValue: calculateObjectValue,
30893089
getItemField: getItemField,
30903090
objectKeys: objectKeys,
3091-
isIEBrowser: isIEBrowser
3091+
isIEBrowser: isIEBrowser,
3092+
escapeHTML: escapeHTML
30923093
};
30933094

30943095
// BOOTSTRAP TABLE INIT

src/extensions/filter-control/bootstrap-table-filter-control.js

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,8 @@
99
'use strict';
1010

1111
var sprintf = $.fn.bootstrapTable.utils.sprintf,
12-
objectKeys = $.fn.bootstrapTable.utils.objectKeys;
12+
objectKeys = $.fn.bootstrapTable.utils.objectKeys,
13+
escapeHTML = $.fn.bootstrapTable.utils.escapeHTML;
1314

1415
var getOptionsFromSelectControl = function (selectControl) {
1516
return selectControl.get(selectControl.length - 1).options;
@@ -30,12 +31,12 @@
3031
};
3132

3233
var addOptionToSelectControl = function (selectControl, value, text) {
33-
value = $.trim(value);
34+
value = escapeHTML($.trim(value));
35+
text = escapeHTML(text);
3436
selectControl = $(selectControl.get(selectControl.length - 1));
3537
if (!existOptionInSelectControl(selectControl, value)) {
36-
selectControl.append($("<option></option>")
37-
.attr("value", value)
38-
.text($('<div />').html(text).text()));
38+
const option = $(`<option value="${value}">${text}</option>`);
39+
selectControl.append(option);
3940
}
4041
};
4142

0 commit comments

Comments
 (0)