fix(dropdown): add safety check for setIsOpen

palaslet · wesleycho · commit 60e431608fb0 · 2015-07-28T07:28:54.000-07:00
- Ensure `setIsOpen` is a function, due to it optionally being falsy Closes #4030
diff --git a/src/dropdown/dropdown.js b/src/dropdown/dropdown.js
@@ -220,7 +220,9 @@ angular.module('ui.bootstrap.dropdown', ['ui.bootstrap.position'])
       self.selectedOption = null;
     }
 
-    setIsOpen($scope, isOpen);
+    if (angular.isFunction(setIsOpen)) {
+      setIsOpen($scope, isOpen);
+    }
   });
 
   $scope.$on('$locationChangeSuccess', function() {

Original file line number	Diff line number	Diff line change
`@@ -220,7 +220,9 @@ angular.module('ui.bootstrap.dropdown', ['ui.bootstrap.position'])`
`220`	`220`	`self.selectedOption = null;`
`221`	`221`	`}`
`222`	`222`
`223`		`- setIsOpen($scope, isOpen);`
	`223`	`+ if (angular.isFunction(setIsOpen)) {`
	`224`	`+ setIsOpen($scope, isOpen);`
	`225`	`+ }`
`224`	`226`	`});`
`225`	`227`
`226`	`228`	`$scope.$on('$locationChangeSuccess', function() {`