Bug 15482 - Koha does not like the full-width commercial character in an email address
Summary: Koha does not like the full-width commercial character in an email address
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Patrons (show other bugs)
Version: Main
Hardware: All All
: P5 - low minor (vote)
Assignee: Bugs List
QA Contact: Testopia
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-01-06 15:10 UTC by Marcel de Rooy
Modified: 2016-01-18 10:45 UTC (History)
3 users (show)

See Also:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Marcel de Rooy 2016-01-06 15:10:16 UTC
Funny bug, really happened:
Put @ (full-width commercial; U+FF20) in a patron email address.
And see that Letters.pm does not like it. The address will not be accepted, no mail sent.
The address appeared to be copied from a page where the full-width character is used to discourage spammers.

For the solution two directions immediately come up:
Left: Change it where we edit it.
Right: But suppose that we like the anti-spam effect of it (although this is actually minimal of course), we could do a replace in Letters.pm (and maybe some more places where we directly send an email).

Your opinion is welcome.
Comment 1 Marc Véron 2016-01-08 15:14:25 UTC
Tested and confirmed.

I would go to the left (accept the letter in validation JavaScript and while validation with JavaScript disabled, replace it before storing to the database).
Comment 2 Marcel de Rooy 2016-01-18 10:45:50 UTC
This does not apply anymore to master. It is not possible in opac and staff to put this email address in the patron record. (This check was not in older versions of Koha.)