fix(calendar.js): do not alter original execution context angular-ui#107

angular-ui#87 angular-ui#105 Signed-off-by: joshkurz <[email protected]>
joshkurz · Jul 2, 2014 · 2d2406e · 2d2406e
1 parent d6550d1
commit 2d2406e
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 4 deletions.
diff --git a/src/calendar.js b/src/calendar.js
@@ -26,8 +26,9 @@ angular.module('ui.calendar', [])
                       // In this way the function will be safely executed on the next digest.
 
                       var args = arguments;
+                      var _this = this;
                       $timeout(function(){
-                          functionToWrap.apply(this, args);
+                        functionToWrap.apply(_this, args);
                       });
                   };
               }

diff --git a/test/calendar.spec.js b/test/calendar.spec.js
@@ -375,9 +375,7 @@ describe('uiCalendar', function () {
               var fullCalendarConfig = calendarCtrl.getFullCalendarConfig(scope.uiConfig.calendar, {});
 
               fullCalendarConfig[key]();
-
               $timeout.flush();
-
               expect($rootScope.$apply.callCount).toBe(functionCount);
               expect(scope.uiConfig.calendar[key]).toHaveBeenCalled();
               $rootScope.$apply.isSpy = false;
@@ -414,7 +412,6 @@ describe('uiCalendar', function () {
 
               scope.$apply();
               $timeout.flush();
-
               expect($rootScope.$apply.callCount).toBe(functionCount*2);
               expect(scope.uiConfig.calendar[key]).toHaveBeenCalled();
             }