jquery validation - is there a more efficient way to do this?

Question

Background Information

I have a table where each row represents contact information. It's not necessary to have contact data, but if the user specifies an extension number, they must specify data for every field.

problem

The code I have so far to check for the majority of the fields is working.. but I have a set of checkboxes representing a day in the week. At least one day must be selected. The code right now fails... even when I have one checkbox selected, I keep getting an error.

Also i'm wondering if there's a way to simplify my IF statement that checks the checkbox status. The code is very verbose... and ugly. And here's the HTML and javascript (partial) that is called when they try to submit the form:

$('.tcdata tr').each(function(i, row) {
  var $row = $(row);
  var $ext = $row.find('input[name*="extension"]');
  if ($ext.val()) {
    //extension data has been defined.. make sure you have start time / end time
    var $start = $row.find('input[name*="starttime"]');
    var $end = $row.find('input[name*="endtime"]');
    var $domain = $row.find('input[name*="domain"]');
    if (!$start.val() || !$end.val() || !$domain.val()) {
      $(".btn-warning").html("Whoops! Every time condition must have a start / end time, and a domain");
      e.preventDefault();
      return;
    }

    //check days of week. 
    var $dow_m = $row.find('input[name*="dow_m"]');
    var $dow_t = $row.find('input[name*="dow_t"]');
    var $dow_w = $row.find('input[name*="dow_w"]');
    var $dow_r = $row.find('input[name*="dow_r"]');
    var $dow_f = $row.find('input[name*="dow_f"]');
    var $dow_s = $row.find('input[name*="dow_s"]');
    var $dow_s = $row.find('input[name*="dow_n"]');
    if ((!$("#dow_m").is(':checked')) && (!$("#dow_t").is(':checked')) && (!$("#dow_w").is(':checked')) && (!$("#dow_r").is(':checked')) && (!$("#dow_f").is(':checked')) && (!$("#dow_s").is(':checked')) && (!$("#dow_n").is(':checked'))) {
      //nothing selected
      $(".btn-warning").html("Whoops! Every time condition must have a day of week assign to it");
      e.preventDefault();
      return;
    }


  }
});



  
    
      
        Time Conditions
      
    
    
      Default Rule:
    

    
      
      
      
      
      
        Days of Week:
        Mon 
        Tue 
        Wed 
        Thu 
        Fri 
        Sat 
        Sun 
      
      
    
    
      Addtional Rules:
    
    
      

      
      
      
      Days of Week:Mon Tue Wed Thu Fri Sat Sun

One additional comment - in case it helps... You'll notice that for each row in my table the names of the fields are identical except that the number at the end is incremented. So for example:

  name="dow_w"
  name="dow_w2"

these rows are generated dynamically by jquery... but i don't think it's relevant so I haven't included the code. The html code you see above is the RENDERED html... so it's pretty accurate.

Any help figuring out where my bug is would be appreciated.

jquery validation - is there a more efficient way to do this?

Answers (1)

Related Questions