merge

EvandroLG · Sep 13, 2016 · 173e846 · 173e846
2 parents 3cb7d4c + 59a13fa
commit 173e846
Show file tree

Hide file tree

Showing 4 changed files with 11 additions and 1 deletion.
diff --git a/dist/page-accelerator.min.js b/dist/page-accelerator.min.js
diff --git a/src/page-accelerator.js b/src/page-accelerator.js
@@ -110,6 +110,11 @@
       var links = doc.querySelectorAll('a:not([data-pageAccelerator="false"])');
 
       [].forEach.call(links, function(element) {
+        if (element.hostname !== window.location.hostname ||
+            element.protocol !== window.location.protocol) {
+          return;
+        }
+
         element.addEventListener('click', function(e) {
           e.preventDefault();
           that._onClick.call(that, this);

diff --git a/test/SpecRunner.html b/test/SpecRunner.html
@@ -10,6 +10,7 @@
     <div id="container-test" style="display: none;">
       <a href="#" id="link_1">mock 1</a>
       <a href="#" id="link_2" data-pageAccelerator="false">mock 2</a>
+      <a href="http://github.com" id="link_3">mock 3</a>
     </div>
     <script src="../src/ajax.js"></script>
     <script src="../src/page-accelerator.js"></script>

diff --git a/test/spec.js b/test/spec.js
@@ -67,6 +67,10 @@
       it('should not call ajax method when user requests a new page by a link with data-pageAccelerator="false"', function() {
         verifyAjaxWasCalled(_find('#link_2'), 'false');
       });
+
+      it('should not call ajax method when user requests a new page on an external domain', function() {
+        verifyAjaxWasCalled(_find('#link_3'), 'false');
+      });
     });
 
   });