From 3be0294465ae32528ba2b854bdb7ad30ce804a7e Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?R=C3=A9gis=20Hanol?= <regis@hanol.fr>
Date: Mon, 26 Feb 2018 16:05:35 +0100
Subject: [PATCH] FIX: local post onebox was always pointing to 1st post

---
 lib/onebox/templates/discourse_topic_onebox.hbs | 2 +-
 lib/oneboxer.rb                                 | 1 +
 spec/components/oneboxer_spec.rb                | 1 +
 3 files changed, 3 insertions(+), 1 deletion(-)

diff --git a/lib/onebox/templates/discourse_topic_onebox.hbs b/lib/onebox/templates/discourse_topic_onebox.hbs
index 8b6d746b71f..856d3342d47 100644
--- a/lib/onebox/templates/discourse_topic_onebox.hbs
+++ b/lib/onebox/templates/discourse_topic_onebox.hbs
@@ -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='quote-controls'></div>
     {{{avatar}}}
diff --git a/lib/oneboxer.rb b/lib/oneboxer.rb
index 682251ed505..935233ead73 100644
--- a/lib/oneboxer.rb
+++ b/lib/oneboxer.rb
@@ -202,6 +202,7 @@ module Oneboxer
       else
         args = {
           topic_id: topic.id,
+          post_number: post.post_number,
           avatar: PrettyText.avatar_img(post.user.avatar_template, "tiny"),
           original_url: url,
           title: PrettyText.unescape_emoji(CGI::escapeHTML(topic.title)),
diff --git a/spec/components/oneboxer_spec.rb b/spec/components/oneboxer_spec.rb
index 1bbc1e76b50..6ba23c3a950 100644
--- a/spec/components/oneboxer_spec.rb
+++ b/spec/components/oneboxer_spec.rb
@@ -54,6 +54,7 @@ describe Oneboxer do
 
       onebox = preview(public_reply.url, user, public_category)
       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"))
 
       onebox = preview(public_reply.url, user, public_category, public_topic)