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

Merge pull request #2879 from DSpace/backport-2877-to-dspace-7_x

Browse Source 
[Port dspace-7_x] System-wide alert shouldn't be created when the message is empty
This commit is contained in:
Alan Orth
2024-03-26 11:26:31 +03:00
committed by GitHub
parent 4c693a1294 d05438dc3e
commit 43137fe676
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(