DGNum/demarches-normaliennes
15
1
Fork 0
Code Issues 1 Pull requests Projects Releases Packages Wiki Activity

refactor(dossier): use each instead of filter with side effects

Browse source
This commit is contained in:
Paul Chavard 2024-07-16 09:03:56 +02:00
parent 7630cc39fa
commit 42ed6d7c87
No known key found for this signature in database
1 changed files with 6 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
app/models/dossier.rb
View file

@ -936,7 +936,12 @@ class Dossier < ApplicationRecord
end end
def remove_piece_justificative_file_not_visible! def remove_piece_justificative_file_not_visible!
champs.filter { _1.piece_justificative_file.attached? && !_1.visible? && _1.piece_justificative_file.purge_later } champs.each do |champ|
next unless champ.piece_justificative_file.attached?
next if champ.visible?
champ.piece_justificative_file.purge_later
end
end end
def check_mandatory_and_visible_champs def check_mandatory_and_visible_champs