From b2118dd440285d2b04fe81621cae6216a4b4c725 Mon Sep 17 00:00:00 2001
From: Billy Monk <billy.monk@gmail.com>
Date: Mon, 26 May 2014 21:00:00 -0400
Subject: [PATCH] Use correct 'this' when setting data-interchange-last-path.

Because the 'this' being used was for the wrong scope
data-interchange-last-path is not set. This can lead to making multiple
requests for the same content unnecessarily.
---
 js/foundation/foundation.interchange.js |  5 +++--
 spec/interchange/interchange.js         | 13 +++++++++++++
 2 files changed, 16 insertions(+), 2 deletions(-)

diff --git a/js/foundation/foundation.interchange.js b/js/foundation/foundation.interchange.js
index c9fb7f59fb..7e99dd6d90 100644
--- a/js/foundation/foundation.interchange.js
+++ b/js/foundation/foundation.interchange.js
@@ -53,7 +53,8 @@
 
             return trigger(el[0].src);
           }
-          var last_path = el.data(this.data_attr + '-last-path');
+          var last_path = el.data(this.data_attr + '-last-path'),
+              self = this;
 
           if (last_path == path) return;
 
@@ -65,7 +66,7 @@
 
           return $.get(path, function (response) {
             el.html(response);
-            el.data(this.data_attr + '-last-path', path);
+            el.data(self.data_attr + '-last-path', path);
             trigger();
           });
 
diff --git a/spec/interchange/interchange.js b/spec/interchange/interchange.js
index 97bca03bd6..be67bdc621 100644
--- a/spec/interchange/interchange.js
+++ b/spec/interchange/interchange.js
@@ -57,4 +57,17 @@ describe('interchange:', function() {
         });
       });
   }));
+
+  describe('setting data-interchange-last-path', function() {
+    beforeEach(function() {
+      document.body.innerHTML = __html__['spec/interchange/basic.html'];
+    });
+
+    it('should set data-interchange-last-path on element when replace occurs', function() {
+      Foundation.libs.interchange.update_nodes();
+      Foundation.libs.interchange.resize();
+
+      expect($('div[data-interchange]').data('data-interchange-last-path')).toMatch(/.+html$/)
+    });
+  });
 });