about summary refs log tree commit diff
diff options
context:
space:
mode:
authorEugen Rochko <eugen@zeonfederated.com>2016-03-21 11:43:21 +0100
committerEugen Rochko <eugen@zeonfederated.com>2016-03-21 11:43:21 +0100
commit35aafdba961c90ecad04f38d4eec69100da36697 (patch)
tree294516de0da1177dd8bc28e774de3c4ab46ef80b
parent7e00a21ea6f76f8167f1070fbfe231bbd65c6cad (diff)
Ancestors and descendants of statuses
-rw-r--r--app/models/status.rb8
-rw-r--r--app/views/stream_entries/_status.html.haml5
-rw-r--r--spec/controllers/api/subscriptions_controller_spec.rb2
3 files changed, 12 insertions, 3 deletions
diff --git a/app/models/status.rb b/app/models/status.rb
index 08b5ee75e..2fdd8d123 100644
--- 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
index 16f036ad3..dff03716c 100644
--- 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
index e2f2ddd7e..de4c8f56d 100644
--- 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