laredotornado-8iDLEspWrrZBDgjK7y7TUQ@public.gmane.org
2008-Jan-29 15:51 UTC
I only want to validate when field is not empty
Hi, I have a field in my user model: <p><label for="user_work_phone">Work phone</label><br/> <%= text_field ''user'', ''work_phone'' %></p> But I only want to validate it if it is non-empty. So I put this into my user model: validates_as_phone :work_phone, :if => !:work_phone.blank? && :is_client?, :message => "is not a valid phone number" But if I leave the field blank and submit the form, the form is returned to me with the message "Work phone is not a valid phone number:". What is wrong with the above statement? Thanks, - Dave --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "Ruby on Rails: Talk" group. To post to this group, send email to rubyonrails-talk-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org To unsubscribe from this group, send email to rubyonrails-talk-unsubscribe-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org For more options, visit this group at http://groups.google.com/group/rubyonrails-talk?hl=en -~----------~----~----~----~------~----~------~--~---
On Jan 29, 2008, at 9:51 AM, laredotornado-8iDLEspWrrZBDgjK7y7TUQ@public.gmane.org wrote:> > Hi, > > I have a field in my user model: > > <p><label for="user_work_phone">Work phone</label><br/> > <%= text_field ''user'', ''work_phone'' %></p> > > But I only want to validate it if it is non-empty. So I put this into > my user model: > > validates_as_phone :work_phone, :if => !:work_phone.blank? > && :is_client?, :message => "is not a valid phone number" > > But if I leave the field blank and submit the form, the form is > returned to me with the message "Work phone is not a valid phone > number:". What is wrong with the above statement? > > Thanks, - DaveTry :allow_nil => true Peace, Phillip --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "Ruby on Rails: Talk" group. To post to this group, send email to rubyonrails-talk-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org To unsubscribe from this group, send email to rubyonrails-talk-unsubscribe-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org For more options, visit this group at http://groups.google.com/group/rubyonrails-talk?hl=en -~----------~----~----~----~------~----~------~--~---
laredotornado-8iDLEspWrrZBDgjK7y7TUQ@public.gmane.org
2008-Jan-29 16:59 UTC
Re: I only want to validate when field is not empty
Thanks. It''s still not working but I''m using a "validate_as_phone" plugin validates_as_phone :work_phone, :allow_nil => true, :message => "is not a valid phone number" How did you envision doing this without such a plug-in? Best, - Dave On Jan 29, 10:08 am, Phillip Koebbe <phillipkoe...-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> wrote:> On Jan 29, 2008, at 9:51 AM, laredotorn...-8iDLEspWrrZBDgjK7y7TUQ@public.gmane.org wrote: > > > > > > > > > Hi, > > > I have a field in my user model: > > > <p><label for="user_work_phone">Work phone</label><br/> > > <%= text_field ''user'', ''work_phone'' %></p> > > > But I only want to validate it if it is non-empty. So I put this into > > my user model: > > > validates_as_phone :work_phone, :if => !:work_phone.blank? > > && :is_client?, :message => "is not a valid phone number" > > > But if I leave the field blank and submit the form, the form is > > returned to me with the message "Work phone is not a valid phone > > number:". What is wrong with the above statement? > > > Thanks, - Dave > > Try :allow_nil => true > > Peace, > Phillip- Hide quoted text - > > - Show quoted text ---~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "Ruby on Rails: Talk" group. To post to this group, send email to rubyonrails-talk-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org To unsubscribe from this group, send email to rubyonrails-talk-unsubscribe@googlegroups.com For more options, visit this group at http://groups.google.com/group/rubyonrails-talk?hl=en -~----------~----~----~----~------~----~------~--~---
On Jan 29, 2008, at 10:59 AM, laredotornado-8iDLEspWrrZBDgjK7y7TUQ@public.gmane.org wrote:> > Thanks. It''s still not working but I''m using a "validate_as_phone" > plugin > > validates_as_phone :work_phone, :allow_nil => true, :message > => "is not a valid phone number" > > How did you envision doing this without such a plug-in? Best, - Dave >Didn''t see that as a plugin. Sorry. It seems that the plugin would need to support blank values. Apart from using the plugin, I guess you could use some regex. Look in the plugin to see how it''s done. You could probably just create your own private method in the class to handle it. Or, you might be able to extend the plugin. Peace, Phillip --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "Ruby on Rails: Talk" group. To post to this group, send email to rubyonrails-talk-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org To unsubscribe from this group, send email to rubyonrails-talk-unsubscribe-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org For more options, visit this group at http://groups.google.com/group/rubyonrails-talk?hl=en -~----------~----~----~----~------~----~------~--~---
Not used the validates_as_phone plugin before, but looking at the source for validates_as_phone, it is only allowing lengths of 7, 10 and 11 for the string length. You could either ''monkey patch'' the plugin to do what you want (e.g. accept string length of 0), or use the plugin''s code as a base for writing your own validation plugin. Rails 2 also has an allow_blank property for validations but i''m not sure if this would be supported by this custom plugin. I can''t see anywhere in the code where it would check this explicitly but you could try this if you are on v2. allow_nil will only validate if the actual value from the parameters hash is nil, not if it is "" The plugin doesn''t really look too amazing, to be honest, It only strips out any non-numeric characters and checks for length. Sure that US phone numbers are more complex than that! It is also not very optimised, calling the same methods several times (namely new_value.length) instead of storing this computation in a variable. Good luck joe On Jan 29, 5:09 pm, Phillip Koebbe <phillipkoe...-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> wrote:> On Jan 29, 2008, at 10:59 AM, laredotorn...-8iDLEspWrrZBDgjK7y7TUQ@public.gmane.org wrote: > > > > > Thanks. It''s still not working but I''m using a "validate_as_phone" > > plugin > > > validates_as_phone :work_phone, :allow_nil => true, :message > > => "is not a valid phone number" > > > How did you envision doing this without such a plug-in? Best, - Dave > > Didn''t see that as a plugin. Sorry. It seems that the plugin would > need to support blank values. > > Apart from using the plugin, I guess you could use some regex. Look > in the plugin to see how it''s done. You could probably just create > your own private method in the class to handle it. Or, you might be > able to extend the plugin. > > Peace, > Phillip--~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "Ruby on Rails: Talk" group. To post to this group, send email to rubyonrails-talk-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org To unsubscribe from this group, send email to rubyonrails-talk-unsubscribe-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org For more options, visit this group at http://groups.google.com/group/rubyonrails-talk?hl=en -~----------~----~----~----~------~----~------~--~---
laredotornado-8iDLEspWrrZBDgjK7y7TUQ@public.gmane.org
2008-Jan-29 23:46 UTC
Re: I only want to validate when field is not empty
Thanks for this feedback. I am very open to using other methods of validating US phone numbers, I just happened to find this one in a Google search. What do you use? - Dave On Jan 29, 3:15 pm, Wildtangent <wildtang...-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> wrote:> Not used the validates_as_phone plugin before, but looking at the > source for validates_as_phone, it is only allowing lengths of 7, 10 > and 11 for the string length. > You could either ''monkey patch'' the plugin to do what you want (e.g. > accept string length of 0), or use the plugin''s code as a base for > writing your own validation plugin. > > Rails 2 also has an allow_blank property for validations but i''m not > sure if this would be supported by this custom plugin. I can''t see > anywhere in the code where it would check this explicitly but you > could try this if you are on v2. allow_nil will only validate if the > actual value from the parameters hash is nil, not if it is "" > > The plugin doesn''t really look too amazing, to be honest, It only > strips out any non-numeric characters and checks for length. Sure that > US phone numbers are more complex than that! It is also not very > optimised, calling the same methods several times (namely > new_value.length) instead of storing this computation in a variable. > > Good luck > joe > > On Jan 29, 5:09 pm, Phillip Koebbe <phillipkoe...-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> wrote: > > > > > On Jan 29, 2008, at 10:59 AM, laredotorn...-8iDLEspWrrZBDgjK7y7TUQ@public.gmane.org wrote: > > > > Thanks. It''s still not working but I''m using a "validate_as_phone" > > > plugin > > > > validates_as_phone :work_phone, :allow_nil => true, :message > > > => "is not a valid phone number" > > > > How did you envision doing this without such a plug-in? Best, - Dave > > > Didn''t see that as a plugin. Sorry. It seems that the plugin would > > need to support blank values. > > > Apart from using the plugin, I guess you could use some regex. Look > > in the plugin to see how it''s done. You could probably just create > > your own private method in the class to handle it. Or, you might be > > able to extend the plugin. > > > Peace, > > Phillip- Hide quoted text - > > - Show quoted text ---~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "Ruby on Rails: Talk" group. To post to this group, send email to rubyonrails-talk-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org To unsubscribe from this group, send email to rubyonrails-talk-unsubscribe@googlegroups.com For more options, visit this group at http://groups.google.com/group/rubyonrails-talk?hl=en -~----------~----~----~----~------~----~------~--~---