fixes #100

Author: JohnstonCode

package.json

@@ -98,6 +98,11 @@
         "command": "svn.patch",
         "title": "Svn patch",
         "category": "SVN"
+      },
+      {
+        "command": "svn.remove",
+        "title": "Remove Selected",
+        "category": "SVN"
       }
     ],
     "menus": {
@@ -142,6 +147,11 @@
           "command": "svn.revert",
           "when": "scmProvider == svn && scmResourceGroup == changes",
           "group": "1_modification"
+        },
+        {
+          "command": "svn.remove",
+          "when": "scmProvider == svn && scmResourceGroup == changes",
+          "group": "2_modification"
         }
       ],
       "editor/title": []

src/commands.ts

@@ -365,6 +365,41 @@ export class SvnCommands {
     }
   }
 
+  @command("svn.remove", { repository: true })
+  async remove(
+    repository: Repository,
+    ...resourceStates: Resource[]
+  ): Promise<void> {
+    let keepLocal;
+    const answer = await window.showWarningMessage(
+      "Would you like to keep a local copy of the files?.",
+      "Yes",
+      "No"
+    );
+
+    if (!answer) {
+      return;
+    }
+
+    if (answer === "Yes") {
+      keepLocal = true;
+    } else {
+      keepLocal = false;
+    }
+
+    try {
+      const paths = resourceStates.map(state => {
+        return state.resourceUri.fsPath;
+      });
+
+      const result = await repository.repository.removeFiles(paths, keepLocal);
+      repository.update();
+    } catch (error) {
+      console.error(error);
+      window.showErrorMessage("Unable to remove files");
+    }
+  }
+
   private runByRepository<T>(
     resource: Uri,
     fn: (repository: Repository, resource: Uri) => Promise<T>

src/repository.ts

@@ -167,7 +167,7 @@ export class Repository {
     const fileConfig = workspace.getConfiguration("files");
     const svnConfig = workspace.getConfiguration("svn");
 
-    const filesToExclude = fileConfig.get<any>("exclude", {});
+    const filesToExclude = fileConfig.get<any>("exclude", null);
 
     let excludeList: string[] = [];
     for (const pattern in filesToExclude) {

src/svn.ts

@@ -271,4 +271,22 @@ export class Svn {
   patch(root: string) {
     return this.exec(root, ["diff"]);
   }
+
+  remove(files: any[], keepLocal: boolean) {
+    let args = ["remove"];
+
+    if (keepLocal) {
+      args.push("--keep-local");
+    }
+
+    for (let file of files) {
+      if (file instanceof Uri) {
+        args.push(file.fsPath);
+      } else {
+        args.push(file);
+      }
+    }
+
+    return this.exec("", args);
+  }
 }

src/svnRepository.ts

@@ -264,4 +264,14 @@ export class Repository {
     const message = result.stdout;
     return message;
   }
+
+  async removeFiles(files: any[], keepLocal: boolean) {
+    const result = await this.svn.remove(files, keepLocal);
+
+    if (result.exitCode !== 0) {
+      throw new Error(result.stderr);
+    }
+
+    return result.stdout;
+  }
 }