bterkuile/fizzy
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Simplify comment in skip-detached override

Browse Source 
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This commit is contained in:
Joseph Hale
2026-03-02 14:40:21 -07:00
parent c61f49cce9
commit 7611236a67
1 changed files with 2 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files
lib/rails_ext/active_storage_analyze_job_skip_detached.rb
+2 -6
View File
@@ -1,9 +1,5 @@
# Skip analysis for blobs whose attachments have already been destroyed.
#
# When an upload is deleted before AnalyzeJob runs, PurgeOnLastAttachment has
# already destroyed the attachment row and enqueued PurgeJob for the blob's S3
# object. Analyzing at this point hits a missing key. Same check as
# PurgeOnLastAttachment#purge_blob_if_last.
# Skip analysis for blobs whose attachments have already been destroyed. Avoids
# Aws::S3::Errors::NoSuchKey when an upload is deleted before AnalyzeJob runs.
module ActiveStorageAnalyzeJobSkipDetached
def perform(blob)
return unless blob.attachments.exists?