add option to sort individual blocks of imports separately

Signed-off-by: Anupam Kumar <[email protected]>

Author: kyteinsky

README.md

@@ -59,6 +59,7 @@ You can override these default settings according to your needs.
     "namespaceResolver.sortOnSave": false,              // Auto sort when a file is saved
     "namespaceResolver.sortAlphabetically": false,      // Sort imports in alphabetical order instead of line length
     "namespaceResolver.sortNatural": false,             // Sort imports using a 'natural order' algorithm
+    "namespaceResolver.sortBlockWise": false,           // Sort imports in individual blocks separated by blank lines
     "namespaceResolver.leadingSeparator": true,         // Expand class with leading namespace separator
     "namespaceResolver.highlightOnSave": false,         // Auto highlight not imported and not used when a file is saved
     "namespaceResolver.highlightOnOpen": false          // Auto highlight not imported and not used when a file is opened

package.json

@@ -116,6 +116,11 @@
                     "default": false,
                     "description": "Sort imports using a 'natural order' algorithm"
                 },
+                "namespaceResolver.sortBlockWise": {
+                    "type": "boolean",
+                    "default": false,
+                    "description": "Sort imports in individual blocks separated by blank lines"
+                },
                 "namespaceResolver.leadingSeparator": {
                     "type": "boolean",
                     "default": true,

src/Resolver.js

@@ -448,13 +448,7 @@ class Resolver {
         return parsedNamespaces;
     }
 
-    sortImports() {
-        let [useStatements,] = this.getDeclarations();
-
-        if (useStatements.length <= 1) {
-            throw new Error('$(issue-opened)  Nothing to sort.');
-        }
-
+    getSortFunction() {
         let sortFunction = (a, b) => {
             if (this.config('sortAlphabetically')) {
                 if (a.text.toLowerCase() < b.text.toLowerCase()) return -1;
@@ -481,7 +475,21 @@ class Resolver {
             };
         }
 
-        let sorted = useStatements.slice().sort(sortFunction);
+        return sortFunction;
+    }
+
+    sortImports() {
+        const sortFunction = this.getSortFunction();
+
+        const [useStatements, sorted] = (() => {
+            if (this.config('sortBlockWise')) {
+                const useStatements = this.getUseStatementsArray(true);
+                return [useStatements.flat(), useStatements.map(block => block.slice().sort(sortFunction)).flat()];
+            }
+
+            const useStatements = this.getDeclarations();
+            return [useStatements, useStatements.slice().sort(sortFunction)];
+        })();
 
         this.activeEditor().edit(textEdit => {
             for (let i = 0; i < sorted.length; i++) {
@@ -507,21 +515,28 @@ class Resolver {
         return false;
     }
 
-    getUseStatementsArray() {
-        let useStatements = [];
+    getUseStatementsArray(blocked = false) {
+        const useStatements = [];
+        const matchRegex = new RegExp(/^(use\s[\w\\]+;)/, 'g');
 
         for (let line = 0; line < this.activeEditor().document.lineCount; line++) {
-            let text = this.activeEditor().document.lineAt(line).text;
-
-            if (text.startsWith('use ')) {
-                useStatements.push(
-                    text.match(/(\w+?);/)[1]
-                );
+            const text = this.activeEditor().document.lineAt(line).text;
+            const matchRes = text.match(matchRegex);
+
+            if (matchRes != null) {
+                const prevLine = this.activeEditor().document.lineAt(Math.max(0, line - 1)).text;
+                if (prevLine === '' || !prevLine.match(matchRegex) || useStatements.length === 0)
+                    useStatements.push([{ text: matchRes[0], line }]);
+                else
+                    useStatements[useStatements.length - 1].push({ text: matchRes[0], line });
             } else if (/(class|trait|interface)\s+\w+/.test(text)) {
                 break;
             }
         }
 
+        if (!blocked)
+            return useStatements.flat();
+
         return useStatements;
     }