[Lift] Re: Bug in MappedEmail: emailPattern is wrong

2009-09-21 Thread David Pollak
Awaiting review board approval. On Sun, Sep 20, 2009 at 2:23 AM, Richard Dallaway wrote: > > Apologies if I've duplicated the ticket, but I couldn't find it one, > and this issue is going to impact me too. > > http://github.com/dpp/liftweb/issues/#issue/65 > > Cheers > Richard > > On Fri, Sep 18,

[Lift] Re: Bug in MappedEmail: emailPattern is wrong

2009-09-20 Thread Richard Dallaway
Apologies if I've duplicated the ticket, but I couldn't find it one, and this issue is going to impact me too. http://github.com/dpp/liftweb/issues/#issue/65 Cheers Richard On Fri, Sep 18, 2009 at 6:32 PM, David Pollak wrote: > Please open a ticket > > On Wed, Sep 16, 2009 at 11:27 AM, harryh

[Lift] Re: Bug in MappedEmail: emailPattern is wrong

2009-09-18 Thread David Pollak
Please open a ticket On Wed, Sep 16, 2009 at 11:27 AM, harryh wrote: > > Hrm, MappedEmail is now rejecting e-mails with a - in the part before > the @. > > is %-+ being interpreted as % through + and not % OR - OR + ? > > -harryh, who is easily confused by regular expressions > > On Aug 26, 7:32

[Lift] Re: Bug in MappedEmail: emailPattern is wrong

2009-09-16 Thread harryh
Hrm, MappedEmail is now rejecting e-mails with a - in the part before the @. is %-+ being interpreted as % through + and not % OR - OR + ? -harryh, who is easily confused by regular expressions On Aug 26, 7:32 pm, David Pollak wrote: > Thanks. > > Changed and pushed to GitHub.  Allow 2 hours f

[Lift] Re: Bug in MappedEmail: emailPattern is wrong

2009-08-27 Thread Marc Boschma
Your right... I've been thinking about that and how the current mapper only does the regex check.. could always override the validate method... Marc On 27/08/2009, at 6:39 PM, Viktor Klang wrote: > > > On Thu, Aug 27, 2009 at 10:27 AM, Marc Boschma > wrote: > Personally I use: > > "^(?:[a-z0-9

[Lift] Re: Bug in MappedEmail: emailPattern is wrong

2009-08-27 Thread Viktor Klang
On Thu, Aug 27, 2009 at 10:27 AM, Marc Boschma > wrote: > Personally I use: > > > "^(?:[a-z0-9!#$%&'*+/=?^_`{|}~-]+(?:\.[a-z0-9!#$%&'*+/=?^_`{|}~-]+)*|"(?:[\x01-\x08\x0b\x0c\x0e-\x1f\x21\x23-\x5b\x5d-\x7f]|\\[\x01-\x09\x0b\x0c\x0e-\x7f])*")@(?:[a-z0-9](?:[a-z0-9-]*[a-z0-9])?\.)+[a-z0-9](?:[a-z0-9

[Lift] Re: Bug in MappedEmail: emailPattern is wrong

2009-08-27 Thread Marc Boschma
Personally I use: "^(?:[a-z0-9!#$%&'*+/=?^_`{|}~-]+(?:\.[a-z0-9!#$%&'*+/=?^_`{|}~-] +)*|"(?:[\x01-\x08\x0b\x0c\x0e-\x1f\x21\x23-\x5b\x5d-\x7f]|\\[\x01- \x09\x0b\x0c\x0e-\x7f])*")@(?:[a-z0-9](?:[a-z0-9-]*[a-z0-9])?\.)+[a- z0-9](?:[a-z0-9-]*[a-z0-9])?$" Marc On 27/08/2009, at 9:32 AM, David Pol

[Lift] Re: Bug in MappedEmail: emailPattern is wrong

2009-08-26 Thread David Pollak
Thanks. Changed and pushed to GitHub. Allow 2 hours for Maven availability. On Wed, Aug 26, 2009 at 4:15 PM, harryh wrote: > > > What's the change to the RegEx? > > val emailPattern = Pattern.compile("^[a-z0-9._%-+]+@(?:[a-z0-9-]+\\.)+ > [a-z]{2,4}$") > > note the addition of the + to the allo

[Lift] Re: Bug in MappedEmail: emailPattern is wrong

2009-08-26 Thread David Pollak
On Wed, Aug 26, 2009 at 3:15 PM, harryh wrote: > > It doesn't allow for + in the name part of the e-mail (harryh > +l...@gmail.com). > Can one of the lift commiters fix this? It's a one character change. What's the change to the RegEx? > > > -harryh > > > > -- Lift, the simply functional

[Lift] Re: Bug in MappedEmail: emailPattern is wrong

2009-08-26 Thread harryh
> What's the change to the RegEx? val emailPattern = Pattern.compile("^[a-z0-9._%-+]+@(?:[a-z0-9-]+\\.)+ [a-z]{2,4}$") note the addition of the + to the allowed characters before the @ -harryh --~--~-~--~~~---~--~~ You received this message because you are subscr