CODEWITHSUNDEEP
javascriptformsvalidation
Aleski
Aleski

Reputation: 1442

Javascript Form Validation Issue (<SELECT> Tag)

$(document).ready(function(){

$('#userForm').submit(function(){

    user_id = $('#user_id').val();

    valid = true;

    if (user_id > 0) {
        $('#user_id').removeClass('error');
        } else {
        valid = false;
        $('#user_id').addClass('error');
    }



    return valid;

    });

});

The above code should add a class to the user_id select I am using. This code snipet I am using on another page and it works fine but this one just seems to not like me. I have checkec all the variable names and its still not working.

This is validationg a SELECT (drop down menu)

Here it the relative HTML

<form method="post" action="index.php?action=save&course_id=<?php echo $_REQUEST['course_id']?>" id="userForm">
  <input name="company_id" value="<?php echo $companyData->id?>" type="hidden" size="20" />
  <select name="user_id" id="user_id" class="">
    <option value="0" selected="selected">Select a User</option>
    <?php if (count($userListData) > 0) {
            foreach ($userListData as $userList) { ?>
    <option value="<?=$userList->id?>"><?=$userList->firstname?> <?=$userList->surname?></option>
<?php } } ?>
  </select>
  <input name="submit" type="submit" value="Assign this user" />
  </form>

Upvotes: 0

Views: 675

Answers (1)

tkahn
tkahn

Reputation: 1417

Try:

if (user_id.length > 0)

This checks if there's a value for user_id at all, but if the select contains values like "-123" or "0" you might want to try a different approach:

if(user_id.length > 0) {
   if(parseInt(user_id) > 0) {
       // Has a value and is bigger than zero
   }
}

Upvotes: 1

Related Questions