? Pending

User tests: Successful: Unsuccessful:

avatar shagungarg0
shagungarg0
9 Feb 2022

Pull Request for Issue #36981

###Summary Of Changes
$searchEmail = '([\w.'-+]+@(?:[a-z0-9.-]+.)+(?:[a-zA-Z0-9-]{2,10}))';
This accepts only 10 characters due to which some emails were not picked by server and hence people were facing problem.

Actual result BEFORE applying this Pull Request

Once the plugin(email-cloak) is activated, part of the TLD will be truncated. Example:
office@some.international
becomes:
office@some.internatio

Expected result AFTER applying this Pull Request

E-mail will not be truncated

avatar shagungarg0 shagungarg0 - open - 9 Feb 2022
avatar shagungarg0 shagungarg0 - change - 9 Feb 2022
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 9 Feb 2022
Category Front End Plugins
avatar zero-24
zero-24 - comment - 9 Feb 2022

Hi, can you please send the PR against the 3.10 branch? After that it will be merged up to 4.1 and 4.2

From my understanding we should just remove the caracter limit for the domain part? Why do you want to change more things?

Maybe there is a regex from the rfc sonewhere?

avatar shagungarg0
shagungarg0 - comment - 9 Feb 2022

Hi, can you please send the PR against the 3.10 branch? After that it will be merged up to 4.1 and 4.2

From my understanding we should just remove the caracter limit for the domain part? Why do you want to change more things?

Maybe there is a regex from the rfc sonewhere?

#36986

avatar zero-24
zero-24 - comment - 9 Feb 2022

Thanks

avatar richard67
richard67 - comment - 9 Feb 2022

Closing in favor of #36986 .

avatar richard67 richard67 - change - 9 Feb 2022
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2022-02-09 21:30:35
Closed_By richard67
Labels Added: ?
avatar richard67 richard67 - close - 9 Feb 2022

Add a Comment

Login with GitHub to post a comment