Added support for multiple .svn folders (Close #30)

Author: edgardmessias

CHANGELOG.md

@@ -17,6 +17,12 @@
 * @JohnstonCode Commit info message now shows what revision it was
 * @JohnstonCode Added svn update to scm title commands
 * @JohnstonCode Added confirmation message to revert command
+* @edgardmessias Added support for multiple svn folders. To configure,
+  edit the options:
+  * "svn.multipleFolders.enabled" : Allow to find subfolders using SVN
+  * "svn.layout.depth" : Maximum depth to find subfolders using SVN
+  * "svn.multipleFolders.ignore" : Folders to ignore using SVN
+    (Ex.: '\*\*/vendor', '\*\*/node_modules')
 
 ## Bug Fixes
 

package.json

@@ -163,6 +163,23 @@
           "description":
             "Relative path for 'tags' in SVN URL, 'null' to disable. (Ex.: 'tags', 'stamps')",
           "default": "tags"
+        },
+        "svn.multipleFolders.enabled": {
+          "type": "boolean",
+          "description": "Allow to find subfolders using SVN",
+          "default": false
+        },
+        "svn.multipleFolders.depth": {
+          "type": "number",
+          "minimum": 0,
+          "description": "Maximum depth to find subfolders using SVN",
+          "default": 4
+        },
+        "svn.multipleFolders.ignore": {
+          "type": "array",
+          "minimum": 4,
+          "description": "Folders to ignore using SVN",
+          "default": ["**/vendor", "**/node_modules"]
         }
       }
     }

src/model.ts

@@ -20,6 +20,8 @@ export class Model {
   private disposables: Disposable[] = [];
   private enabled = false;
   private possibleSvnRepositoryPaths = new Set<string>();
+  private ignorePattern: RegExp = /^$/;
+  private maxDepth: number = 0;
 
   get repositories(): Repository[] {
     return this.openRepositories.map(r => r.repository);
@@ -31,6 +33,31 @@ export class Model {
 
     if (this.enabled) {
       this.init();
+
+      const multipleFolders = config.get<boolean>(
+        "multipleFolders.enabled",
+        false
+      );
+
+      if (multipleFolders) {
+        this.maxDepth = config.get<number>("multipleFolders.depth", 0);
+
+        const ignoreList = config.get("multipleFolders.ignore", []);
+
+        // Base on https://github.com/aleclarson/glob-regex/blob/master/index.js
+        const pattern = ignoreList
+          .join("|")
+          .replace(/\./g, "\\.")
+          .replace(/\*\*\//g, "(.+[\\\\\/])?")
+          .replace(/\*\*/g, "(.+[\\\\\/])?*")
+          .replace(/\*/g, "[^\\\\\/]+");
+
+        try {
+          this.ignorePattern = new RegExp("^(" + pattern + ")$");
+        } catch (error) {
+          window.showErrorMessage("Invalid pattern for: " + pattern);
+        }
+      }
     } else {
       this.disable();
     }
@@ -119,9 +146,8 @@ export class Model {
     }
   }
 
-  async tryOpenRepository(path: string, level = 1): Promise<void> {
-    // @todo add a config value and run a recursive function to scan folders for the defined depth
-    if (this.getRepository(path) || level > 4) {
+  async tryOpenRepository(path: string, level = 0): Promise<void> {
+    if (this.getRepository(path)) {
       return;
     }
 
@@ -137,12 +163,15 @@ export class Model {
       this.open(repository);
     } catch (err) {
       const newLevel = level + 1;
-      fs.readdirSync(path).forEach(file => {
-        const dir = path + "/" + file;
-        if (fs.statSync(dir).isDirectory()) {
-          this.tryOpenRepository(dir, newLevel);
-        }
-      });
+
+      if (newLevel <= this.maxDepth) {
+        fs.readdirSync(path).forEach(file => {
+          const dir = path + "/" + file;
+          if (fs.statSync(dir).isDirectory() && !this.ignorePattern.test(dir)) {
+            this.tryOpenRepository(dir, newLevel);
+          }
+        });
+      }
 
       return;
     }