Added warning message if empty (Close #235,#240)

Author: edgardmessias

src/commands.ts

@@ -154,13 +154,13 @@ export class SvnCommands implements IDisposable {
 
   @command("svn.commitWithMessage", { repository: true })
   async commitWithMessage(repository: Repository) {
-    const message = repository.inputBox.value;
-    if (!message) {
+    const choice = await inputCommitChangelist(repository);
+    if (!choice) {
       return;
     }
 
-    const choice = await inputCommitChangelist(repository);
-    if (!choice) {
+    const message = await inputCommitMessage(repository.inputBox.value, false);
+    if (message === undefined) {
       return;
     }
 
@@ -311,14 +311,17 @@ export class SvnCommands implements IDisposable {
       const paths = resources.map(resource => resource.fsPath);
 
       try {
-        const message = await inputCommitMessage();
+        const message = await inputCommitMessage(repository.inputBox.value);
 
         if (message === undefined) {
           return;
         }
 
+        repository.inputBox.value = message;
+
         const result = await repository.commitFiles(message, paths);
         window.showInformationMessage(result);
+        repository.inputBox.value = "";
       } catch (error) {
         console.error(error);
         window.showErrorMessage("Unable to commit");

src/messages.ts

@@ -4,20 +4,31 @@ export function noChangesToCommit() {
   return window.showInformationMessage("There are no changes to commit.");
 }
 
-export function inputCommitMessage(message?: string) {
-  return new Promise<string>((resolve, reject) => {
-    if (message) {
-      resolve(message);
-      return;
-    }
+export async function inputCommitMessage(
+  message?: string,
+  promptNew: boolean = true
+): Promise<string | undefined> {
+  if (promptNew) {
+    message = await window.showInputBox({
+      value: message,
+      placeHolder: "Commit message",
+      prompt: "Please enter a commit message",
+      ignoreFocusOut: true
+    });
+  }
+
+  if (!message) {
+    const allowEmpty = await window.showWarningMessage(
+      "Do you really want to commit an empty message?",
+      { modal: true },
+      "Yes"
+    );
 
-    window
-      .showInputBox({
-        value: "",
-        placeHolder: "Commit message",
-        prompt: "Please enter a commit message",
-        ignoreFocusOut: true
-      })
-      .then(input => resolve(input));
-  });
+    if (allowEmpty === "Yes") {
+      return "";
+    } else {
+      return undefined;
+    }
+  }
+  return message;
 }

src/svn.ts

@@ -184,7 +184,7 @@ export class Svn {
     }
 
     if (options.log !== false) {
-      const argsOut = args.map(arg => (/ /.test(arg) ? `'${arg}'` : arg));
+      const argsOut = args.map(arg => (/ |^$/.test(arg) ? `'${arg}'` : arg));
       this.logOutput(
         `[${this.lastCwd.split(/[\\\/]+/).pop()}]$ svn ${argsOut.join(" ")}\n`
       );