Merge pull request DSpace#5666 from DSpace/backport-4999-to-dspace-8_x

[Port dspace-8_x] fix: use built-in Angular email validator for eperson form

Author: tdonohue

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

@@ -316,15 +316,28 @@ describe('EPersonFormComponent', () => {
       });
     });
 
+    describe('with uppercased email specified', () => {
+      beforeEach(() => {
+        component.formGroup.controls.firstName.setValue('test');
+        component.formGroup.controls.lastName.setValue('test');
+        component.formGroup.controls.email.setValue('TEST@test.com');
+        fixture.detectChanges();
+      });
+
+      it('passes validation check', () => {
+        expect(component.formGroup.controls.email.valid).toBeTrue();
+        expect(component.formGroup.controls.email.errors).toBeNull();
+      });
+    });
 
     describe('after inserting email wrong should show pattern validation error', () => {
       beforeEach(() => {
-        component.formGroup.controls.email.setValue('test@test');
+        component.formGroup.controls.email.setValue('test');
         fixture.detectChanges();
       });
       it('email should not be valid because the email pattern', () => {
         expect(component.formGroup.controls.email.valid).toBeFalse();
-        expect(component.formGroup.controls.email.errors.pattern).toBeTruthy();
+        expect(component.formGroup.controls.email.errors.email).toBeTruthy();
       });
     });
 

src/app/access-control/epeople-registry/eperson-form/eperson-form.component.ts

@@ -307,12 +307,12 @@ export class EPersonFormComponent implements OnInit, OnDestroy {
       name: 'email',
       validators: {
         required: null,
-        pattern: '^[a-z0-9._%+-]+@[a-z0-9.-]+\\.[a-z]{2,4}$',
+        email: null,
       },
       required: true,
       errorMessages: {
         emailTaken: 'error.validation.emailTaken',
-        pattern: 'error.validation.NotValidEmail',
+        email: 'error.validation.NotValidEmail',
       },
       hint: this.translateService.instant(`${this.messagePrefix}.emailHint`),
     });