Modernize to use let and const

Author: Overload119

lib/rules/jsx-sort-props.js

@@ -57,7 +57,7 @@ function alphabeticalCompare(a, b, ignoreCase) {
 function getGroupsOfSortableAttributes(attributes) {
   const sortableAttributeGroups = [];
   let groupCount = 0;
-  for (var i = 0; i < attributes.length; i++) {
+  for (let i = 0; i < attributes.length; i++) {
     const lastAttr = attributes[i - 1];
     // If we have no groups or if the last attribute was JSXSpreadAttribute
     // then we start a new group. Append attributes to the group until we
@@ -77,30 +77,30 @@ function getGroupsOfSortableAttributes(attributes) {
   return sortableAttributeGroups;
 }
 
-function generateFixerFunction(node, context) {
-  var sourceCode = context.getSourceCode();
-  var attributes = node.attributes.slice(0);
-  var configuration = context.options[0] || {};
-  var ignoreCase = configuration.ignoreCase || false;
+const generateFixerFunction = (node, context) => {
+  const sourceCode = context.getSourceCode();
+  const attributes = node.attributes.slice(0);
+  const configuration = context.options[0] || {};
+  const ignoreCase = configuration.ignoreCase || false;
 
   // Sort props according to the context. Only supports ignoreCase.
   // Since we cannot safely move JSXSpreadAttribute (due to potential variable overrides),
   // we only consider groups of sortable attributes.
   const sortableAttributeGroups = getGroupsOfSortableAttributes(attributes);
-  const sortedAttributeGroups = sortableAttributeGroups.slice(0).map(function(group) {
-    return group.slice(0).sort(function(a, b) {
-      return alphabeticalCompare(propName(a), propName(b), ignoreCase);
-    });
-  });
+  const sortedAttributeGroups = sortableAttributeGroups.slice(0).map(group =>
+    group.slice(0).sort((a, b) =>
+      alphabeticalCompare(propName(a), propName(b), ignoreCase)
+    )
+  );
 
   return function(fixer) {
     const fixers = [];
 
     // Replace each unsorted attribute with the sorted one.
-    sortableAttributeGroups.forEach(function(sortableGroup, ii) {
-      sortableGroup.forEach(function(attr, jj) {
-        var sortedAttr = sortedAttributeGroups[ii][jj];
-        var sortedAttrText = sourceCode.getText(sortedAttr);
+    sortableAttributeGroups.forEach((sortableGroup, ii) => {
+      sortableGroup.forEach((attr, jj) => {
+        const sortedAttr = sortedAttributeGroups[ii][jj];
+        const sortedAttrText = sourceCode.getText(sortedAttr);
         fixers.push(
           fixer.replaceTextRange([attr.start, attr.end], sortedAttrText)
         );
@@ -109,7 +109,7 @@ function generateFixerFunction(node, context) {
 
     return fixers;
   };
-}
+};
 
 /**
  * Checks if the `reservedFirst` option is valid