about summary refs log tree commit diff
path: root/app
diff options
context:
space:
mode:
authorThibG <thib@sitedethib.com>2019-07-15 01:04:31 +0200
committerGitHub <noreply@github.com>2019-07-15 01:04:31 +0200
commit5ccd011cc3bafc00a557dabbb47da004f3d4a381 (patch)
treec64218e9cdd13570181b0a1b9c86289aa4b22254 /app
parent1b074d2a50a331cdd03296170f04a75eec97a519 (diff)
parent3ad8c9ea61759c8ac12744f2b7b263711403b41c (diff)
Merge pull request #1161 from ThibG/glitch-soc/cherry-pick-upstream
Cherry pick changes from upstream
Diffstat (limited to 'app')
-rw-r--r--app/models/status.rb2
-rw-r--r--app/services/backup_service.rb2
-rw-r--r--app/services/block_service.rb2
-rw-r--r--app/workers/web/push_notification_worker.rb2
4 files changed, 5 insertions, 3 deletions
diff --git a/app/models/status.rb b/app/models/status.rb
index 5ddce72de..5adccb722 100644
--- a/app/models/status.rb
+++ b/app/models/status.rb
@@ -82,7 +82,7 @@ class Status < ApplicationRecord
   default_scope { recent }
 
   scope :recent, -> { reorder(id: :desc) }
-  scope :remote, -> { where(local: false).or(where.not(uri: nil)) }
+  scope :remote, -> { where(local: false).where.not(uri: nil) }
   scope :local,  -> { where(local: true).or(where(uri: nil)) }
 
   scope :without_replies, -> { where('statuses.reply = FALSE OR statuses.in_reply_to_account_id = statuses.account_id') }
diff --git a/app/services/backup_service.rb b/app/services/backup_service.rb
index bd9e77223..cc9fb1f4e 100644
--- a/app/services/backup_service.rb
+++ b/app/services/backup_service.rb
@@ -164,5 +164,7 @@ class BackupService < BaseService
         io.write(buffer)
       end
     end
+  rescue Errno::ENOENT
+    Rails.logger.warn "Could not backup file #{filename}: file not found"
   end
 end
diff --git a/app/services/block_service.rb b/app/services/block_service.rb
index 9050a4858..0d9a6eccd 100644
--- a/app/services/block_service.rb
+++ b/app/services/block_service.rb
@@ -8,7 +8,7 @@ class BlockService < BaseService
 
     UnfollowService.new.call(account, target_account) if account.following?(target_account)
     UnfollowService.new.call(target_account, account) if target_account.following?(account)
-    RejectFollowService.new.call(account, target_account) if target_account.requested?(account)
+    RejectFollowService.new.call(target_account, account) if target_account.requested?(account)
 
     block = account.block!(target_account)
 
diff --git a/app/workers/web/push_notification_worker.rb b/app/workers/web/push_notification_worker.rb
index 8e8a35973..901043975 100644
--- a/app/workers/web/push_notification_worker.rb
+++ b/app/workers/web/push_notification_worker.rb
@@ -3,7 +3,7 @@
 class Web::PushNotificationWorker
   include Sidekiq::Worker
 
-  sidekiq_options backtrace: true
+  sidekiq_options backtrace: true, retry: 5
 
   def perform(subscription_id, notification_id)
     subscription = ::Web::PushSubscription.find(subscription_id)