From 9a86044f90399c714867c4155bafd18e57199da3 Mon Sep 17 00:00:00 2001
From: Peter Lada <peter@guidebook.com>
Date: Wed, 28 Jan 2015 18:27:16 -0800
Subject: [PATCH] changed existing refresh method to reload. changed the data
 check to trigger a refresh on the element based on the already bound data
 attr

---
 lib/proto/attach-link.js       |  6 ++----
 lib/proto/parse-element.js     |  2 +-
 lib/{refresh.js => reload.js}  |  0
 tests/index.html               | 10 +++++-----
 tests/lib/proto/attach-link.js |  6 +++---
 5 files changed, 11 insertions(+), 13 deletions(-)
 rename lib/{refresh.js => reload.js} (100%)

diff --git a/lib/proto/attach-link.js b/lib/proto/attach-link.js
index f0ad789..d53e351 100644
--- a/lib/proto/attach-link.js
+++ b/lib/proto/attach-link.js
@@ -44,8 +44,8 @@ var linkAction = function(el, event) {
 
   // don’t do "nothing" if user try to reload the page by clicking the same link twice
   if (el.href === window.location.href.split("#")[0]) {
-    el.setAttribute(attrClick, "refresh")
-    this.refresh()
+    el.setAttribute(attrClick, "reload")
+    this.reload()
     return
   }
 
@@ -56,8 +56,6 @@ var linkAction = function(el, event) {
 module.exports = function(el) {
   var that = this
 
-  el.setAttribute("data-pjax-enabled", "true");
-
   on(el, "click", function(event) {
     linkAction.call(that, el, event)
   })
diff --git a/lib/proto/parse-element.js b/lib/proto/parse-element.js
index 4f7a8cd..736fdd7 100644
--- a/lib/proto/parse-element.js
+++ b/lib/proto/parse-element.js
@@ -2,7 +2,7 @@ module.exports = function(el) {
   switch (el.tagName.toLowerCase()) {
   case "a":
     // only attach link if el does not already have link attached
-    if (!el.getAttribute("data-pjax-enabled")) {
+    if (!el.hasAttribute('data-pjax-click-state')) {
       this.attachLink(el)
     }
     break
diff --git a/lib/refresh.js b/lib/reload.js
similarity index 100%
rename from lib/refresh.js
rename to lib/reload.js
diff --git a/tests/index.html b/tests/index.html
index 4c8c6b6..a9ef972 100644
--- a/tests/index.html
+++ b/tests/index.html
@@ -12091,8 +12091,8 @@ var linkAction = function(el, event) {
 
   // don’t do "nothing" if user try to reload the page by clicking the same link twice
   if (el.href === window.location.href.split("#")[0]) {
-    el.setAttribute(attrClick, "refresh")
-    this.refresh()
+    el.setAttribute(attrClick, "reload")
+    this.reload()
     return
   }
 
@@ -12409,8 +12409,8 @@ tape("test attach link prototype method", function(t) {
 
   attachLink.call({
     options: {},
-    refresh: function() {
-      t.equal(a.getAttribute(attr), "refresh", "triggering exact same url refresh the page")
+    reload: function() {
+      t.equal(a.getAttribute(attr), "reload", "triggering exact same url reload the page")
     },
     loadUrl: function() {
       t.equal(a.getAttribute(attr), "load", "triggering a internal link actually load the page")
@@ -12444,7 +12444,7 @@ tape("test attach link prototype method", function(t) {
 
   a.href = internalUri
   trigger(a, "click")
-  // see refresh defined above
+  // see reload defined above
 
   a.href = window.location.protocol + "//" + window.location.host + "/internal"
   trigger(a, "click")
diff --git a/tests/lib/proto/attach-link.js b/tests/lib/proto/attach-link.js
index aa66a88..29202c1 100644
--- a/tests/lib/proto/attach-link.js
+++ b/tests/lib/proto/attach-link.js
@@ -13,8 +13,8 @@ tape("test attach link prototype method", function(t) {
 
   attachLink.call({
     options: {},
-    refresh: function() {
-      t.equal(a.getAttribute(attr), "refresh", "triggering exact same url refresh the page")
+    reload: function() {
+      t.equal(a.getAttribute(attr), "reload", "triggering exact same url reload the page")
     },
     loadUrl: function() {
       t.equal(a.getAttribute(attr), "load", "triggering a internal link actually load the page")
@@ -48,7 +48,7 @@ tape("test attach link prototype method", function(t) {
 
   a.href = internalUri
   trigger(a, "click")
-  // see refresh defined above
+  // see reload defined above
 
   a.href = window.location.protocol + "//" + window.location.host + "/internal"
   trigger(a, "click")