🔨 change how ids work in scm providers. remove drag

This commit is contained in:
Joao Moreno
2017-03-24 16:44:38 +01:00
parent ce1222ba68
commit e4a765da2d
7 changed files with 18 additions and 42 deletions

View File

@@ -454,8 +454,8 @@ export function createApiFactory(initData: IInitData, threadService: IThreadServ
}
@proposed(extension)
registerSCMProvider(id, provider) {
return extHostSCM.registerSCMProvider(id, provider);
registerSCMProvider(provider) {
return extHostSCM.registerSCMProvider(provider);
}
}

View File

@@ -253,7 +253,6 @@ export interface SCMProviderFeatures {
label: string;
supportsOpen: boolean;
supportsAcceptChanges: boolean;
supportsDrag: boolean;
supportsOriginalResource: boolean;
}
@@ -412,7 +411,6 @@ export abstract class ExtHostTerminalServiceShape {
export abstract class ExtHostSCMShape {
$open(id: string, resourceGroupId: string, uri: string): TPromise<void> { throw ni(); }
$acceptChanges(id: string): TPromise<void> { throw ni(); }
$drag(id: string, fromResourceGroupId: string, fromUri: string, toResourceGroupId: string): TPromise<void> { throw ni(); }
$getOriginalResource(id: string, uri: URI): TPromise<URI> { throw ni(); }
$onInputBoxValueChange(value: string): TPromise<void> { throw ni(); }
}

View File

@@ -141,7 +141,9 @@ export class ExtHostSCM {
return resource;
}
registerSCMProvider(providerId: string, provider: vscode.SCMProvider): Disposable {
registerSCMProvider(provider: vscode.SCMProvider): Disposable {
const providerId = provider.id;
if (this._providers[providerId]) {
throw new Error(`Provider ${providerId} already registered`);
}
@@ -153,7 +155,6 @@ export class ExtHostSCM {
label: provider.label,
supportsOpen: !!provider.open,
supportsAcceptChanges: !!provider.acceptChanges,
supportsDrag: !!provider.drag,
supportsOriginalResource: !!provider.getOriginalResource
});
@@ -228,26 +229,6 @@ export class ExtHostSCM {
return asWinJsPromise(token => provider.acceptChanges(token));
}
$drag(providerId: string, fromResourceGroupId: string, fromUri: string, toResourceGroupId: string): TPromise<void> {
const provider = this._providers[providerId];
if (!provider) {
return TPromise.as(null);
}
const providerCache = this.cache[providerId];
const fromResourceGroup = providerCache[fromResourceGroupId];
const resource = fromResourceGroup && fromResourceGroup.resources[fromUri];
const toResourceGroup = providerCache[toResourceGroupId];
const resourceGroup = toResourceGroup && toResourceGroup.resourceGroup;
if (!resource || !resourceGroup) {
return TPromise.as(null);
}
return asWinJsPromise(token => provider.drag(resource, resourceGroup, token));
}
$getOriginalResource(id: string, uri: URI): TPromise<URI> {
const provider = this._providers[id];

View File

@@ -60,14 +60,6 @@ class MainThreadSCMProvider implements ISCMProvider {
return this.proxy.$acceptChanges(this.id);
}
drag(from: ISCMResource, to: ISCMResourceGroup): TPromise<void> {
if (!this.features.supportsDrag) {
return TPromise.as(null);
}
return this.proxy.$drag(this.id, from.resourceGroupId, from.uri.toString(), to.id);
}
getOriginalResource(uri: URI): TPromise<URI> {
if (!this.features.supportsOriginalResource) {
return TPromise.as(null);