Skip to content

Commit

Permalink
Update validate.js
Browse files Browse the repository at this point in the history
I simply put  
new RegExp(" ... ")
around the email validation RegEx.
Reason : At least one very famous JS minifier which is in widespread use does stumble over this complex pattern, especially because they have problem recognizing them as such in the first place.
The simple 'wrap' makes it clearly and surely possible to identify the RegEx as such. 
The Minifier I tested the most with and draw my conclusion from is JShrink from tedious ( https://github.com/tedious/JShrink ).
  • Loading branch information
Lab5-Switzerland committed Aug 4, 2020
1 parent 6e84eb7 commit dcf5230
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion media/system/js/validate.js

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

0 comments on commit dcf5230

Please sign in to comment.