fix(accordion): coerce to boolean

inukshuk · jasonaden · commit e73da732226b · 2015-09-29T20:30:40.000-07:00
- Coerce `toggleClass` argument to boolean due to jQuery ignoring `undefined` Closes angular-ui#4385
diff --git a/src/accordion/accordion.js b/src/accordion/accordion.js
@@ -82,7 +82,7 @@ angular.module('ui.bootstrap.accordion', ['ui.bootstrap.collapse'])
       scope.openClass = attrs.openClass || 'panel-open';
       scope.panelClass = attrs.panelClass;
       scope.$watch('isOpen', function(value) {
-        element.toggleClass(scope.openClass, value);
+        element.toggleClass(scope.openClass, Boolean(value));
         if (value) {
           accordionCtrl.closeOthers(scope);
         }

Original file line number	Diff line number	Diff line change
`@@ -82,7 +82,7 @@ angular.module('ui.bootstrap.accordion', ['ui.bootstrap.collapse'])`
`82`	`82`	`scope.openClass = attrs.openClass \|\| 'panel-open';`
`83`	`83`	`scope.panelClass = attrs.panelClass;`
`84`	`84`	`scope.$watch('isOpen', function(value) {`
`85`		`- element.toggleClass(scope.openClass, value);`
	`85`	`+ element.toggleClass(scope.openClass, Boolean(value));`
`86`	`86`	`if (value) {`
`87`	`87`	`accordionCtrl.closeOthers(scope);`
`88`	`88`	`}`