github-mirror/discourse
2
0
Fork 0
mirror of https://github.com/discourse/discourse.git synced 2024-11-25 18:23:43 +08:00
Code Issues Actions 6 Packages Projects Releases Wiki Activity

FIX: local post onebox was always pointing to 1st post

Browse Source
This commit is contained in:
Régis Hanol 2018-02-26 16:05:35 +01:00
parent ac701696b3
commit 3be0294465
3 changed files with 3 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
lib/onebox/templates/discourse_topic_onebox.hbs
View File

@ -1,4 +1,4 @@
<aside class='quote' data-post="1" data-topic="{{topic_id}}"> <aside class='quote' data-post="{{post_number}}" data-topic="{{topic_id}}">
<div class='title'> <div class='title'>
<div class='quote-controls'></div> <div class='quote-controls'></div>
{{{avatar}}} {{{avatar}}}

1
lib/oneboxer.rb
View File

@ -202,6 +202,7 @@ module Oneboxer
else else
args = { args = {
topic_id: topic.id, topic_id: topic.id,
post_number: post.post_number,
avatar: PrettyText.avatar_img(post.user.avatar_template, "tiny"), avatar: PrettyText.avatar_img(post.user.avatar_template, "tiny"),
original_url: url, original_url: url,
title: PrettyText.unescape_emoji(CGI::escapeHTML(topic.title)), title: PrettyText.unescape_emoji(CGI::escapeHTML(topic.title)),

1
spec/components/oneboxer_spec.rb
View File

@ -54,6 +54,7 @@ describe Oneboxer do
onebox = preview(public_reply.url, user, public_category) onebox = preview(public_reply.url, user, public_category)
expect(onebox).to include(public_reply.excerpt) expect(onebox).to include(public_reply.excerpt)
expect(onebox).to include(%{data-post="2"})
expect(onebox).to include(PrettyText.avatar_img(replier.avatar_template, "tiny")) expect(onebox).to include(PrettyText.avatar_img(replier.avatar_template, "tiny"))
onebox = preview(public_reply.url, user, public_category, public_topic) onebox = preview(public_reply.url, user, public_category, public_topic)