diff options
| author | Claire <claire.github-309c@sitedethib.com> | 2021-09-13 18:59:37 +0200 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2021-09-13 18:59:37 +0200 |
| commit | db57bff11d6a9e101d8aa0adc635367365c83901 (patch) | |
| tree | b10d41b6cbc89695d49cd4fce24d63a1c2b493aa /app/models | |
| parent | 75027b51a40a2ba26a3d89c89d2d679f3831c372 (diff) | |
Stop setting a shortcode to newly-created media attachments (#16730)
* Stop setting a shortcode to newly-created media attachments The WebUI has stopped using the “short media URL” in ages. This isn't used anywhere except for mail notifications. Deprecating it would allow us to eventually get rid of at least a database column and corruption-prone index, as well as a controller. * Fix tests
Diffstat (limited to 'app/models')
| -rw-r--r-- | app/models/media_attachment.rb | 11 |
1 files changed, 2 insertions, 9 deletions
diff --git a/app/models/media_attachment.rb b/app/models/media_attachment.rb index 3515f6895..66d800b7b 100644 --- a/app/models/media_attachment.rb +++ b/app/models/media_attachment.rb @@ -255,7 +255,7 @@ class MediaAttachment < ApplicationRecord after_commit :reset_parent_cache, on: :update before_create :prepare_description, unless: :local? - before_create :set_shortcode + before_create :set_unknown_type before_create :set_processing after_post_process :set_meta @@ -298,15 +298,8 @@ class MediaAttachment < ApplicationRecord private - def set_shortcode + def set_unknown_type self.type = :unknown if file.blank? && !type_changed? - - return unless local? - - loop do - self.shortcode = SecureRandom.urlsafe_base64(14) - break if MediaAttachment.find_by(shortcode: shortcode).nil? - end end def prepare_description |