mastofe

commit: 35aafdba961c90ecad04f38d4eec69100da36697
parent: 7e00a21ea6f76f8167f1070fbfe231bbd65c6cad
Author: Eugen Rochko <eugen@zeonfederated.com>
Date:   Mon, 21 Mar 2016 11:43:21 +0100

Ancestors and descendants of statuses

Diffstat:
M
app/models/status.rb
8
++++++++
M
app/views/stream_entries/_status.html.haml
5
+++--
M
spec/controllers/api/subscriptions_controller_spec.rb
2
+-

3 files changed, 12 insertions(+), 3 deletions(-)
diff --git a/app/models/status.rb b/app/models/status.rb
@@ -76,6 +76,14 @@ class Status < ActiveRecord::Base
     m
   end
 
+  def ancestors
+    Status.where(id: Status.find_by_sql(['WITH RECURSIVE search_tree(id, in_reply_to_id, path) AS (SELECT id, in_reply_to_id, ARRAY[id] FROM statuses WHERE id = ? UNION ALL SELECT statuses.id, statuses.in_reply_to_id, path || statuses.id FROM search_tree JOIN statuses ON statuses.id = search_tree.in_reply_to_id WHERE NOT statuses.id = ANY(path)) SELECT id FROM search_tree ORDER BY path DESC', self.id]) - [self])
+  end
+
+  def descendants
+    Status.where(id: Status.find_by_sql(['WITH RECURSIVE search_tree(id, path) AS (SELECT id, ARRAY[id] FROM statuses WHERE id = ? UNION ALL SELECT statuses.id, path || statuses.id FROM search_tree JOIN statuses ON statuses.in_reply_to_id = search_tree.id WHERE NOT statuses.id = ANY(path)) SELECT id FROM search_tree ORDER BY path', self.id]) - [self])
+  end
+
   after_create do
     self.account.stream_entries.create!(activity: self)
   end
diff --git a/app/views/stream_entries/_status.html.haml b/app/views/stream_entries/_status.html.haml
@@ -1,5 +1,6 @@
 - if status.reply? && include_threads
-  = render partial: 'status', locals: { status: status.thread, include_threads: false, is_predecessor: true, is_successor: false }
+  - status.ancestors.with_includes.with_counters.each do |status|
+    = render partial: 'status', locals: { status: status, include_threads: false, is_predecessor: true, is_successor: false }
 
 .entry{ class: entry_classes(status, is_predecessor, is_successor, include_threads) }
   - if status.reblog?
@@ -32,5 +33,5 @@
         = status.reblog? ? (status.reblog.local? ? linkify(status.reblog) : status.reblog.content.html_safe) : (status.local? ? linkify(status) : status.content.html_safe)
 
 - if include_threads
-  - status.replies.each do |status|
+  - status.descendants.with_includes.with_counters.each do |status|
     = render partial: 'status', locals: { status: status, include_threads: false, is_successor: true, is_predecessor: false }
diff --git a/spec/controllers/api/subscriptions_controller_spec.rb b/spec/controllers/api/subscriptions_controller_spec.rb
@@ -5,7 +5,7 @@ RSpec.describe Api::SubscriptionsController, type: :controller do
 
   describe 'GET #show' do
     before do
-      get :show, id: account.id, 'hub.topic': 'topic_url', 'hub.verify_token': 123, 'hub.challenge': '456'
+      get :show, :id => account.id, 'hub.topic' => 'topic_url', 'hub.verify_token' => 123, 'hub.challenge' => '456'
     end
 
     it 'returns http success' do