diff --git a/src/vs/workbench/services/workspace/node/workspaceEditingService.ts b/src/vs/workbench/services/workspace/node/workspaceEditingService.ts index 2f539b87b1d8dd27a592752493d6bf4688cc42a8..aa6a81981233bdedd291cab5605d71f03d3ae566 100644 --- a/src/vs/workbench/services/workspace/node/workspaceEditingService.ts +++ b/src/vs/workbench/services/workspace/node/workspaceEditingService.ts @@ -29,7 +29,7 @@ export class WorkspaceEditingService implements IWorkspaceEditingService { } public addRoots(rootsToAdd: URI[]): TPromise { - if (!this.supported) { + if (!this.isSupported()) { return TPromise.as(void 0); // we need a workspace to begin with } @@ -39,7 +39,7 @@ export class WorkspaceEditingService implements IWorkspaceEditingService { } public removeRoots(rootsToRemove: URI[]): TPromise { - if (!this.supported) { + if (!this.isSupported()) { return TPromise.as(void 0); // we need a workspace to begin with } @@ -49,13 +49,12 @@ export class WorkspaceEditingService implements IWorkspaceEditingService { return this.doSetRoots(roots.filter(root => rootsToRemoveRaw.indexOf(root.toString()) === -1)); } - private supported(): boolean { - if (this.contextService.hasMultiFolderWorkspace()) { - return false; // we need a multi folder workspace to begin with - } - + private isSupported(): boolean { // TODO@Ben multi root - return this.environmentService.appQuality !== 'stable'; // not yet enabled in stable + return ( + this.environmentService.appQuality !== 'stable' // not yet enabled in stable + && this.contextService.hasMultiFolderWorkspace() // we need a multi folder workspace to begin with + ); } private doSetRoots(newRoots: URI[]): TPromise { @@ -87,4 +86,4 @@ export class WorkspaceEditingService implements IWorkspaceEditingService { const validatedRoots = distinct(roots.map(root => root.toString(true /* skip encoding */))); return validatedRoots; } -} \ No newline at end of file +}