fix(carousel): cancel timer on scope destruction

Closes #1414

Author: chrisirhc

src/carousel/carousel.js

@@ -106,6 +106,7 @@ angular.module('ui.bootstrap.carousel', ['ui.bootstrap.transition'])
   function restartTimer() {
     if (currentTimeout) {
       $timeout.cancel(currentTimeout);
+      currentTimeout = null;
     }
     function go() {
       if (isPlaying) {
@@ -131,6 +132,7 @@ angular.module('ui.bootstrap.carousel', ['ui.bootstrap.transition'])
       isPlaying = false;
       if (currentTimeout) {
         $timeout.cancel(currentTimeout);
+        currentTimeout = null;
       }
     }
   };
@@ -163,6 +165,13 @@ angular.module('ui.bootstrap.carousel', ['ui.bootstrap.transition'])
       currentIndex--;
     }
   };
+
+  $scope.$on('$destroy', function () {
+    if (currentTimeout) {
+      $timeout.cancel(currentTimeout);
+      currentTimeout = null;
+    }
+  });
 }])
 
 /**

src/carousel/test/carousel.spec.js

@@ -294,6 +294,20 @@ describe('carousel', function() {
         expect(ctrl.slides.length).toBe(1);
         expect(ctrl.currentSlide).toBe(ctrl.slides[0]);
       });
+
+      it('issue 1414 - should not continue running timers after scope is destroyed', function() {
+        spyOn(scope, 'next').andCallThrough();
+        scope.interval = 2000;
+        scope.$digest();
+
+        $timeout.flush();
+        expect(scope.next.calls.length).toBe(1);
+
+        scope.$destroy();
+
+        $timeout.flush(scope.interval);
+        expect(scope.next.calls.length).toBe(1);
+      });
     });
   });
 });