Fix trying to privatize empty media attachments (#15414)

4 years ago · 473abc35a1
parent 444b21b55f
commit 473abc35a1
2 changed files with 4 additions and 0 deletions
--- a/app/services/suspend_account_service.rb
+++ b/app/services/suspend_account_service.rb
@ -65,6 +65,8 @@ class SuspendAccountService < BaseService
        attachment = media_attachment.public_send(attachment_name)
        styles     = [:original] | attachment.styles.keys

+        next if attachment.blank?
+
        styles.each do |style|
          case Paperclip::Attachment.default_options[:storage]
          when :s3
--- a/app/services/unsuspend_account_service.rb
+++ b/app/services/unsuspend_account_service.rb
@ -56,6 +56,8 @@ class UnsuspendAccountService < BaseService
        attachment = media_attachment.public_send(attachment_name)
        styles     = [:original] | attachment.styles.keys

+        next if attachment.blank?
+
        styles.each do |style|
          case Paperclip::Attachment.default_options[:storage]
          when :s3