Merge pull request DSpace#5461 from guillermo-escire/feature/5454

Fix DSpace#5454: Group edits are immediately reflected in the user interface

Author: tdonohue

src/app/access-control/group-registry/group-form/group-form.component.spec.ts

@@ -3,7 +3,9 @@ import { HttpClient } from '@angular/common/http';
 import { CUSTOM_ELEMENTS_SCHEMA } from '@angular/core';
 import {
   ComponentFixture,
+  fakeAsync,
   TestBed,
+  tick,
   waitForAsync,
 } from '@angular/core/testing';
 import {
@@ -344,6 +346,19 @@ describe('GroupFormComponent', () => {
         component.ngOnInit();
       });
 
+      it('should update the form fields with the new values after successful edit', fakeAsync(() => {
+        component.groupName.setValue('newGroupName');
+        component.groupDescription.setValue(groupDescription);
+
+        component.onSubmit();
+        tick();
+
+        expect(component.formGroup.value.groupName).toBe(expected2.name);
+        expect(component.formGroup.value.groupDescription).toBe(
+          expected2.firstMetadataValue('dc.description'),
+        );
+      }));
+
       it('should edit with name and description operations', () => {
         component.groupName.setValue('newGroupName');
         component.onSubmit();

src/app/access-control/group-registry/group-form/group-form.component.ts

@@ -416,6 +416,16 @@ export class GroupFormComponent implements OnInit, OnDestroy {
       getFirstCompletedRemoteData(),
     ).subscribe((rd: RemoteData<Group>) => {
       if (rd.hasSucceeded) {
+
+        const updatedGroup = rd.payload;
+
+        this.groupRegistryService.editGroup(updatedGroup);
+
+        this.formGroup.patchValue({
+          groupName: updatedGroup.name,
+          groupDescription: updatedGroup.firstMetadataValue('dc.description'),
+        });
+
         this.notificationsService.success(this.translateService.get(this.messagePrefix + '.notification.edited.success', { name: this.dsoNameService.getName(rd.payload) }));
         this.submitForm.emit(rd.payload);
       } else {