Discource-C
/
discourse
mirror of https://github.com/discourse/discourse.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

FIX: Allow link redirects inside oneboxed posts

This commit is contained in:
Robin Ward 2013-11-29 14:29:28 -05:00
parent 298b54c0e7
commit 769223014c
2 changed files with 7 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
app/assets/javascripts/discourse/lib/click_track.js
View File

@ -38,10 +38,9 @@ Discourse.ClickTrack = {
userId = $link.data('user-id');
if (!userId) userId = $article.data('user-id');
var ownLink = userId && (userId === Discourse.User.currentProp('id'));
// Build a Redirect URL
var trackingUrl = Discourse.getURL("/clicks/track?url=" + encodeURIComponent(href));
var ownLink = userId && (userId === Discourse.User.currentProp('id')),
trackingUrl = Discourse.getURL("/clicks/track?url=" + encodeURIComponent(href));
if (postId && (!$link.data('ignore-post-id'))) {
trackingUrl += "&post_id=" + encodeURI(postId);
}

6
app/models/topic_link_click.rb
View File

@ -22,7 +22,11 @@ class TopicLinkClick < ActiveRecord::Base
# If no link is found, return the url for relative links
unless link.present?
return args[:url] if args[:url] =~ /^\//
return nil
# If we have it somewhere else on the site, just allow the redirect. This is
# likely due to a onebox of another topic.
link = TopicLink.where(url: args[:url]).first
return link.present? ? link.url : nil
end
return args[:url] if (args[:user_id] && (link.user_id == args[:user_id]))