fix(jqLite): addClass should ignore falsy values

This commit is contained in:
Igor Minar 2011-09-01 15:13:42 -07:00
parent db78aa1ce1
commit 622c3ec974
2 changed files with 16 additions and 2 deletions

View file

@ -162,8 +162,8 @@ function JQLiteRemoveClass(element, selector) {
);
}
function JQLiteAddClass(element, selector ) {
if (!JQLiteHasClass(element, selector)) {
function JQLiteAddClass(element, selector) {
if (selector && !JQLiteHasClass(element, selector)) {
element.className = trim(element.className + ' ' + selector);
}
}

View file

@ -197,6 +197,20 @@ describe('jqLite', function(){
expect(jqLite(a).hasClass('abc')).toEqual(true);
expect(jqLite(b).hasClass('abc')).toEqual(true);
});
it('should ignore falsy values', function() {
var jqA = jqLite(a);
expect(jqA[0].className).toBe('');
jqA.addClass(undefined);
expect(jqA[0].className).toBe('');
jqA.addClass(null);
expect(jqA[0].className).toBe('');
jqA.addClass(false);
expect(jqA[0].className).toBe('');
});
});