Merge remote-tracking branch 'remotes/upstream/master' into rework_changes

# Conflicts:
#	src/commands.ts

Author: edgardmessias

.prettierignore

@@ -0,0 +1 @@
+README.md

CHANGELOG.md

@@ -1,3 +1,21 @@
+# **v1.11.1**
+
+## Changes
+
+* @JohnstonCode Updated readme file
+
+# **v1.11.0**
+
+## What's New
+
+* @JohnstonCode Added commit message list
+
+# **v1.10.0**
+
+## What's New
+
+* @JohnstonCode Added conflict support
+
 # **v1.9.0**
 
 ## What's New

README.md

@@ -33,16 +33,8 @@ If you use [TortoiseSVN](https://tortoisesvn.net/), make sure the option
 **Command Line Tools** is checked during installation and
 `C:\Program Files\TortoiseSVN\bin` is available in PATH.
 
-## Features
-
-* [x] Source Control View
-* [x] Quick Diffs in gutter
-* [x] Status Bar
-* [ ] SVN Commands
-
 ## Feedback & Contributing
 
-* This is a work in progress and im sure there will be some issues.
 * Please report any bugs, suggestions or documentation requests via the
   [Issues](https://github.com/JohnstonCode/svn-scm/issues)
 * Feel free to submit
@@ -54,3 +46,69 @@ If you use [TortoiseSVN](https://tortoisesvn.net/), make sure the option
 * @edgardmessias
 * @csholmq
 * @rwatts3
+
+## Features
+
+* Source Control View
+* Quick Diffs in gutter
+* Status Bar
+* Create changelists
+* Add files
+* Revert edits
+* Remove files
+* Create branches
+* Switch branches
+* Create patches
+* Diff changes
+* Commit changes/changelists
+* See commit messages
+
+## Settings
+
+`svn.enabled`
+  * Enables Svn as a SCM in VS Code.  
+  `"default"` — true
+
+`svn.path`
+  * Path to the svn executable  
+  `"default"` — null
+
+`svn.diff.withHead`
+  * Show diff changes using latest revision in the repository. Set false to use latest revision in local folder  
+  `"default"` — true
+
+`svn.layout.trunk`
+  * Relative path for 'trunk' in SVN URL, 'null' to disable. (Ex.: 'trunk', 'main')  
+  `"default"` — trunk
+
+`svn.layout.branches`
+  * Relative path for 'branches' in SVN URL, 'null' to disable. (Ex.: 'branches', 'versions')  
+  `"default"` — branches
+
+`svn.layout.tags`
+  * Relative path for 'tags' in SVN URL, 'null' to disable. (Ex.: 'tags', 'stamps')  
+  `"default"` — tags
+
+`svn.multipleFolders.enabled`
+  * Allow to find subfolders using SVN  
+  `"default"` — false
+
+`svn.multipleFolders.depth`
+  * Maximum depth to find subfolders using SVN  
+  `"default"` — 4
+
+`svn.multipleFolders.ignore`
+  * Folders to ignore using SVN  
+  `"default"` — `["**/.git", "**/.hg", "**/vendor", "**/node_modules"]`
+
+`svn.sourceControl.ignoreOnCommit`
+  * Changelists to ignore on commit  
+  `"default"` — `["ignore-on-commit"]`
+
+`svn.sourceControl.showExternal`
+  * Allow to show in source control the list the external folders  
+  `"default"` — false
+
+`svn.log.length`
+  * Number of commit messages to log  
+  `"default"` — 50

package.json

@@ -2,7 +2,7 @@
   "name": "svn-scm",
   "displayName": "SVN",
   "description": "Integrated Subversion source control",
-  "version": "1.9.0",
+  "version": "1.11.1",
   "publisher": "johnstoncode",
   "engines": {
     "vscode": "^1.17.0"
@@ -148,6 +148,16 @@
         "command": "svn.remove",
         "title": "Remove Selected",
         "category": "SVN"
+      },
+      {
+        "command": "svn.resolve",
+        "title": "Resolve Conflicts",
+        "category": "SVN"
+      },
+      {
+        "command": "svn.log",
+        "title": "Show commit messages",
+        "category": "SVN"
       }
     ],
     "menus": {
@@ -186,6 +196,14 @@
         {
           "command": "svn.patch",
           "when": "config.svn.enabled"
+        },
+        {
+          "command": "svn.resolve",
+          "when": "config.svn.enabled"
+        },
+        {
+          "command": "svn.log",
+          "when": "config.svn.enabled"
         }
       ],
       "scm/resourceGroup/context": [],
@@ -354,6 +372,12 @@
           "description":
             "Allow to show in source control the list the external folders",
           "default": false
+        },
+        "svn.log.length": {
+          "type": "number",
+          "minimum": 1,
+          "description": "Number of commit messages to log",
+          "default": 50
         }
       }
     }

