Change to use multiple-autofixes. (#650)

https://eslint.org/blog/2017/06/eslint-v4.1.0-released#applying-multiple-autofixes-simultaneously

Author: ota-meshi

lib/rules/attributes-order.js

@@ -77,25 +77,10 @@ function create (context) {
         const attributes = node.parent.attributes
         const shiftAttrs = attributes.slice(attributes.indexOf(previousNode), attributes.indexOf(node) + 1)
 
-        // If we can upgrade requirements to `eslint@>4.1.0`, this code can be replaced by:
-        // return shiftAttrs.map((attr, i) => {
-        //   const text = attr === previousNode ? sourceCode.getText(node) : sourceCode.getText(shiftAttrs[i - 1])
-        //   return fixer.replaceText(attr, text)
-        // })
-        const replaceDataList = shiftAttrs.map((attr, i) => {
+        return shiftAttrs.map((attr, i) => {
           const text = attr === previousNode ? sourceCode.getText(node) : sourceCode.getText(shiftAttrs[i - 1])
-          return {
-            range: attr.range,
-            text
-          }
+          return fixer.replaceText(attr, text)
         })
-        const replaceRange = [previousNode.range[0], node.range[1]]
-        let text = sourceCode.text.slice(replaceRange[0], replaceRange[1])
-        replaceDataList.reverse().forEach((data) => {
-          const textRange = data.range.map(r => r - replaceRange[0])
-          text = text.slice(0, textRange[0]) + data.text + text.slice(textRange[1], text.length)
-        })
-        return fixer.replaceTextRange(replaceRange, text)
       }
     })
   }

lib/rules/component-name-in-template-casing.js

@@ -51,7 +51,6 @@ module.exports = {
     const caseType = allowedCaseOptions.indexOf(caseOption) !== -1 ? caseOption : defaultCase
     const ignores = options.ignores || []
     const tokens = context.parserServices.getTemplateBodyTokenStore && context.parserServices.getTemplateBodyTokenStore()
-    const sourceCode = context.getSourceCode()
 
     let hasInvalidEOF = false
 
@@ -92,13 +91,10 @@ module.exports = {
                 return fixer.replaceText(open, `<${casingName}`)
               }
               const endTagOpen = tokens.getFirstToken(endTag)
-              // If we can upgrade requirements to `eslint@>4.1.0`, this code can be replaced by:
-              // return [
-              //   fixer.replaceText(open, `<${casingName}`),
-              //   fixer.replaceText(endTagOpen, `</${casingName}`)
-              // ]
-              const code = `<${casingName}${sourceCode.text.slice(open.range[1], endTagOpen.range[0])}</${casingName}`
-              return fixer.replaceTextRange([open.range[0], endTagOpen.range[1]], code)
+              return [
+                fixer.replaceText(open, `<${casingName}`),
+                fixer.replaceText(endTagOpen, `</${casingName}`)
+              ]
             }
           })
         }

lib/rules/no-shared-component-data.js

@@ -66,16 +66,10 @@ module.exports = {
             fix (fixer) {
               const tokens = getFirstAndLastTokens(p.value, sourceCode)
 
-              // If we can upgrade requirements to `eslint@>4.1.0`, this code can be replaced by:
-              //     return [
-              //       fixer.insertTextBefore(tokens.first, 'function() {\nreturn '),
-              //       fixer.insertTextAfter(tokens.last, ';\n}')
-              //     ]
-              // See: https://eslint.org/blog/2017/06/eslint-v4.1.0-released#applying-multiple-autofixes-simultaneously
-              const range = [tokens.first.range[0], tokens.last.range[1]]
-              const valueText = sourceCode.text.slice(range[0], range[1])
-              const replacement = `function() {\nreturn ${valueText};\n}`
-              return fixer.replaceTextRange(range, replacement)
+              return [
+                fixer.insertTextBefore(tokens.first, 'function() {\nreturn '),
+                fixer.insertTextAfter(tokens.last, ';\n}')
+              ]
             }
           })
         })