move settings sync options into single interface

This commit is contained in:
Sandeep Somavarapu
2020-09-22 08:57:46 +02:00
parent e649ef3701
commit 4ad1655304
2 changed files with 15 additions and 13 deletions

View File

@@ -14,7 +14,7 @@ export class WebUserDataAutoSyncService extends UserDataAutoSyncService implemen
isEnabled(): boolean {
if (this.enabled === undefined) {
this.enabled = this.workbenchEnvironmentService.options?.enableSettingsSync;
this.enabled = this.workbenchEnvironmentService.options?.settingsSyncOptions?.enabled;
}
if (this.enabled === undefined) {
this.enabled = super.isEnabled(this.workbenchEnvironmentService.options?.enableSyncByDefault);
@@ -25,8 +25,8 @@ export class WebUserDataAutoSyncService extends UserDataAutoSyncService implemen
protected setEnablement(enabled: boolean) {
if (this.enabled !== enabled) {
this.enabled = enabled;
if (this.workbenchEnvironmentService.options?.settingsSyncEnablementHandler) {
this.workbenchEnvironmentService.options.settingsSyncEnablementHandler(this.enabled);
if (this.workbenchEnvironmentService.options?.settingsSyncOptions?.enablementHandler) {
this.workbenchEnvironmentService.options.settingsSyncOptions.enablementHandler(this.enabled);
}
}
}