diff options
| author | Eugen Rochko <eugen@zeonfederated.com> | 2019-10-09 07:10:46 +0200 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2019-10-09 07:10:46 +0200 |
| commit | 354fdd317e9c495ed721013911bc5274d5e0e1f8 (patch) | |
| tree | ae63530dfd6836be8ad212502958aa4bb6720b76 /app/models/concerns | |
| parent | 538db85d3cc6d8fcb3c0a89f7eef069a686c19f4 (diff) | |
Fix attachment not being re-downloaded even if file is not stored (#12125)
Change the behaviour of remotable concern. Previously, it would skip downloading an attachment if the stored remote URL is identical to the new one. Now it would not be skipped if the attachment is not actually currently stored by Paperclip.
Diffstat (limited to 'app/models/concerns')
| -rw-r--r-- | app/models/concerns/remotable.rb | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/app/models/concerns/remotable.rb b/app/models/concerns/remotable.rb index 082302619..b7a476c87 100644 --- a/app/models/concerns/remotable.rb +++ b/app/models/concerns/remotable.rb @@ -18,7 +18,7 @@ module Remotable return end - return if !%w(http https).include?(parsed_url.scheme) || parsed_url.host.blank? || self[attribute_name] == url + return if !%w(http https).include?(parsed_url.scheme) || parsed_url.host.blank? || (self[attribute_name] == url && send("#{attachment_name}_file_name").present?) begin Request.new(:get, url).perform do |response| |