RE: [vote] deprecate/remove EmailAddressValidator and move RfcCompliantEmailAddressValidator to core

2012-07-23 Thread Burton, Tom F (DOR)
[mailto:guillaume.s...@gmail.com] Sent: Sunday, July 22, 2012 9:18 AM To: dev@wicket.apache.org Subject: Re: [vote] deprecate/remove EmailAddressValidator and move RfcCompliantEmailAddressValidator to core Hi Martijn, On Fri, Jul 20, 2012 at 2:49 PM, Martijn Dashorst martijn.dasho...@gmail.com wrote

Re: [vote] deprecate/remove EmailAddressValidator and move RfcCompliantEmailAddressValidator to core

2012-07-22 Thread Emond Papegaaij
@wicket.apache.org Subject: [vote] deprecate/remove EmailAddressValidator and move RfcCompliantEmailAddressValidator to core Date: Fri, Jul 20, 2012 4:50 am EmailAddressValidator is broken. For example, it doesn't validate Martijn Dashorst martijn.dasho...@gmail.com properly, which

Re: [vote] deprecate/remove EmailAddressValidator and move RfcCompliantEmailAddressValidator to core

2012-07-22 Thread Guillaume Smet
Hi Martijn, On Fri, Jul 20, 2012 at 2:49 PM, Martijn Dashorst martijn.dasho...@gmail.com wrote: [X] Keep it as currently implemented Or change their names if they are not clear enough. I think both implementations cover different use cases. In our apps, we usually favor a field for the email

Re: [vote] deprecate/remove EmailAddressValidator and move RfcCompliantEmailAddressValidator to core

2012-07-21 Thread Martin Grigorov
[+1] Copy RfcCompliantEmailAddressValidator over EmailAddressValidator, On Fri, Jul 20, 2012 at 3:49 PM, Martijn Dashorst martijn.dasho...@gmail.com wrote: EmailAddressValidator is broken. For example, it doesn't validate Martijn Dashorst martijn.dasho...@gmail.com properly, which is standards

Re: [vote] deprecate/remove EmailAddressValidator and move RfcCompliantEmailAddressValidator to core

2012-07-21 Thread Andrea Del Bene
[X] Copy RfcCompliantEmailAddressValidator over EmailAddressValidator, deprecate RfcCompliantEmailAddressValidator EmailAddressValidator is broken. For example, it doesn't validate Martijn Dashorst martijn.dasho...@gmail.com properly, which is standards compliant. Rfc validator does work. The

Re: [vote] deprecate/remove EmailAddressValidator and move RfcCompliantEmailAddressValidator to core

2012-07-21 Thread Burton, Tom F (DOR)
@wicket.apache.org Subject: [vote] deprecate/remove EmailAddressValidator and move RfcCompliantEmailAddressValidator to core Date: Fri, Jul 20, 2012 4:50 am EmailAddressValidator is broken. For example, it doesn't validate Martijn Dashorst martijn.dasho...@gmail.com properly, which is standards

Re: [vote] deprecate/remove EmailAddressValidator and move RfcCompliantEmailAddressValidator to core

2012-07-21 Thread Martin Grigorov
from my HTC - Reply message - From: Martijn Dashorst martijn.dasho...@gmail.com To: dev@wicket.apache.org dev@wicket.apache.org Subject: [vote] deprecate/remove EmailAddressValidator and move RfcCompliantEmailAddressValidator to core Date: Fri, Jul 20, 2012 4:50 am

Re: [vote] deprecate/remove EmailAddressValidator and move RfcCompliantEmailAddressValidator to core

2012-07-20 Thread Martijn Dashorst
My vote: [+1] Copy RfcCompliantEmailAddressValidator over EmailAddressValidator, deprecate RfcCompliantEmailAddressValidator [ ] Deprecate EmailAddressValidator, move RfcCompliantEmailAddressValidator to core, favor RFC in EmailTextField [ ] Keep it as currently implemented Martijn

Re: [vote] deprecate/remove EmailAddressValidator and move RfcCompliantEmailAddressValidator to core

2012-07-20 Thread Ron Smits
+1 having the rfc compliant validator in is a good thing On Jul 20, 2012 2:51 PM, Martijn Dashorst martijn.dasho...@gmail.com wrote: My vote: [+1] Copy RfcCompliantEmailAddressValidator over EmailAddressValidator, deprecate RfcCompliantEmailAddressValidator [ ] Deprecate

Re: [vote] deprecate/remove EmailAddressValidator and move RfcCompliantEmailAddressValidator to core

2012-07-20 Thread Sebastien
[+1] Copy RfcCompliantEmailAddressValidator over EmailAddressValidator, deprecate RfcCompliantEmailAddressValidator On Fri, Jul 20, 2012 at 3:22 PM, Ron Smits ron.sm...@gmail.com wrote: +1 having the rfc compliant validator in is a good thing On Jul 20, 2012 2:51 PM, Martijn Dashorst

Re: [vote] deprecate/remove EmailAddressValidator and move RfcCompliantEmailAddressValidator to core

2012-07-20 Thread Martin Dilger
[x] Copy RfcCompliantEmailAddressValidator over EmailAddressValidator, deprecate RfcCompliantEmailAddressValidator 2012/7/20 Martijn Dashorst martijn.dasho...@gmail.com EmailAddressValidator is broken. For example, it doesn't validate Martijn Dashorst martijn.dasho...@gmail.com properly,

Re: [vote] deprecate/remove EmailAddressValidator and move RfcCompliantEmailAddressValidator to core

2012-07-20 Thread Sven Meier
[+1] Copy ... Sven Martijn Dashorst martijn.dasho...@gmail.com schrieb: My vote: [+1] Copy RfcCompliantEmailAddressValidator over EmailAddressValidator, deprecate RfcCompliantEmailAddressValidator [ ] Deprecate EmailAddressValidator, move RfcCompliantEmailAddressValidator to core, favor RFC in

Re: [vote] deprecate/remove EmailAddressValidator and move RfcCompliantEmailAddressValidator to core

2012-07-20 Thread Emond Papegaaij
[x] Keep it as currently implemented EmailAddressValidator is too strict, but at least you can see what it does. The regexp in RfcCompliantEmailAddressValidator is crazy, outdated and probably very inefficient. IMHO, we should mark it as deprecated. The best solution would be to use a parser

Re: [vote] deprecate/remove EmailAddressValidator and move RfcCompliantEmailAddressValidator to core

2012-07-20 Thread Martijn Dashorst
On Fri, Jul 20, 2012 at 4:13 PM, Emond Papegaaij emond.papega...@topicus.nl wrote: [x] Keep it as currently implemented EmailAddressValidator is too strict, but at least you can see what it does. The regexp in RfcCompliantEmailAddressValidator is crazy, outdated and probably very inefficient.