DavyJonesLocker / client_side_validations-simple_form

Simple Form plugin for ClientSideValidations
MIT License
253 stars 102 forks source link

inputs with :as => :radio_buttons are being skipped in validation #20

Open vitalim opened 11 years ago

vitalim commented 11 years ago

In a form I'm building using ClientSideValidations + SimpleForm (with the gem plugin) the client side validation works for all fields except the :radio_buttons. The HTML that is produced for this group is similar to the other working ones:

<div class="control-group radio_buttons required"> ... </div>

Yet when the submit button is clicked, all the other inputs get the 'error' class but the radio_buttons input does not. If I remove the :as => :radio_buttons, the client side validations works just fine.

Any idea what is causing this? Thanks!

bcardarella commented 11 years ago

Radio buttons were filtered out of the parent ClientSideValidations gem but were always intended to be put back in with ClientSideValidations-SimpleForm and ClientSideValidations-Formtastic.

They are being filtered because in the regular Rails form builder there is no concept of wrapping the radio buttons in a single group so it is not possible to render the errors in a similar manner as the server will render.

However, with Formtastic and SimpleForm it does render as a group so the render function can attach to this wrapper element. For some reason I never got around to putting this functionality back in. I should prioritize it.

Thank you for bringing it to my attention.

daliusg commented 11 years ago

Is there a hotfix for this?

jerryshen commented 11 years ago

I'm having this same problem. is there any hot fix for that?

matenia commented 11 years ago

+1 for hotfix .. having a hard time implementing a reliable solution in client_side_validations-simple_form

KODerFunk commented 11 years ago

+1 Bug in local presense validator. $().val() from check-box return value, without considering that it must be checked.

hau commented 11 years ago

+1

danielballan commented 11 years ago

+1

alexfalkowski commented 11 years ago

@bcardarella - Any chance on providing some guidance on what needs to be do to implement this fix?

elijahmurray commented 10 years ago

+1

AlecRust commented 9 years ago

+1

manishie commented 9 years ago

Here's a quick hack around the problem. It seems as though even though radio buttons are not processed, the gem is still sending data about the radio button validations to the browser. Type 'window.ClientSideValidations.forms' into the JS console and you'll see what gets sent down. So I wrote a bit of jQuery that checks all of my forms for radio buttons, then sees if there are any presence validations for those buttons. If so, it displays an error message. Note that in Rails, you'll need to wrap your related radio buttons into a div. Hope it helps someone. http://pastebin.com/FMzS9dBj

tagliala commented 9 years ago

@manishie thanks. I'm not confident to add this at the moment but this will definitely help.

manishie commented 9 years ago

oh, I didn't mean for you to add it! It's just a quick hack I threw together to get around the problem. Thanks!

tagliala commented 9 years ago

@manishie sure I got you didn't mean that. I just was informing everyone that I will not do this soon. PR with proper tests are welcomed

august-9426 commented 7 years ago

had this fixed?

tagliala commented 7 years ago

@August7752 sorry for the very late reply, I've missed this notification.

This feature is not implemented and I will probably not implement this. PR with proper tests are very welcomed

@cooljl31 please use GitHub reactions to add upvotes. Other +1s are here just because there weren't github reactions at the time

sekrett commented 5 years ago

@manishie I did some experimentation and realised, that not all data is sent from server. For example radio buttons and check boxes are not present in form data attribute, regardless what validations (standard or custom) I add to them. Maybe it worked on an older version of client-side-validations?

jhovad commented 5 years ago

+1 please

MichalRemis commented 4 years ago

Yes validations for radio_buttons etc.. aren't in CSV Hash, because SimpleForm doesn't use (anymore) FormBuilder's methods (which CSV overrides to build validation hash) to to build these special inputs. But it uses ActionView:Helpers::Tags for these now, so there is some more overriding needed. I am looking into it.. wan't to make it work.

MichalRemis commented 4 years ago

I implemented missing validations into CSV SimpleForm (checkboxes, radios, date_time selects). Meanwhile, until it will be merged into main gem you may use it from my repo.

gem 'client_side_validations-simple_form', github: 'MichalRemis/client_side_validations-simple_form'

It works only when JS is added via sprockets

//= require rails.validations
//= require rails.validations.simple_form.bootstrap4

because I don't provide npm package.

tagliala commented 4 years ago

because I don't provide npm package.

it is possible to point npm packages to github repos:

In package.json:

    "package": "user/repo#branch",