Problem/Motivation

Found a line of js that wasn't used.


                  var checkval = function (inval) {
                    // Do loose comparisons to support things like "true", "1",
                    // etc.
                    if (dependency.attr('type') === 'radio') {
                      var val = $('[name="' + dependency.attr('name') + '"]:checked').val();
                      return val == inval;
                    }
                    else if (dependency.attr('type') === 'checkbox') {
                      return dependency.is(':checked') && inval == dependency.val();
                    }
                    else {
                      return dependency.val() == inval;
                    }
                    return false; //LINE REMOVED
                  };
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

marathonerdj created an issue. See original summary.

Jyaai’s picture

Status: Active » Needs review
FileSize
432 bytes
DamienMcKenna’s picture

That seems reasonable.

DamienMcKenna’s picture

Status: Needs review » Fixed

Committed.

  • DamienMcKenna committed 8689bd8 on 7.x-3.x authored by Jyaai
    Issue #3201469 by Jyaai, DamienMcKenna: Remove unreachable JS code.
    

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.