@@ -, +, @@ --- opac/opac-memberentry.pl | 2 +- t/Koha/Email.t | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) --- a/opac/opac-memberentry.pl +++ a/opac/opac-memberentry.pl @@ -439,7 +439,7 @@ sub CheckForInvalidFields { my $borrower = shift; my @invalidFields; if ($borrower->{'email'}) { - unless ( Koha::Email->is_valid($borrower->{email}) { + unless ( Koha::Email->is_valid($borrower->{email}) ) { push(@invalidFields, "email"); } elsif ( C4::Context->preference("PatronSelfRegistrationEmailMustBeUnique") ) { my $patrons_with_same_email = Koha::Patrons->search( # FIXME Should be search_limited? --- a/t/Koha/Email.t +++ a/t/Koha/Email.t @@ -255,10 +255,10 @@ subtest 'is_valid' => sub { is(Koha::Email->is_valid('Fróm '), 1); is(Koha::Email->is_valid('from@example.com'), 1); is(Koha::Email->is_valid(''), 1); + is(Koha::Email->is_valid('root@localhost'), 1); # See bug 28017 is(Koha::Email->is_valid(''), 0); # "In accordance with RFC 822 and its descendants, this module demands that email addresses be ASCII only" isnt(Koha::Email->is_valid('@example.com'), 1); isnt(Koha::Email->is_valid('example.com'), 1); - isnt(Koha::Email->is_valid('root@localhost'), 1); isnt(Koha::Email->is_valid('from'), 1); }; --