about summary refs log tree commit diff
diff options
context:
space:
mode:
authorEugen Rochko <eugen@zeonfederated.com>2016-03-16 10:46:15 +0100
committerEugen Rochko <eugen@zeonfederated.com>2016-03-16 10:46:15 +0100
commit75b3339a99bd32c59004effd05dcd0edfc0b7c2a (patch)
tree047a33e5e866527c43c098ed50896e1e53dd6a29
parent1eb65e2accab3dbc84e1a3ed3fd7c4940769be99 (diff)
Respect "delete" verb on remote feeds
-rw-r--r--.dockerignore2
-rw-r--r--app/models/status.rb2
-rw-r--r--app/services/process_feed_service.rb14
3 files changed, 15 insertions, 3 deletions
diff --git a/.dockerignore b/.dockerignore
index 665da4550..de0577641 100644
--- a/.dockerignore
+++ b/.dockerignore
@@ -1,2 +1,4 @@
 .env
 .env.*
+public/system
+public/assets
diff --git a/app/models/status.rb b/app/models/status.rb
index 52e6f8a21..81744c25e 100644
--- a/app/models/status.rb
+++ b/app/models/status.rb
@@ -7,7 +7,7 @@ class Status < ActiveRecord::Base
   has_one :stream_entry, as: :activity, dependent: :destroy
 
   has_many :favourites, inverse_of: :status, dependent: :destroy
-  has_many :reblogs, foreign_key: 'reblog_of_id', class_name: 'Status', inverse_of: :reblog
+  has_many :reblogs, foreign_key: 'reblog_of_id', class_name: 'Status', inverse_of: :reblog, dependent: :destroy
   has_many :replies, foreign_key: 'in_reply_to_id', class_name: 'Status', inverse_of: :thread
   has_many :mentioned_accounts, class_name: 'Mention', dependent: :destroy
 
diff --git a/app/services/process_feed_service.rb b/app/services/process_feed_service.rb
index 7d8563a58..605f615fd 100644
--- a/app/services/process_feed_service.rb
+++ b/app/services/process_feed_service.rb
@@ -16,15 +16,21 @@ class ProcessFeedService < BaseService
 
       status = Status.find_by(uri: activity_id(entry))
 
+      # If we already have a post and the verb is now "delete", we gotta delete it and move on!
+      if verb(entry) == :delete
+        delete_post!(status)
+        next
+      end
+
       next unless status.nil?
 
       status = Status.new(uri: activity_id(entry), url: activity_link(entry), account: account, text: content(entry), created_at: published(entry), updated_at: updated(entry))
 
-      if object_type(entry) == :comment
+      if object_type(entry) == :comment && verb(entry) == :post
         add_reply!(entry, status)
       elsif verb(entry) == :share
         add_reblog!(entry, status)
-      else
+      elsif verb(entry) == :post
         add_post!(entry, status)
       end
 
@@ -71,6 +77,10 @@ class ProcessFeedService < BaseService
     status.save!
   end
 
+  def delete_post!(status)
+    status.destroy!
+  end
+
   def find_original_status(_xml, id)
     return nil if id.nil?