src/commands.ts

@@ -19,6 +19,7 @@ import { toSvnUri, fromSvnUri } from "./uri";
 import * as fs from "fs";
 import * as path from "path";
 import { start } from "repl";
+import { getConflictPickOptions } from "./conflictItems";
 
 interface CommandOptions {
   repository?: boolean;
@@ -180,10 +181,7 @@ export class SvnCommands {
       picks.push(new ChangeListItem(repository.changes));
     }
 
-    const svnConfig = workspace.getConfiguration(
-      "svn",
-      Uri.file(repository.workspaceRoot)
-    );
+    const svnConfig = workspace.getConfiguration("svn");
     const ignoreOnCommitList = svnConfig.get<string[]>(
       "sourceControl.ignoreOnCommit",
       []
@@ -764,6 +762,44 @@ export class SvnCommands {
     }
   }
 
+  @command("svn.resolve", { repository: true })
+  async resolve(repository: Repository) {
+    const conflicts = repository.conflicts.resourceStates;
+
+    if (!conflicts.length) {
+      window.showInformationMessage("No Conflicts");
+    }
+
+    for (const conflict of conflicts) {
+      const placeHolder = `Select conflict option for ${
+        conflict.resourceUri.path
+      }`;
+      const picks = getConflictPickOptions();
+
+      const choice = await window.showQuickPick(picks, { placeHolder });
+
+      if (!choice) {
+        return;
+      }
+
+      await repository.resolve(conflict.resourceUri.path, choice.label);
+    }
+  }
+
+  @command("svn.log", { repository: true })
+  async log(repository: Repository) {
+    try {
+      const result = await repository.repository.log();
+      // send the log results to a new tab
+      workspace.openTextDocument({ content: result }).then(doc => {
+        window.showTextDocument(doc);
+      });
+    } catch (error) {
+      console.error(error);
+      window.showErrorMessage("Unable to log");
+    }
+  }
+
   private getSCMResource(uri?: Uri): Resource | undefined {
     uri = uri
       ? uri

src/conflictItems.ts

@@ -0,0 +1,55 @@
+import { QuickPickItem } from "vscode";
+
+interface ConflictOption {
+  label: string;
+  description: string;
+}
+
+const ConflictOptions = [
+  {
+    label: "base",
+    description:
+      "Choose the file that was the (unmodified) BASE revision before you tried to integrate changes"
+  },
+  {
+    label: "working",
+    description:
+      "Assuming that you've manually handled the conflict resolution, choose the version of the file as it currently stands in your working copy."
+  },
+  {
+    label: "mine-full",
+    description:
+      "Preserve all local modifications and discarding all changes fetched"
+  },
+  {
+    label: "theirs-full",
+    description:
+      "Discard all local modifications and integrating all changes fetched"
+  },
+  {
+    label: "mine-conflict",
+    description:
+      "Resolve conflicted files by preferring local modifications over the changes fetched"
+  },
+  {
+    label: "theirs-conflict",
+    description:
+      "Resolve conflicted files by preferring the changes fetched from the server over local modifications"
+  }
+];
+
+class ConflictItem implements QuickPickItem {
+  constructor(private option: ConflictOption) {}
+
+  get label(): string {
+    return this.option.label;
+  }
+
+  get description(): string {
+    return this.option.description;
+  }
+}
+
+export function getConflictPickOptions() {
+  return ConflictOptions.map(option => new ConflictItem(option));
+}

src/repository.ts

@@ -29,6 +29,7 @@ export class Repository {
   public unversioned: SourceControlResourceGroup;
   public external: SourceControlResourceGroup;
   public changelists: Map<string, SourceControlResourceGroup> = new Map();
+  public conflicts: SourceControlResourceGroup;
   private disposables: Disposable[] = [];
   public currentBranch = "";
   public isSwitchingBranch: boolean = false;
@@ -133,10 +134,15 @@ export class Repository {
       "external",
       "External"
     );
+    this.conflicts = this.sourceControl.createResourceGroup(
+      "conflicts",
+      "conflicts"
+    );
 
     this.changes.hideWhenEmpty = true;
     this.unversioned.hideWhenEmpty = true;
     this.external.hideWhenEmpty = true;
+    this.conflicts.hideWhenEmpty = true;
 
     this.disposables.push(
       toDisposable(() => clearInterval(this.branchesTimer))
@@ -163,14 +169,15 @@ export class Repository {
     let changes: any[] = [];
     let unversioned: any[] = [];
     let external: any[] = [];
+    let conflicts: any[] = [];
     let changelists: Map<string, Resource[]> = new Map();
 
     const statuses = (await this.repository.getStatus()) || [];
 
-    const fileConfig = workspace.getConfiguration("files");
+    const fileConfig = workspace.getConfiguration("files", Uri.file(this.root));
     const svnConfig = workspace.getConfiguration("svn");
 
-    const filesToExclude = fileConfig.get<any>("exclude", null);
+    const filesToExclude = fileConfig.get<any>("exclude");
 
     let excludeList: string[] = [];
     for (const pattern in filesToExclude) {
@@ -188,15 +195,20 @@ export class Repository {
         ? Uri.file(path.join(this.workspaceRoot, status.rename))
         : undefined;
 
-      const resouce = new Resource(uri, status.status, renameUri, status.props);
+      const resource = new Resource(
+        uri,
+        status.status,
+        renameUri,
+        status.props
+      );
 
       if (status.status === Status.NORMAL && status.props === PropStatus.NONE) {
         // On commit, `svn status` return all locked files with status="normal" and props="none"
         return;
-      } else if (status.status === Status.UNVERSIONED) {
-        unversioned.push(resouce);
       } else if (status.status === Status.EXTERNAL) {
-        external.push(resouce);
+        external.push(resource);
+      } else if (status.status === Status.CONFLICTED) {
+        conflicts.push(resource);
       } else {
         if (status.status === Status.UNVERSIONED) {
           const matches = status.path.match(
@@ -210,24 +222,27 @@ export class Repository {
             statuses.some(s => s.path === matches[1])
           ) {
             return;
+          } else {
+            unversioned.push(resource);
           }
         }
 
         if (!status.changelist) {
-          changes.push(resouce);
+          changes.push(resource);
         } else {
           let changelist = changelists.get(status.changelist);
           if (!changelist) {
             changelist = [];
           }
-          changelist.push(resouce);
+          changelist.push(resource);
           changelists.set(status.changelist, changelist);
         }
       }
     });
 
     this.changes.resourceStates = changes;
     this.unversioned.resourceStates = unversioned;
+    this.conflicts.resourceStates = conflicts;
 
     this.changelists.forEach((group, changelist) => {
       group.resourceStates = [];
@@ -357,4 +372,13 @@ export class Repository {
       this._onDidChangeBranch.fire();
     }
   }
+
+  async resolve(file: string, action: string) {
+    try {
+      const response = await this.repository.resolve(file, action);
+      window.showInformationMessage(response);
+    } catch (error) {
+      window.showErrorMessage(error);
+    }
+  }
 }