0
votes

I have a ul with a total of 13 list items, but for some reason 12 of them gets checked when I run my function. I do not see the flaw in my code that automatically check them. I need help.

HTML:

<!--THIRD SPAN::mcs-->   
        <span>
            <ul>
                <li>
                    <input type="checkbox" name="0" id="mcs0" class="cl">
                    <label>0</label>
                </li>
                <li>
                    <input type="checkbox" name="1" id="mcs1" class="cl">
                    <label>1</label>
                </li>
                <li>
                    <input type="checkbox" name="2" id="mcs2" class="cl">
                    <label>2</label>
                </li>
                <li>
                    <input type="checkbox" name="3" id="mcs3" class="cl">
                    <label>3</label>
                </li>
                <li>
                    <input type="checkbox" name="4" id="mcs4" class="cl">
                    <label>4</label>
                </li>
                <li>
                    <input type="checkbox" name="5" id="mcs5" class="cl">
                    <label>5</label>
                </li>
                <li>
                    <input type="checkbox" name="6" id="mcs6" class="cl">
                    <label>6</label>
                </li>
                <li>
                    <input type="checkbox" name="7" id="mcs7" class="cl">
                    <label>7</label>
                </li>
                <li>
                    <input type="checkbox" name="8" id="mcs8" class="cl">
                    <label>8</label>
                </li>
                <li>
                    <input type="checkbox" name="9" id="mcs9" class="cl">
                    <label>9</label>
                </li>
                <li>
                    <input type="checkbox" name="10" id="mcs10" class="cl">
                    <label>10</label>
                </li>
                <li>
                    <input type="checkbox" name="11" id="mcs11" class="cl">
                    <label>11</label>
                </li>
                <li>
                    <input type="checkbox" name="12" id="mcs12" class="cl">
                    <label>12</label>
                </li>
            </ul>
        </span>

JavaScript:

  var mcs0 = document.getElementById("mcs0");
  var mcs1 = document.getElementById("mcs1");
  var mcs2 = document.getElementById("mcs2");
  var mcs3 = document.getElementById("mcs3");
  var mcs4 = document.getElementById("mcs4");
  var mcs5 = document.getElementById("mcs5");
  var mcs6 = document.getElementById("mcs6");
  var mcs7 = document.getElementById("mcs7");
  var mcs8 = document.getElementById("mcs8");
  var mcs9 = document.getElementById("mcs9");
  var mcs10 = document.getElementById("mcs10");
  var mcs11 = document.getElementById("mcs11");
  var mcs12 = document.getElementById("mcs12");


  if (mcs0.checked == true){
mcsArray.push(0)
}   if(mcs1.checked = true){
  mcsArray.push(1)
}   if(mcs2.checked = true){
  mcsArray.push(2)
}   if(mcs3.checked = true){
  mcsArray.push(3)
}   if(mcs4.checked = true){
  mcsArray.push(4)
}   if(mcs5.checked = true){
  mcsArray.push(5)
}   if(mcs6.checked = true){
  mcsArray.push(6)
}   if(mcs7.checked = true){
  mcsArray.push(7)
}   if(mcs8.checked = true){
  mcsArray.push(8)
}   if(mcs9.checked = true){
  mcsArray.push(9)
}   if(mcs10.checked = true){
  mcsArray.push(10)
}   if(mcs11.checked = true){
  mcsArray.push(11)
}   if(mcs12.checked = true){
  mcsArray.push(12)
}

I do have a similar ul that is used for the same thing, but does not have that problem. If you need more code to understand, just ask. Thanks

mcs1.checked = true this is assignment not comparsion - Chris Li
replace mcsXXX.checked = true with mcsXXX.checked == true like you have with the first one ... - derpirscher