Question

$('form#injuryReportSubmit').submit(function(){             
if(($('form#injuryReportSubmit textarea.description').val().length <=0) || ($('form#injuryReportSubmit select.part').val()== "choose one...") ||($('form#injuryReportSubmit select.type').val() == "choose one...") || ($('form#injuryReportSubmit select.weather').val()=="choose one..."));
            {  
                $('div#errorMessage').show();
                return false;
            }           
}); 

The code above is used to validate a form before it submits. The problem is the form will not submit even when all the test are false. Can someone help?
the form is on a jsp and looks like

<form id ="injuryReportSubmit" method ="post" action="injuryReportingPage.html" >
        <p class ="first" >Date Of Injury</p>
        <p class ="second">Date Reported to Manager</p>
        <input type="text" id="dateOfInjury" name="dateOfInjury">       
        <input type="text" id="dateReported" name ="dateReported">
        <p class ="weather">Weather</p>
        <p class ="injuryType">Injury Type</p>
        <p class ="bodyPart">Body Part</p>
        <p class ="time">Time Injury Occurred</p>
        <p class ="description">Description</p>
        <select class ="weather" name="weather">
        <%if(InjuryReportController.getWeatherList() != null){ %>   
        <%   for(Weather weather : InjuryReportController.getWeatherList()){%>
        <option><%= weather.getWeatherCondition() %></option>
        <%} }%>
        <option >choose one...</option>
        </select>

        <select class ="type" name="injuryType">        
        <%if(InjuryReportController.getInjuryTypeList() != null){ %>    
        <%   for(InjuryType injuryType : InjuryReportController.getInjuryTypeList()){%>
        <option><%= injuryType.getInjuryTypeName() %></option>
        <%} }%>
        <option>choose one...</option>
        </select>

        <select class ="part" name="bodyPart">      
        <%if(InjuryReportController.getBodyPartList() != null){ %>  
        <%   for(BodyPart bodyPart : InjuryReportController.getBodyPartList()){%>
        <option><%= bodyPart.getBodyPartName() %></option>
        <%} }%>
        <option >choose one...</option>
        </select>

        <input type="text" id="timeP" name ="timeOfInjury" value="01:00  AM">
        <textarea class ="description"  rows="120" cols="670" name="description"></textarea>

        <input id ="report" value="Submit Report" type ="submit">

        </form>

Answer 1

Remove the semicolon from the end of line 5 of your code.

Currently what you’ve got is like this:

if (/*your conditions*/);        // <- note the semicolon
{
    ...
    return false;
}

This means that the block with the curly braces is not associated with the if statement and so will execute every time. Obviously then returning false every time cancels every submit.