style: after lint

Author: Scrum

lib/backup.js

@@ -13,11 +13,11 @@ const cloneDeep = require('fclone')
  * @return {Object} backup Backup Locals
  */
 function makeLocalsBackup (keys, locals) {
-  let backup = {}
+  const backup = {}
 
   for (let i = 0; i < keys.length; i++) {
     const key = keys[i]
-    if (locals.hasOwnProperty(key)) backup[key] = cloneDeep(locals[key])
+    if (Object.prototype.hasOwnProperty.call(locals, key)) backup[key] = cloneDeep(locals[key])
   }
 
   return backup
@@ -41,7 +41,7 @@ function revertBackupedLocals (keys, locals, backup) {
     delete locals[key]
 
     // revert copied key value
-    if (backup.hasOwnProperty(key)) locals[key] = backup[key]
+    if (Object.prototype.hasOwnProperty.call(backup, key)) locals[key] = backup[key]
   }
 
   return locals

lib/index.js

@@ -9,7 +9,7 @@ const makeLocalsBackup = require('./backup').make
 const revertBackupedLocals = require('./backup').revert
 const placeholders = require('./placeholders')
 
-let delimitersSettings = []
+const delimitersSettings = []
 let conditionals, switches, loops, scopes, ignored
 
 /**
@@ -177,8 +177,8 @@ function walk (opts, nodes) {
     // if we have a string, match and replace it
     if (typeof node === 'string') {
       // if node contains ignored expression delimiter, output as-is
-      let before = delimitersSettings[1].text[0]
-      let after = delimitersSettings[1].text[1]
+      const before = delimitersSettings[1].text[0]
+      const after = delimitersSettings[1].text[1]
       const ignoredDelimiter = new RegExp(`@${before}(.+?)${after}`, 'g')
 
       if (ignoredDelimiter.test(node)) {
@@ -198,7 +198,7 @@ function walk (opts, nodes) {
 
     // if not, we have an object, so we need to run the attributes and contents
     if (node.attrs) {
-      for (let key in node.attrs) {
+      for (const key in node.attrs) {
         node.attrs[key] = placeholders(node.attrs[key], ctx, delimitersSettings)
       }
     }
@@ -351,7 +351,7 @@ function walk (opts, nodes) {
           m.push(executeLoop(keys, target[index], index, opts.locals, treeString))
         }
       } else {
-        for (let key in target) {
+        for (const key in target) {
           opts.locals.loop = getLoopMeta(key, target)
           m.push(executeLoop(keys, target[key], key, opts.locals, treeString))
         }

lib/placeholders.js

@@ -55,7 +55,7 @@ function placeholders (input, ctx, settings) {
 
       if (/\W+/.test(expression)) {
         value = vm.runInContext(expression, ctx)
-      } else if (ctx.hasOwnProperty(expression)) {
+      } else if (Object.prototype.hasOwnProperty.call(ctx, expression)) {
         value = ctx[expression]
       }
 

lib/tags.js

@@ -22,7 +22,7 @@ function getNextTag (nodes, i) {
     }
   }
 
-  return [ i, { tag: undefined } ]
+  return [i, { tag: undefined }]
 }
 
 /**

test/test-conditionals.js

@@ -15,7 +15,7 @@ const expect = (file) => {
 }
 
 function process (t, name, options, log = false) {
-  return posthtml([ expressions(options) ])
+  return posthtml([expressions(options)])
     .process(fixture(name))
     .then((result) => {
       log && console.log(result.html)
@@ -28,7 +28,7 @@ function process (t, name, options, log = false) {
 }
 
 function error (name, cb) {
-  return posthtml([ expressions() ])
+  return posthtml([expressions()])
     .process(fixture(name))
     .catch(cb)
 }

test/test-core.js

@@ -15,7 +15,7 @@ const expect = (file) => {
 }
 
 function process (t, name, options, log = false) {
-  return posthtml([ expressions(options) ])
+  return posthtml([expressions(options)])
     .process(fixture(name))
     .then((result) => {
       log && console.log(result.html)
@@ -28,7 +28,7 @@ function process (t, name, options, log = false) {
 }
 
 function error (name, cb) {
-  return posthtml([ expressions() ])
+  return posthtml([expressions()])
     .process(fixture(name))
     .catch(cb)
 }

test/test-loops.js

@@ -15,7 +15,7 @@ const expect = (file) => {
 }
 
 function process (t, name, options, log = false) {
-  return posthtml([ expressions(options) ])
+  return posthtml([expressions(options)])
     .process(fixture(name))
     .then((result) => {
       log && console.log(result.html)
@@ -28,7 +28,7 @@ function process (t, name, options, log = false) {
 }
 
 function error (name, cb) {
-  return posthtml([ expressions() ])
+  return posthtml([expressions()])
     .process(fixture(name))
     .catch(cb)
 }

test/test-scopes.js

@@ -15,7 +15,7 @@ const expect = (file) => {
 }
 
 function process (t, name, options, log = false) {
-  return posthtml([ expressions(options) ])
+  return posthtml([expressions(options)])
     .process(fixture(name))
     .then((result) => {
       log && console.log(result.html)

test/test-switch.js

@@ -15,7 +15,7 @@ const expect = (file) => {
 }
 
 function process (t, name, options, log = false) {
-  return posthtml([ expressions(options) ])
+  return posthtml([expressions(options)])
     .process(fixture(name))
     .then((result) => {
       log && console.log(result.html)
@@ -28,7 +28,7 @@ function process (t, name, options, log = false) {
 }
 
 function error (name, cb) {
-  return posthtml([ expressions() ])
+  return posthtml([expressions()])
     .process(fixture(name))
     .catch(cb)
 }