MOBILE-3565 core: Fix some ESLint of CoreUpdateManagerProvider

main
Dani Palou 2020-10-14 08:32:09 +02:00
parent e514000623
commit 5cb0c6fe0c
1 changed files with 7 additions and 4 deletions

View File

@ -20,6 +20,8 @@ import CoreConfigConstants from '@app/config.json';
import { makeSingleton } from '@singletons/core.singletons'; import { makeSingleton } from '@singletons/core.singletons';
import { CoreLogger } from '@singletons/logger'; import { CoreLogger } from '@singletons/logger';
const VERSION_APPLIED = 'version_applied';
/** /**
* Factory to handle app updates. This factory shouldn't be used outside of core. * Factory to handle app updates. This factory shouldn't be used outside of core.
* *
@ -27,12 +29,12 @@ import { CoreLogger } from '@singletons/logger';
*/ */
@Injectable() @Injectable()
export class CoreUpdateManagerProvider implements CoreInitHandler { export class CoreUpdateManagerProvider implements CoreInitHandler {
// Data for init delegate. // Data for init delegate.
name = 'CoreUpdateManager'; name = 'CoreUpdateManager';
priority = CoreInitDelegate.MAX_RECOMMENDED_PRIORITY + 300; priority = CoreInitDelegate.MAX_RECOMMENDED_PRIORITY + 300;
blocking = true; blocking = true;
protected VERSION_APPLIED = 'version_applied';
protected logger: CoreLogger; protected logger: CoreLogger;
constructor() { constructor() {
@ -45,11 +47,11 @@ export class CoreUpdateManagerProvider implements CoreInitHandler {
* *
* @return Promise resolved when the update process finishes. * @return Promise resolved when the update process finishes.
*/ */
async load(): Promise<any> { async load(): Promise<void> {
const promises = []; const promises = [];
const versionCode = CoreConfigConstants.versioncode; const versionCode = CoreConfigConstants.versioncode;
const versionApplied: number = await CoreConfig.instance.get(this.VERSION_APPLIED, 0); const versionApplied: number = await CoreConfig.instance.get(VERSION_APPLIED, 0);
if (versionCode >= 3900 && versionApplied < 3900 && versionApplied > 0) { if (versionCode >= 3900 && versionApplied < 3900 && versionApplied > 0) {
// @todo: H5P update. // @todo: H5P update.
@ -58,11 +60,12 @@ export class CoreUpdateManagerProvider implements CoreInitHandler {
try { try {
await Promise.all(promises); await Promise.all(promises);
await CoreConfig.instance.set(this.VERSION_APPLIED, versionCode); await CoreConfig.instance.set(VERSION_APPLIED, versionCode);
} catch (error) { } catch (error) {
this.logger.error(`Error applying update from ${versionApplied} to ${versionCode}`, error); this.logger.error(`Error applying update from ${versionApplied} to ${versionCode}`, error);
} }
} }
} }
export class CoreUpdateManager extends makeSingleton(CoreUpdateManagerProvider) {} export class CoreUpdateManager extends makeSingleton(CoreUpdateManagerProvider) {}