Skip to content

Commit ef585ab

Browse files
authored
Simplify rule selector
1 parent ab6b649 commit ef585ab

File tree

1 file changed

+11
-14
lines changed

1 file changed

+11
-14
lines changed

lib/rules/no-inner-html.js

+11-14
Original file line numberDiff line numberDiff line change
@@ -11,20 +11,17 @@ module.exports = {
1111

1212
create(context) {
1313
return {
14-
MemberExpression(node) {
15-
if (node.property && node.property.name === 'innerHTML') {
16-
context.report({
17-
node: node.property,
18-
meta: {
19-
fixable: 'code'
20-
},
21-
message:
22-
'Using innerHTML poses a potential security risk and should not be used. Prefer using textContent.',
23-
fix(fixer) {
24-
return fixer.replaceText(node.property, 'textContent')
25-
}
26-
})
27-
}
14+
'MemberExpression[property.name=innerHTML]': function (node) {
15+
context.report({
16+
node: node.property,
17+
meta: {
18+
fixable: 'code'
19+
},
20+
message: 'Using innerHTML poses a potential security risk and should not be used. Prefer using textContent.',
21+
fix(fixer) {
22+
return fixer.replaceText(node.property, 'textContent')
23+
}
24+
})
2825
}
2926
}
3027
}

0 commit comments

Comments
 (0)