Skip to content

Commit a0d52a7

Browse files
committed
fix(dialog-service): updated dialog services
1 parent ec3be7d commit a0d52a7

3 files changed

Lines changed: 20 additions & 2 deletions

File tree

src/app/features/project/project-addons/services/addon-dialog.service.ts

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -35,6 +35,11 @@ export class AddonDialogService {
3535
},
3636
});
3737

38+
if (!dialogRef) {
39+
const errorMessage = this.translateService.instant('common.errorMessages.dialogOpenError');
40+
throw new Error(errorMessage);
41+
}
42+
3843
return dialogRef.onClose;
3944
}
4045

@@ -57,6 +62,11 @@ export class AddonDialogService {
5762
},
5863
});
5964

65+
if (!dialogRef) {
66+
const errorMessage = this.translateService.instant('common.errorMessages.dialogOpenError');
67+
throw new Error(errorMessage);
68+
}
69+
6070
return dialogRef.onClose;
6171
}
6272
}

src/app/shared/services/custom-dialog.service.ts

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,6 +27,13 @@ export class CustomDialogService {
2727
header: config?.header ? this.translateService.instant(config.header) : undefined,
2828
};
2929

30-
return this.dialogService.open(component, finalConfig);
30+
const dialogRef = this.dialogService.open(component, finalConfig);
31+
32+
if (!dialogRef) {
33+
const errorMessage = this.translateService.instant('common.errorMessages.dialogOpenError');
34+
throw new Error(errorMessage);
35+
}
36+
37+
return dialogRef;
3138
}
3239
}

src/assets/i18n/en.json

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -173,7 +173,8 @@
173173
"viewOnlyLinksBanner": "You are viewing OSF through a view-only link, which may limit the data you have permission to see."
174174
},
175175
"errorMessages": {
176-
"serverError": "An unexpected error occurred. Please try again later."
176+
"serverError": "An unexpected error occurred. Please try again later.",
177+
"dialogOpenError": "Failed to open dialog."
177178
}
178179
},
179180
"navigation": {

0 commit comments

Comments
 (0)