[Carousel] TypeError: Cannot read property 'index' of undefined

[sylvainbx]

In carousel.js#L76, if slides.length == 0, newIndex will be NaN.
This will cause function getSlideByIndex(index) to throw an error on L52 :

TypeError: Cannot read property 'index' of undefined
    at getSlideByIndex (ui-bootstrap-tpls.js?body=1:363)
    at Scope.$scope.next (ui-bootstrap-tpls.js?body=1:391)
    at timerFn (ui-bootstrap-tpls.js?body=1:429)
    at angular.js?body=1:13331
    at Scope.$eval (angular.js?body=1:14467)
    at Scope.$digest (angular.js?body=1:14283)
    at Scope.$apply (angular.js?body=1:14572)
    at tick (angular.js?body=1:10429)

[rvanbaalen]

Plunkr please.

[wesleycho]

Here is a reproduction in Plunker.

[wesleycho]

What is happening is that the carousel uses $interval to time sliding to the next slide, but if the slides disappear in the interim, then the callback fired ends up with a false assumption about the existence of an index in the slides array.

[Gericho91]

currentIndex = slides.length == currentIndex ? 0 : currentIndex;
add to the 566 line in ui-bootstrap-tpls.js

function goNext(slide, index, direction) {
if (destroyed) {
return;
}
currentIndex = slides.length == currentIndex ? 0 : currentIndex;
angular.extend(slide, {direction: direction});
angular.extend(slides[currentIndex].slide || {}, {direction: direction});