jquery procedure partially working

Marius Vasile

I have a jquery to trigger submit button but out of 4 ifs, only two are working properly the other two are not. I was looking for error for some hours but I don't see it. bool1 and 3 works, 2 and 4 doesn't

$("#btnSubmitNM").click(function () {
            var msg = "";
            if (smLocation.lenght > 3 && smActivity.lenght > 3 && smDescription.lenght > 3) {
                //bool 1
                if (($("#Equip").prop('checked') == true) && ($("#PTW").prop('checked') == true)) {
                    if (equipText.lenght > 3 && ptwText.lenght > 3) {
                        $('#NMNew').submit();
                    }
                    else {
                        msg = msg + "All fields have to be filled";
                    }
                }
                //bool 2
                else if (($("#Equip").prop('checked') == true) && ($("#PTW").prop('checked') == false)) {
                    if (equipText.lenght > 3) {
                        $('#NMNew').submit();
                    }
                    else {
                        msg = msg + "All fields have to be filled";
                    }
                }
                //bool 3
                else if (($("#Equip").prop('checked') == false) && ($("#PTW").prop('checked') == true)) {
                    if (ptwText.lenght > 3) {
                        $('#NMNew').submit();
                    }
                    else {
                        msg = msg + "All fields have to be filled";
                    }
                }
                //bool 4
                else if (($("#Equip").prop('checked') == false) && ($("#PTW").prop('checked') == false)) {
                    $('#NMNew').submit();
                }
            }
            else {
                msg = msg + "All fields have to be filled";
            }
            $("#msg").text(msg);
        });

and html


        Near Miss
        
            
                Date/Time:
                
            
            
                User Name:
                
                
            
            
                Location:
                
            
        

        
            
                Was a Permit to Work active?
            
            
                
            
            
                
            
        

        
            
                Was an equipment involved?
            
            
                
            
            
                
            
        

        
            
                Activity performed:
                
            
            
                Description of Near-Miss:

7 Replies

Know the answer? Post it — somebody with the same question will find it here.

Sachin Singh5y ago
Accepted answer
make sure to set type="button" when dealing with buttons, otherwise, it would be considered as a submit button causing postback.
Marius Vasile5y ago
Sachin, I have the same problem, it does not consider Equip checkbox and equipText
Sachin Singh5y ago
yes, that is what I am doing with just 2 conditions.
Marius Vasile5y ago
Thank you Salman, both checkboxes are built similarly even in the model class
Marius Vasile5y ago
Thank you Sachin, so, I have 4 situation, A and B checkboxes, both checked, both not checked, A checked B unchecked and A unchecked B checked. If a checkbox is checked I want to make sure the associated texbox is filled
Salman Beg5y ago
The code looks good to me but what you can do is put breakpoint inside your click function and can check what is the value you are getting for 2 and 4 condition. Maybe for 2 and 4 condition you are getting some other value like null or undefined and based on that you can properly handle the code. Thanks
Sachin Singh5y ago
I could not understand the conditions,
As per your conditions, the form will be submitted, even if both checkboxes are not checked.
Then why not just 2 conditions
if(($("#Equip").prop('checked') == true)$$equipText.lenght > 3)||($("#PTW").prop('checked') == true)&& ptwText.lenght > 3))
{
$('#NMNew').submit();
}
else if

(($("#Equip").prop('checked') == false) && ($("#PTW").prop('checked') == false)) {
                    $('#NMNew').submit();
                }
else
{
msg = msg + "All fields have to be filled";
}