FIX: Everyone should be aware a cached summary is outdated. (#23438)

This should be the case even they cannot regenerate it.
2023-09-06 12:09:21 -03:00 · 2023-09-06 12:09:21 -03:00 · 0f6ac634f4
parent f73d8346c2
commit 0f6ac634f4
1 changed files with 1 additions and 3 deletions
--- a/app/services/topic_summarization.rb
+++ b/app/services/topic_summarization.rb
@ -18,9 +18,7 @@ class TopicSummarization

    if use_cached?(existing_summary, can_summarize, current_topic_sha, !!opts[:skip_age_check])
      # It's important that we signal a cached summary is outdated
-      if can_summarize && new_targets?(existing_summary, current_topic_sha)
-        existing_summary.mark_as_outdated
-      end
+      existing_summary.mark_as_outdated if new_targets?(existing_summary, current_topic_sha)

      return existing_summary
    end