Merge pull request #217 from PowerShell/daviwil/extension-fix

daviwil · daviwil · commit 78e24547b2f2 · 2016-05-10T16:19:35.000-07:00
Fix return value of EditorObject.RegisterCommand
diff --git a/src/PowerShellEditorServices/Extensions/EditorObject.cs b/src/PowerShellEditorServices/Extensions/EditorObject.cs
@@ -58,9 +58,10 @@ public EditorObject(
         /// Registers a new command in the editor.
         /// </summary>
         /// <param name="editorCommand">The EditorCommand to be registered.</param>
-        public void RegisterCommand(EditorCommand editorCommand)
+        /// <returns>True if the command is newly registered, false if the command already exists.</returns>
+        public bool RegisterCommand(EditorCommand editorCommand)
         {
-            this.extensionService.RegisterCommand(editorCommand);
+            return this.extensionService.RegisterCommand(editorCommand);
         }
 
         /// <summary>

Original file line number	Diff line number	Diff line change
`@@ -58,9 +58,10 @@ public EditorObject(`
`58`	`58`	`/// Registers a new command in the editor.`
`59`	`59`	`/// </summary>`
`60`	`60`	`/// <param name="editorCommand">The EditorCommand to be registered.</param>`
`61`		`- public void RegisterCommand(EditorCommand editorCommand)`
	`61`	`+ /// <returns>True if the command is newly registered, false if the command already exists.</returns>`
	`62`	`+ public bool RegisterCommand(EditorCommand editorCommand)`
`62`	`63`	`{`
`63`		`- this.extensionService.RegisterCommand(editorCommand);`
	`64`	`+ return this.extensionService.RegisterCommand(editorCommand);`
`64`	`65`	`}`
`65`	`66`
`66`	`67`	`/// <summary>`