Use Discourse.getURL for /clicks/track so clicks can be tracked on re…

…lative URLs (#8079)
discourse · Sep 10, 2019 · 21c11c2 · 21c11c2
1 parent 39a427e
commit 21c11c2
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/app/assets/javascripts/discourse/lib/ajax.js.es6 b/app/assets/javascripts/discourse/lib/ajax.js.es6
@@ -153,7 +153,7 @@ export function ajax() {
   if (
     args.type &&
     args.type.toUpperCase() !== "GET" &&
-    url !== "/clicks/track" &&
+    url !== Discourse.getURL("/clicks/track") &&
     !Discourse.Session.currentProp("csrfToken")
   ) {
     promise = new Ember.RSVP.Promise((resolve, reject) => {

diff --git a/app/assets/javascripts/discourse/lib/click-track.js.es6 b/app/assets/javascripts/discourse/lib/click-track.js.es6
@@ -102,9 +102,9 @@ export default {
         data.append("url", href);
         data.append("post_id", postId);
         data.append("topic_id", topicId);
-        navigator.sendBeacon("/clicks/track", data);
+        navigator.sendBeacon(Discourse.getURL("/clicks/track"), data);
       } else {
-        trackPromise = ajax("/clicks/track", {
+        trackPromise = ajax(Discourse.getURL("/clicks/track"), {
           type: "POST",
           data: {
             url: href,