Reply To: Broken jQuery code, need help.

[TheDutchCoder]

My best guess is he wants to trigger tn1 and tn2 if they’re supplied, but if tn2 isn’t supplied he wants to trigger tn1 and tn3 instead.

Now that’s perfectly fine, but you need a lot more to make this even remotely safe ;-)

try this:

$.fn.textnet = function(tn, tn1, tn2, tn3) {
  if (tn === 'toggleAttr') {
    if ((tn1 && tn1 !== 'undefined') && (tn2 && tn2 !== 'undefined')) {
      $(this).attr(tn1, tn2);
    } else if ((tn1 && tn1 !== 'undefined') && (tn3 && tn3 !== 'undefined')) {
      $(this).attr(tn1, tn3);
    } else {
      // None of the conditions are met.
      // E.g. tn2 and tn3 are missing.
    }
  }
}

Remember you HAVE to supply tn2, even if you only want to trigger tn3, like so: $('.something').textnet('toggleAttr', 'href', null, '#foobar');