From 9ff0e22acf9a4aff5b78d03f348db7c8df272b8b Mon Sep 17 00:00:00 2001
From: Julie <ju.ralph@gmail.com>
Date: Thu, 11 Apr 2013 13:11:17 -0700
Subject: [PATCH] fix(a): allow jqLite triggerHandler('click') on anchors

Previously, anchor elements could not be used with triggerHandler because
triggerHandler passes null as the event, and any anchor element with an empty
href automatically calls event.preventDefault(). Add a check that event
exists before calling preventDefault.
---
 src/ng/directive/a.js      | 2 +-
 test/ng/directive/aSpec.js | 8 ++++++++
 2 files changed, 9 insertions(+), 1 deletion(-)

diff --git a/src/ng/directive/a.js b/src/ng/directive/a.js
index 7ee8f572ab64..bb96b6b477ad 100644
--- a/src/ng/directive/a.js
+++ b/src/ng/directive/a.js
@@ -36,7 +36,7 @@ var htmlAnchorDirective = valueFn({
       element.bind('click', function(event){
         // if we have no href url, then don't navigate anywhere.
         if (!element.attr('href')) {
-          event.preventDefault();
+          event && event.preventDefault();
         }
       });
     }
diff --git a/test/ng/directive/aSpec.js b/test/ng/directive/aSpec.js
index a284f4bc8199..bd39ab60c549 100644
--- a/test/ng/directive/aSpec.js
+++ b/test/ng/directive/aSpec.js
@@ -58,4 +58,12 @@ describe('a', function() {
 
     expect(element.text()).toBe('hello@you');
   });
+
+  it('should not cause failures with triggerHandler', function() {
+    element = jqLite('<a href="" ng-click="foo=\'bar\'">link</a>"');
+    element = $compile(element)($rootScope);
+
+    element.triggerHandler('click');
+    expect($rootScope.foo).toEqual('bar');
+  });
 });