hazza/dspace-angular
1
0
Fork 1
mirror of https://github.com/DSpace/dspace-angular.git synced 2025-10-07 01:54:15 +00:00
Code Issues Packages Projects Releases Wiki Activity

Prevent system-wide alert to be activated with empty message

Browse Source 
(cherry picked from commit 767de4b5c8)
This commit is contained in:
Alexandre Vryghem
2024-03-23 15:12:48 +01:00
committed by github-actions[bot]
parent 4c693a1294
commit d05438dc3e
2 changed files with 15 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
src/app/system-wide-alert/alert-form/system-wide-alert-form.component.spec.ts
View File

@@ -302,6 +302,14 @@ describe('SystemWideAlertFormComponent', () => {
expect(comp.back).not.toHaveBeenCalled();
});
it('should not create the new alert when the enable button is clicked on an invalid the form', () => {
spyOn(comp as any, 'handleResponse');
comp.formMessage.patchValue('');
comp.save();
expect((comp as any).handleResponse).not.toHaveBeenCalled();
});
});
describe('back', () => {
it('should navigate back to the home page', () => {

2
src/app/system-wide-alert/alert-form/system-wide-alert-form.component.ts
View File

@@ -219,6 +219,7 @@ export class SystemWideAlertFormComponent implements OnInit {
} else {
alert.countdownTo = null;
}
if (this.alertForm.valid) {
if (hasValue(this.currentAlert)) {
const updatedAlert = Object.assign(new SystemWideAlert(), this.currentAlert, alert);
this.handleResponse(this.systemWideAlertDataService.put(updatedAlert), 'system-wide-alert.form.update', navigateToHomePage);
@@ -226,6 +227,7 @@ export class SystemWideAlertFormComponent implements OnInit {
this.handleResponse(this.systemWideAlertDataService.create(alert), 'system-wide-alert.form.create', navigateToHomePage);
}
}
}
private handleResponse(response$: Observable<RemoteData<SystemWideAlert>>, messagePrefix, navigateToHomePage: boolean) {
response$.pipe(