From d7df1a740ea199e9b2210cc30f86830a9bf28db1 Mon Sep 17 00:00:00 2001 From: Go MAEDA Date: Fri, 23 Sep 2022 04:47:49 +0000 Subject: [PATCH] Unify duplicate codes (#37687). Patch by Go MAEDA. git-svn-id: https://svn.redmine.org/redmine/trunk@21832 e93f8b46-1217-0410-a6f0-8f06a7374b81 --- app/helpers/journals_helper.rb | 3 +-- app/models/journal.rb | 2 +- 2 files changed, 2 insertions(+), 3 deletions(-) diff --git a/app/helpers/journals_helper.rb b/app/helpers/journals_helper.rb index 18224c196..066226e29 100644 --- a/app/helpers/journals_helper.rb +++ b/app/helpers/journals_helper.rb @@ -21,8 +21,7 @@ module JournalsHelper # Returns the attachments of a journal that are displayed as thumbnails def journal_thumbnail_attachments(journal) - ids = journal.details.select {|d| d.property == 'attachment' && d.value.present?}.map(&:prop_key) - ids.any? ? Attachment.where(:id => ids).select(&:thumbnailable?).sort_by{|a| ids.index(a.id.to_s)} : [] + journal.attachments.select(&:thumbnailable?) end # Returns the action links for an issue journal diff --git a/app/models/journal.rb b/app/models/journal.rb index a4efcdbee..e60295c34 100644 --- a/app/models/journal.rb +++ b/app/models/journal.rb @@ -143,7 +143,7 @@ class Journal < ActiveRecord::Base def attachments ids = details.select {|d| d.property == 'attachment' && d.value.present?}.map(&:prop_key) - Attachment.where(id: ids).to_a + Attachment.where(id: ids).sort_by {|a| ids.index(a.id.to_s)} end # Returns a string of css classes -- 2.39.5