Revert "Fix tests"

This reverts commit 0554e1a.
eclipse-theia · Apr 13, 2020 · feca093 · feca093
1 parent 4eed751
commit feca093
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 25 deletions.
diff --git a/packages/core/src/browser/quick-open/quick-command-service.ts b/packages/core/src/browser/quick-open/quick-command-service.ts
@@ -50,19 +50,6 @@ export class QuickCommandService implements QuickOpenModel, QuickOpenHandler {
     @inject(CorePreferences)
     protected readonly corePreferences: CorePreferences;
 
-    protected recent: Command[] = [];
-    addRecentCommand(command: Command): void {
-        // If this command is in recent list, remove and append to the start
-        const index = this.recent.findIndex(c => Command.equals(command, c));
-        if (index >= 0) {
-            this.recent.splice(index, 1);
-        }
-
-        const limit: number = this.corePreferences['workbench.commandPalette.history'];
-
-        this.recent = [command, ...this.recent].slice(0, limit);
-    }
-
     protected readonly contexts = new Map<string, string[]>();
     pushCommandContext(commandId: string, when: string): Disposable {
         const contexts = this.contexts.get(commandId) || [];
@@ -90,8 +77,7 @@ export class QuickCommandService implements QuickOpenModel, QuickOpenHandler {
                     {
                         groupLabel: index === 0 ? 'recently used' : '',
                         showBorder: false,
-                    },
-                    this
+                    }
                 )
             ),
             ...other.map((command, index) =>
@@ -102,8 +88,7 @@ export class QuickCommandService implements QuickOpenModel, QuickOpenHandler {
                     {
                         groupLabel: recent.length <= 0 ? '' : index === 0 ? 'other commands' : '',
                         showBorder: recent.length <= 0 ? false : index === 0 ? true : false,
-                    },
-                    this
+                    }
                 )
             ),
         );
@@ -129,7 +114,7 @@ export class QuickCommandService implements QuickOpenModel, QuickOpenHandler {
     private getCommands(): { recent: Command[], other: Command[] } {
 
         // Get the list of recent commands.
-        const recentCommands: Command[] = this.recent;
+        const recentCommands: Command[] = this.commands.recent;
 
         // Get the list of all valid commands.
         const allCommands: Command[] = this.getValidCommands(this.commands.commands);
@@ -225,7 +210,6 @@ export class CommandQuickOpenItem extends QuickOpenGroupItem {
         protected readonly commands: CommandRegistry,
         protected readonly keybindings: KeybindingRegistry,
         protected readonly commandOptions?: QuickOpenGroupItemOptions,
-        protected readonly quickCommand?: QuickCommandService
     ) {
         super(commandOptions);
         this.activeElement = window.document.activeElement as HTMLElement;
@@ -265,8 +249,8 @@ export class CommandQuickOpenItem extends QuickOpenGroupItem {
             this.activeElement.focus({ preventScroll: true });
             this.commands.executeCommand(this.command.id);
 
-            if (this.quickCommand) {
-                this.quickCommand.addRecentCommand(this.command);
+            if (this.command) {
+                this.commands.addRecentCommand(this.command);
             }
         }, 50);
         return true;

diff --git a/packages/core/src/common/command.ts b/packages/core/src/common/command.ts
@@ -289,10 +289,6 @@ export class CommandRegistry implements CommandService {
             await this.fireWillExecuteCommand(commandId, args);
             const result = await handler.execute(...args);
             this.onDidExecuteCommandEmitter.fire({ commandId, args });
-            const command = this.getCommand(commandId);
-            if (command) {
-                this.addRecentCommand(command);
-            }
             return result;
         }
         const argsMessage = args && args.length > 0 ? ` (args: ${JSON.stringify(args)})` : '';