Fix #127: Don't ignore removeDuplicates option when using officialSor…

…ting

The check for removing duplicates is completely separate from the
sorting, even when {officialSorting: false}. Therefore, it shouldn't
harm to also execute this check when {officialSorting: true}.

docs/rules/classnames-order.md

@@ -118,7 +118,7 @@ const customGroups = require('custom-groups').groups;
 
 ### `officialSorting` (default: `false`)
 
-Set `officialSorting` to `true` if you want to use the same ordering rules as the official plugin `prettier-plugin-tailwindcss`. Enabling this settings will cause `groupByResponsive`, `groups`, `prependCustom` and `removeDuplicates` options to be ignored.
+Set `officialSorting` to `true` if you want to use the same ordering rules as the official plugin `prettier-plugin-tailwindcss`. Enabling this setting will cause `groupByResponsive`, `groups`, and `prependCustom` options to be ignored.
 
 ### `prependCustom` (default: `false`)
 

lib/rules/classnames-order.js

@@ -309,6 +309,10 @@ module.exports = {
         return;
       }
 
+      if (removeDuplicates) {
+        removeDuplicatesFromClassnamesAndWhitespaces(classNames, whitespaces, headSpace, tailSpace);
+      }
+
       let orderedClassNames;
       let validatedClassNamesValue = '';
 
@@ -323,10 +327,6 @@ module.exports = {
           }
         }
       } else {
-        if (removeDuplicates) {
-          removeDuplicatesFromClassnamesAndWhitespaces(classNames, whitespaces, headSpace, tailSpace);
-        }
-
         // Sorting
         const mergedSorted = [];
         const mergedExtras = [];

tests/lib/rules/classnames-order.js

@@ -817,6 +817,16 @@ ruleTester.run("classnames-order", rule, {
         },
       ],
     },
+    {
+      code: `<div class="h-4 h-4 w-4">Using official sorting, with duplicates</div>`,
+      output: `<div class="h-4 w-4">Using official sorting, with duplicates</div>`,
+      errors: errors,
+      options: [
+        {
+          officialSorting: true,
+        },
+      ],
+    },
     {
       code: `ctl(\`\${some} container animate-spin first:flex \${bool ? "flex-col flex" : ""}\`)`,
       output: `ctl(\`\${some} container animate-spin first:flex \${bool ? "flex flex-col" : ""}\`)`,