Add DiscourseLocalOnebox

This commit is contained in:
Neil Lalonde 2014-01-28 13:18:19 -05:00
parent 30abc6b291
commit d343e9f360
4 changed files with 177 additions and 3 deletions

View File

@ -0,0 +1,84 @@
module Onebox
module Engine
class DiscourseLocalOnebox
include Engine
matches_regexp Regexp.new("^#{Discourse.base_url.gsub(".","\\.")}.*$", true)
def to_html
uri = URI::parse(@url)
route = Rails.application.routes.recognize_path(uri.path)
args = {original_url: @url}
# Figure out what kind of onebox to show based on the URL
case route[:controller]
when 'topics'
linked = "<a href='#{@url}'>#{@url}</a>"
if route[:post_number].present? && route[:post_number].to_i > 1
# Post Link
post = Post.where(topic_id: route[:topic_id], post_number: route[:post_number].to_i).first
return linked unless post
return linked unless Guardian.new.can_see?(post)
topic = post.topic
slug = Slug.for(topic.title)
excerpt = post.excerpt(SiteSetting.post_onebox_maxlength)
excerpt.gsub!("\n"," ")
# hack to make it render for now
excerpt.gsub!("[/quote]", "[quote]")
quote = "[quote=\"#{post.user.username}, topic:#{topic.id}, slug:#{slug}, post:#{post.post_number}\"]#{excerpt}[/quote]"
cooked = PrettyText.cook(quote)
return cooked
else
# Topic Link
topic = Topic.where(id: route[:topic_id].to_i).includes(:user).first
return linked unless topic
return linked unless Guardian.new.can_see?(topic)
post = topic.posts.first
posters = topic.posters_summary.map do |p|
{
username: p[:user].username,
avatar: PrettyText.avatar_img(p[:user].avatar_template, 'tiny'),
description: p[:description],
extras: p[:extras]
}
end
category = topic.category
if category
category = "<a href=\"/category/#{category.slug}\" class=\"badge badge-category\" style=\"background-color: ##{category.color}; color: ##{category.text_color}\">#{category.name}</a>"
end
quote = post.excerpt(SiteSetting.post_onebox_maxlength)
args.merge! title: topic.title,
avatar: PrettyText.avatar_img(topic.user.avatar_template, 'tiny'),
posts_count: topic.posts_count,
last_post: FreedomPatches::Rails4.time_ago_in_words(topic.last_posted_at, false, scope: :'datetime.distance_in_words_verbose'),
age: FreedomPatches::Rails4.time_ago_in_words(topic.created_at, false, scope: :'datetime.distance_in_words_verbose'),
views: topic.views,
posters: posters,
quote: quote,
category: category,
topic: topic.id
@template = 'topic'
end
end
return nil unless @template
Mustache.render(File.read("#{Rails.root}/lib/onebox/templates/discourse_#{@template}_onebox.handlebars"), args)
rescue ActionController::RoutingError
nil
end
end
end
end

View File

@ -0,0 +1,20 @@
<aside class='quote' data-post="1" data-topic="{{topic}}">
<div class='title'>
<div class='quote-controls'></div>
{{{avatar}}}
<a href="{{original_url}}">{{title}}</a> {{{category}}}
</div>
<blockquote>{{{quote}}}
<div class='topic-info'>
<div class='info-line'>
{{posts_count}} posts, last post {{last_post}}, created {{age}}, {{views}} views
</div>
<div class='posters'>
{{#posters}}
<span title="{{{username}}}">{{{avatar}}}</span>
{{/posters}}
</div>
<div class='clearfix'></div>
</div>
</blockquote>
</aside>

View File

@ -1,3 +1,8 @@
Dir["#{Rails.root}/lib/onebox/engine/*_onebox.rb"].each {|f|
require_dependency(f.split('/')[-3..-1].join('/'))
}
module Oneboxer

View File

@ -0,0 +1,65 @@
require 'spec_helper'
describe Onebox::Engine::DiscourseLocalOnebox do
it "matches for a topic url" do
url = "#{Discourse.base_url}/t/hot-topic"
Onebox.has_matcher?(url).should == true
Onebox::Matcher.new(url).oneboxed.should == described_class
end
it "matches for a post url" do
url = "#{Discourse.base_url}/t/hot-topic/23/2"
Onebox.has_matcher?(url).should == true
Onebox::Matcher.new(url).oneboxed.should == described_class
end
context "for a link to a post" do
let(:post) { Fabricate(:post) }
let(:post2) { Fabricate(:post, topic: post.topic, post_number: 2) }
it "returns a link if post isn't found" do
url = "#{Discourse.base_url}/t/not-exist/3/2"
Onebox.preview(url).to_s.should == "<a href='#{url}'>#{url}</a>"
end
it "returns a link if not allowed to see the post" do
url = "#{Discourse.base_url}#{post2.url}"
Guardian.any_instance.stubs(:can_see?).returns(false)
Onebox.preview(url).to_s.should == "<a href='#{url}'>#{url}</a>"
end
it "returns some onebox goodness if post exists and can be seen" do
url = "#{Discourse.base_url}#{post2.url}"
Guardian.any_instance.stubs(:can_see?).returns(true)
html = Onebox.preview(url).to_s
html.should include(post2.user.username)
html.should include(post2.excerpt)
end
end
context "for a link to a topic" do
let(:post) { Fabricate(:post) }
let(:topic) { post.topic }
before { topic.last_posted_at = Time.zone.now; topic.save; } # otherwise errors
it "returns a link if topic isn't found" do
url = "#{Discourse.base_url}/t/not-found/123"
Onebox.preview(url).to_s.should == "<a href='#{url}'>#{url}</a>"
end
it "returns a link if not allowed to see the post" do
url = "#{topic.url}"
Guardian.any_instance.stubs(:can_see?).returns(false)
Onebox.preview(url).to_s.should == "<a href='#{url}'>#{url}</a>"
end
it "returns some onebox goodness if post exists and can be seen" do
url = "#{topic.url}"
Guardian.any_instance.stubs(:can_see?).returns(true)
html = Onebox.preview(url).to_s
html.should include(topic.posts.first.user.username)
html.should include("topic-info")
end
end
end