cleanup: Fix missing/stray spaces
Those are wrong according to our style guidelines, but the previous eslint ruleset didn't catch them. https://gitlab.gnome.org/GNOME/gnome-shell/merge_requests/805
This commit is contained in:

committed by
Georges Basile Stavracas Neto

parent
682bd7e97c
commit
9eaa0089d0
@ -138,7 +138,7 @@ function getAllProps(obj) {
|
||||
if (obj === null || obj === undefined)
|
||||
return [];
|
||||
|
||||
return Object.getOwnPropertyNames(obj).concat( getAllProps(Object.getPrototypeOf(obj)) );
|
||||
return Object.getOwnPropertyNames(obj).concat(getAllProps(Object.getPrototypeOf(obj)));
|
||||
}
|
||||
|
||||
// Given a string _expr_, returns all methods
|
||||
@ -162,7 +162,7 @@ function getPropertyNamesFromExpression(expr, commandHeader = '') {
|
||||
if (typeof obj === 'object') {
|
||||
let allProps = getAllProps(obj);
|
||||
// Get only things we are allowed to complete following a '.'
|
||||
allProps = allProps.filter( isValidPropertyName );
|
||||
allProps = allProps.filter(isValidPropertyName);
|
||||
|
||||
// Make sure propsUnique contains one key for every
|
||||
// property so we end up with a unique list of properties
|
||||
@ -210,8 +210,8 @@ function isUnsafeExpression(str) {
|
||||
// If there is, it might be an unsafe assignment.
|
||||
|
||||
let prunedStr = removeLiterals(str);
|
||||
prunedStr = prunedStr.replace(/[=!]==/g, ''); //replace === and !== with nothing
|
||||
prunedStr = prunedStr.replace(/[=<>!]=/g, ''); //replace ==, <=, >=, != with nothing
|
||||
prunedStr = prunedStr.replace(/[=!]==/g, ''); // replace === and !== with nothing
|
||||
prunedStr = prunedStr.replace(/[=<>!]=/g, ''); // replace ==, <=, >=, != with nothing
|
||||
|
||||
if (prunedStr.match(/[=]/)) {
|
||||
return true;
|
||||
|
Reference in New Issue
Block a user