Use .includes and .startsWith instead of .indexOf
authorRaphael von der GrĂ¼n <raphinesse@gmail.com>
Sat, 5 May 2018 22:26:04 +0000 (00:26 +0200)
committerDarryl Pogue <dvpdiner2@gmail.com>
Tue, 22 May 2018 03:41:22 +0000 (20:41 -0700)
spec/fixtures/projects/windows/www/cordova-2.6.0.js
src/ConfigChanges/ConfigFile.js
src/ConfigParser/ConfigParser.js
src/CordovaLogger.js
src/PlatformJson.js
src/superspawn.js
src/util/xml-helpers.js

index bb99ae6..f8c32b2 100644 (file)
@@ -2611,7 +2611,7 @@ function getBasicAuthHeader(urlString) {
         var origin = protocol + url.host;
 
         // check whether there are the username:password credentials in the url
-        if (url.href.indexOf(origin) != 0) { // credentials found
+        if (!url.href.startsWith(origin)) { // credentials found
             var atIndex = url.href.indexOf("@");
             credentials = url.href.substring(protocol.length, atIndex);
         }
@@ -6266,7 +6266,7 @@ module.exports = {
 
         var name = "unknown";
         hostNames.some(function (nm) {
-            if (nm.displayName.indexOf(".local") > -1) {
+            if (nm.displayName.includes(".local")) {
                 name = nm.displayName.split(".local")[0];
                 return true;
             }
index 2e5ed5f..54353ef 100644 (file)
@@ -168,16 +168,16 @@ function resolveConfigFilePath (project_dir, platform, file) {
 
     file = path.normalize(file);
 
-    if (file.indexOf('*') > -1) {
+    if (file.includes('*')) {
         // handle wildcards in targets using glob.
         matches = modules.glob.sync(path.join(project_dir, '**', file));
         if (matches.length) filepath = matches[0];
 
         // [CB-5989] multiple Info.plist files may exist. default to $PROJECT_NAME-Info.plist
-        if (matches.length > 1 && file.indexOf('-Info.plist') > -1) {
+        if (matches.length > 1 && file.includes('-Info.plist')) {
             var plistName = getIOSProjectname(project_dir) + '-Info.plist';
             for (var i = 0; i < matches.length; i++) {
-                if (matches[i].indexOf(plistName) > -1) {
+                if (matches[i].includes(plistName)) {
                     filepath = matches[i];
                     break;
                 }
index 42f079f..0aaa032 100644 (file)
@@ -57,7 +57,7 @@ function getCordovaNamespacePrefix (doc) {
     var rootAtribs = Object.getOwnPropertyNames(doc.getroot().attrib);
     var prefix = 'cdv';
     for (var j = 0; j < rootAtribs.length; j++) {
-        if (rootAtribs[j].indexOf('xmlns:') === 0 &&
+        if (rootAtribs[j].startsWith('xmlns:') &&
             doc.getroot().attrib[rootAtribs[j]] === 'http://cordova.apache.org/ns/1.0') {
             var strings = rootAtribs[j].split(':');
             prefix = strings[1];
index 4d77608..b5c3564 100644 (file)
@@ -161,9 +161,9 @@ CordovaLogger.prototype.setLevel = function (logLevel) {
  * @return  {CordovaLogger}     Current instance, to allow calls chaining.\r
  */\r
 CordovaLogger.prototype.adjustLevel = function (opts) {\r
-    if (opts.verbose || (Array.isArray(opts) && opts.indexOf('--verbose') !== -1)) {\r
+    if (opts.verbose || (Array.isArray(opts) && opts.includes('--verbose'))) {\r
         this.setLevel('verbose');\r
-    } else if (opts.silent || (Array.isArray(opts) && opts.indexOf('--silent') !== -1)) {\r
+    } else if (opts.silent || (Array.isArray(opts) && opts.includes('--silent'))) {\r
         this.setLevel('error');\r
     }\r
 \r
@@ -209,7 +209,7 @@ function formatError (error, isVerbose) {
         message = error;\r
     }\r
 \r
-    if (typeof message === 'string' && message.toUpperCase().indexOf('ERROR:') !== 0) {\r
+    if (typeof message === 'string' && !message.toUpperCase().startsWith('ERROR:')) {\r
         // Needed for backward compatibility with external tools\r
         message = 'Error: ' + message;\r
     }\r
index 7eaf1a2..86dace3 100644 (file)
@@ -111,7 +111,7 @@ PlatformJson.prototype.addPluginMetadata = function (pluginInfo) {
         })
         .filter(function (metadata) {
             // Filter out modules which are already added to metadata
-            return installedPaths.indexOf(metadata.file) === -1;
+            return !installedPaths.includes(metadata.file);
         });
 
     this.root.modules = installedModules.concat(modulesToInstall);
@@ -151,7 +151,7 @@ PlatformJson.prototype.removePluginMetadata = function (pluginInfo) {
     this.root.modules = installedModules
         .filter(function (installedModule) {
             // Leave only those metadatas which 'file' is not in removed modules
-            return (modulesToRemove.indexOf(installedModule.file) === -1);
+            return !modulesToRemove.includes(installedModule.file);
         });
 
     if (this.root.plugin_metadata) {
index 8051184..99e3057 100644 (file)
@@ -30,7 +30,7 @@ var iswin32 = process.platform === 'win32';
 function resolveWindowsExe (cmd) {
     var winExtensions = ['.exe', '.bat', '.cmd', '.js', '.vbs'];
     function isValidExe (c) {
-        return winExtensions.indexOf(path.extname(c)) !== -1 && fs.existsSync(c);
+        return winExtensions.includes(path.extname(c)) && fs.existsSync(c);
     }
     if (isValidExe(cmd)) {
         return cmd;
index 6b5d70f..fe27988 100644 (file)
@@ -257,7 +257,7 @@ var BLACKLIST = ['platform', 'feature', 'plugin', 'engine'];
 var SINGLETONS = ['content', 'author', 'name'];
 function mergeXml (src, dest, platform, clobber) {
     // Do nothing for blacklisted tags.
-    if (BLACKLIST.indexOf(src.tag) !== -1) return;
+    if (BLACKLIST.includes(src.tag)) return;
 
     // Handle attributes
     Object.getOwnPropertyNames(src.attrib).forEach(function (attribute) {
@@ -289,9 +289,9 @@ function mergeXml (src, dest, platform, clobber) {
         var query = srcTag + '';
         var shouldMerge = true;
 
-        if (BLACKLIST.indexOf(srcTag) !== -1) return;
+        if (BLACKLIST.includes(srcTag)) return;
 
-        if (SINGLETONS.indexOf(srcTag) !== -1) {
+        if (SINGLETONS.includes(srcTag)) {
             foundChild = dest.find(query);
             if (foundChild) {
                 destChild = foundChild;