EVOgeek
/
Vmeda.Online
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

MOBILE-3623 plugins: Wait to display success message if goBackOnSuccess

main
Dani Palou 2021-04-23 12:49:17 +02:00
parent 6a846488ee
commit e44024d980
2 changed files with 11 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/core/features/siteplugins/classes/call-ws-directive.ts
View File

@ -74,7 +74,8 @@ export class CoreSitePluginsCallWSBaseDirective implements OnInit, OnDestroy {
this.onSuccess.emit(result);
return this.wsCallSuccess(result);
// Don't block the promise with the success function.
this.wsCallSuccess(result);
} catch (error) {
this.onError.emit(error);
this.logger.error(`Error calling WS ${this.name}`, error);
@ -108,9 +109,10 @@ export class CoreSitePluginsCallWSBaseDirective implements OnInit, OnDestroy {
* Function called when the WS call is successful.
*
* @param result Result of the WS call.
* @return If async, promise resolved when done.
*/
// eslint-disable-next-line @typescript-eslint/no-unused-vars
protected wsCallSuccess(result: unknown): void {
protected wsCallSuccess(result: unknown): void | Promise<void> {
// Function to be overridden.
}

14
src/core/features/siteplugins/directives/call-ws.ts
View File

@ -65,17 +65,17 @@ export class CoreSitePluginsCallWSDirective extends CoreSitePluginsCallWSOnClick
/**
* @inheritdoc
*/
protected wsCallSuccess(): void {
protected async wsCallSuccess(): Promise<void> {
if (CoreUtils.isTrueOrOne(this.goBackOnSuccess)) {
await CoreNavigator.back();
} else if (CoreUtils.isTrueOrOne(this.refreshOnSuccess) && this.parentContent) {
this.parentContent.refreshContent(true);
}
if (typeof this.successMessage != 'undefined') {
// Display the success message.
CoreDomUtils.showToast(this.successMessage || Translate.instant('core.success'));
}
if (CoreUtils.isTrueOrOne(this.goBackOnSuccess)) {
CoreNavigator.back();
} else if (CoreUtils.isTrueOrOne(this.refreshOnSuccess) && this.parentContent) {
this.parentContent.refreshContent(true);
}
}
}