Added check for register diff command

Author: edgardmessias

package.json

@@ -228,7 +228,7 @@
         },
         {
           "command": "svn.revertSelectedRanges",
-          "when": "config.svn.enabled && svnOpenRepositoryCount != 0"
+          "when": "config.svn.enabled && svnOpenRepositoryCount != 0 && svnHasSupportToRegisterDiffCommand == 1"
         }
       ],
       "scm/title": [
@@ -373,7 +373,7 @@
           "command": "svn.revertSelectedRanges",
           "group": "2_svn@3",
           "when":
-            "config.svn.enabled && svnOpenRepositoryCount != 0 && isInDiffEditor && resourceScheme != merge-conflict.conflict-diff"
+            "config.svn.enabled && svnOpenRepositoryCount != 0 && isInDiffEditor && resourceScheme != merge-conflict.conflict-diff &&  && svnHasSupportToRegisterDiffCommand == 1"
         }
       ]
     },

src/commands.ts

@@ -28,7 +28,7 @@ import * as path from "path";
 import { start } from "repl";
 import { getConflictPickOptions } from "./conflictItems";
 import { applyLineChanges } from "./lineChanges";
-import { IDisposable } from "./util";
+import { IDisposable, hasSupportToRegisterDiffCommand } from "./util";
 import {
   getChangelistPickOptions,
   inputSwitchChangelist,
@@ -124,7 +124,7 @@ export class SvnCommands implements IDisposable {
   constructor(private model: Model) {
     this.disposables = Commands.map(({ commandId, method, options }) => {
       const command = this.createCommand(method, options);
-      if (options.diff) {
+      if (options.diff && hasSupportToRegisterDiffCommand()) {
         return commands.registerDiffInformationCommand(commandId, command);
       } else {
         return commands.registerCommand(commandId, command);

src/extension.ts

@@ -10,7 +10,11 @@ import { SvnFinder } from "./svnFinder";
 import { SvnContentProvider } from "./svnContentProvider";
 import { SvnCommands } from "./commands";
 import { Model } from "./model";
-import { toDisposable, hasSupportToDecorationProvider } from "./util";
+import {
+  toDisposable,
+  hasSupportToDecorationProvider,
+  hasSupportToRegisterDiffCommand
+} from "./util";
 
 async function init(context: ExtensionContext, disposables: Disposable[]) {
   const outputChannel = window.createOutputChannel("Svn");
@@ -52,6 +56,12 @@ async function init(context: ExtensionContext, disposables: Disposable[]) {
   model.onDidCloseRepository(onRepository, null, disposables);
   onRepository();
 
+  commands.executeCommand(
+    "setContext",
+    "svnHasSupportToRegisterDiffCommand",
+    hasSupportToRegisterDiffCommand() ? "1" : "0"
+  );
+
   outputChannel.appendLine("Using svn " + info.version + " from " + info.path);
 
   const onOutput = (str: string) => outputChannel.append(str);

src/util.ts

@@ -1,4 +1,4 @@
-import { Event, window } from "vscode";
+import { Event, window, commands } from "vscode";
 import { sep } from "path";
 
 export interface IDisposable {
@@ -36,9 +36,7 @@ export function dispose(disposables: any[]): any[] {
   return [];
 }
 
-export function combinedDisposable(
-  disposables: IDisposable[]
-): IDisposable {
+export function combinedDisposable(disposables: IDisposable[]): IDisposable {
   return toDisposable(() => dispose(disposables));
 }
 
@@ -103,6 +101,19 @@ export function hasSupportToDecorationProvider() {
   return typeof window.registerDecorationProvider === "function";
 }
 
+export function hasSupportToRegisterDiffCommand() {
+  try {
+    const disposable = commands.registerDiffInformationCommand(
+      "svn.testDiff",
+      () => {}
+    );
+    disposable.dispose();
+    return true;
+  } catch (error) {
+    return false;
+  }
+}
+
 export function timeout(ms: number) {
   return new Promise(resolve => setTimeout(resolve, ms));
 }