From f320605e4b08b75305bd8adb18606e7835ab75a1 Mon Sep 17 00:00:00 2001 From: Sojan Jose Date: Tue, 28 Oct 2025 01:19:43 -0700 Subject: [PATCH] chore: fixes --- enterprise/app/models/captain/document.rb | 21 +++++++++------------ 1 file changed, 9 insertions(+), 12 deletions(-) diff --git a/enterprise/app/models/captain/document.rb b/enterprise/app/models/captain/document.rb index de048fac2..c95178f9e 100644 --- a/enterprise/app/models/captain/document.rb +++ b/enterprise/app/models/captain/document.rb @@ -94,21 +94,18 @@ class Captain::Document < ApplicationRecord end def enqueue_response_builder_job - return if destroyed? - return unless status == 'available' + return unless should_enqueue_response_builder? - status_became_available = saved_change_to_status? - content_now_present = content.present? - content_became_present = saved_change_to_content? && content_now_present + Captain::Documents::ResponseBuilderJob.perform_later(self) + end - should_enqueue = - if pdf_document? - status_became_available - else - (status_became_available && content_now_present) || content_became_present - end + def should_enqueue_response_builder? + return false if destroyed? + return false unless available? - Captain::Documents::ResponseBuilderJob.perform_later(self) if should_enqueue + return saved_change_to_status? if pdf_document? + + (saved_change_to_content? || saved_change_to_status?) && content.present? end def update_document_usage