Skip to main content

This site requires you to update your browser. Your browsing experience maybe affected by not having the most up to date version.

All other Modules

Discuss all other Modules here.

Moderators: martimiz, Sean, biapar, Willr, Ingo, swaiba, simon_w

userforms - double validation scripts


Reply

6 Posts   1426 Views

Avatar
patte

20 April 2010 at 6:43pm Community Member, 63 Posts

Hey guys

is it just me or does the userforms validation add the #Form_Form .validate two times? In my pages or some SilverStripe community projects (example: http://www.fmc-innovation.de/kontakt) there is also this double form validation... is this needed by jquery or userforms? A bug?

patte

Avatar
klikhier

28 October 2010 at 6:10am Community Member, 150 Posts

Today I found that this is indeed the case (at many of my SS websites). Ran into this issue with SS2.4.1 and latest UserForms module...

Avatar
Willr

28 October 2010 at 12:12pm Forum Moderator, 5511 Posts

Sounds like a bug. Raise it as a ticket on open.silverstripe.org and assign it to UserForms and I'll look into it as soon as possible.

Thanks,

Avatar
klikhier

28 October 2010 at 8:09pm Community Member, 150 Posts

Done. Another question/remark:

When using UserForms with Translatable, the only way I got language dependent validation working is to remove

UserDefinedFrom.php line 473: removed $messages from custom validation script;

and then load 'userforms/thirdparty/jquery-validate/localization/messages_nl.js' dynamically, based on the current locale (example _nl for locale nl_NL).

I'm not sure if my solution is a correct approach (apart from changing the core), but, if so, this may be a good enhancement to enable language dependent validation messages in user forms?!

Avatar
klikhier

28 October 2010 at 10:37pm Community Member, 150 Posts

With regards to the initial problem of this topic (double custom JS validation script): this seems to be resolved in the latest trunk release of UDF.

Avatar
patte

29 October 2010 at 4:59am Community Member, 63 Posts

Thanks for that!

Next time I`ll raise a ticket ;-)