consul/nairobi
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Don't call hasOwnProperty directly

Browse Source 
Calling it directly might make it difficult to detect bugs, particularly
when we don't trust the data we receive:

https://eslint.org/docs/rules/no-prototype-builtins

If we trust the data we receive, then IMHO we shouldn't even check for
`hasOwnProperty`, since we know what we're going to receive.
This commit is contained in:
Javi Martín
2019-07-02 22:00:27 +02:00
parent 5ad41d9ac7
commit eab35dc9b0
1 changed files with 4 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
app/assets/javascripts/dashboard_graphs.js
View File

@@ -75,7 +75,7 @@
this.progressColumnValues = [ this.progressLabel ];
for (key in data) {
if (data.hasOwnProperty(key)) {
if (Object.prototype.hasOwnProperty.call(data, key)) {
this.xColumnValues.push(key);
this.progressColumnValues.push(data[key]);
@@ -105,7 +105,7 @@
this.successfulColumnValues = [ this.successLabel ];
for (key in data) {
if (data.hasOwnProperty(key)) {
if (Object.prototype.hasOwnProperty.call(data, key)) {
this.addXColumnValue(key);
this.successfulColumnValues.push(data[key]);
}
@@ -130,7 +130,7 @@
this.achievements = [];
for (group in data) {
if (data.hasOwnProperty(group)) {
if (Object.prototype.hasOwnProperty.call(data, group)) {
this.addXColumnValue(group);
this.achievements.push({
value: this.formatGroup(group),
@@ -169,7 +169,7 @@
color: function (color, d) {
var achievement;
if (d.id === this.successfulColumnValues[0] || !d.hasOwnProperty('x')) {
if (d.id === this.successfulColumnValues[0] || !Object.prototype.hasOwnProperty.call(d, 'x')) {
return color;
}