From 1147f21999edf9a434cd8d24865a6455e744d858 Mon Sep 17 00:00:00 2001 From: Michał Gołębiowski Date: Thu, 5 Dec 2013 14:59:13 +0100 Subject: fix(input): prevent double $digest when using jQuery trigger. If an event was performed natively, jQuery sets the isTrigger property. When triggering event manually, the field is not present. Manually triggered events are performed synchronously which causes the "$digest already in progress" error. Closes #5293 --- test/ng/directive/inputSpec.js | 17 +++++++++++++++++ 1 file changed, 17 insertions(+) (limited to 'test') diff --git a/test/ng/directive/inputSpec.js b/test/ng/directive/inputSpec.js index 48319cfa..c9a345cd 100644 --- a/test/ng/directive/inputSpec.js +++ b/test/ng/directive/inputSpec.js @@ -533,6 +533,23 @@ describe('input', function() { 'event so that form auto complete works',function() { assertBrowserSupportsChangeEvent(true); }); + + if (!_jqLiteMode) { + it('should not cause the double $digest when triggering an event using jQuery', function() { + $sniffer.hasEvent = function(eventName) { + return eventName !== 'input'; + }; + + compileInput(''); + + scope.field = 'fake field'; + scope.$watch('field', function() { + // We need to use _originalTrigger since trigger is modified by Angular Scenario. + inputElm._originalTrigger('change'); + }); + scope.$apply(); + }); + } }); describe('"paste" and "cut" events', function() { -- cgit v1.2.3