onSubmit client script to prevent form submission

patricklatella · ‎05-17-2017

hi all,

I have a javascript question. I have an onSubmit catalog client script to prevent submission of a record producer if the user has not checked a box on the form. This is working properly. However I need this to only apply if another field on the form is set a certain way. Right now here is my script:

this part is working properly:

if(g_form.getValue(user_confirm') == 'false' ){

alert('Please be sure to check the box before submitting a ticket.');

return false;

but it's the first "if" part that I need to tweak, I only want the script to fire if the field on the form "ad_domain" is set to YES. If this field is set to NO I don't want the script to fire.

how do I do this? thanks!

akaupisch · ‎05-17-2017

While that solution works, it's not very user friendly. Why have them submit, then cancel and require them to click a checkbox. You should make their life easier...I'm not sure what this checkbox is, but you have several options:

1) This is the best user experience and what you should be doing... Create a UI policy ...under When to Apply...Add Filter Conition ad_domain = YES or ad_comain=YesB and question_choide = incorrect info. Leave Reverse if False...then Save. After it reloads go to UI Policy Actions, select new, add in the checkbox field and set Mandatory to true. This will pre-empt the user to check it prior to submission. to inform them after submission is the least desirable as it adds clicks...Submit -> Ok on alert -> check box -> Submit and hope you don't rinse repeat due to other onSubmits...

2) If you can't do that for some reason, then don't hate on your users and require them to check a box then have to resubmit... use a confirm and do it for them...

function onSubmit() {   

   if(g_form.getValue('ad_domain') == 'YES' || g_form.getValue('ad_domain') == 'YESb' && g_form.getValue('question_choice') == 'incorrect info' ) //assuming values are YES and YESb   

   {   

             if(g_form.getValue('user_confirm') == 'false' ){   

                   var result = confirm('Are you sure you wish to submit?');//or whatever that checkbox is for

                   if (result) {

                             g_form.setValue('checkbox', true);

                   else

                             return false;

             }   

   

   }   

}

*edit: changed prompt to confirm*

View solution in original post

Pradeep Sharma · ‎05-17-2017

Hello Pat,

The script should be

function onSubmit() {

if(g_form.getValue('ad_domain') == 'YES' || g_form.getValue('ad_domain') == 'YESb' && g_form.getValue('question_choice') == 'incorrect_info')

{

if(g_form.getValue(user_confirm') == 'false' ){ //this is the checkbox

alert('Please be sure to check the box before submitting a ticket.');

return false;

}

patricklatella · ‎05-17-2017

this part doesn't seem to be working

&& g_form.getValue('question_choice') == 'incorrect_info')

because the client script is firing no matter which answer to 'question_choice' is selected.

patricklatella · ‎05-17-2017

it's also NOT firing if ad_domain = YESb, but the other conditions are in place

Pradeep Sharma · ‎05-17-2017

Make sure the choice value is correct I.e YESb ..I'm assuming it is YESB..please check the same once

patricklatella · ‎05-17-2017

yep, that part is correct, YES and YESb are just place holders for my actual values, which are CTS and CTSL, but these are correct in my code and are the 2 values for ad_domain that should trigger the client script. Right now only the CTS value is triggering, but not the CTSL.

if(g_form.getValue('ad_domain') == 'CTS' || g_form.getValue('ad_domain') == 'CTSL' && g_form.getValue('cts_question_issue') == 'cts_question_incorrect')

this is my actual code, where 'cts_question_isse' and 'cts_question-incorrect' are the field and value respectively that should as well be true for the client script to fire.

again, as it is any answer to the 'cts_question_issue' question is causing it to fire...but again only if the ad_domain is 'CTS'. so there are a couple holes still. the rest of the code I've triple checked against yours and matches.