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

small comment explaining why we need to ensure used columns are present in procedure.columns

Browse source
This commit is contained in:
simon lehericey 2024-11-15 16:45:32 +01:00
parent fa64e8f112
commit aa0b7f53ef
No known key found for this signature in database
GPG key ID: CDE670D827C7B3C5
1 changed files with 2 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
app/models/concerns/columns_concern.rb
View file

@ -37,6 +37,7 @@ module ColumnsConcern
columns.concat(procedure_chorus_columns) if chorusable? && chorus_configuration.complete? columns.concat(procedure_chorus_columns) if chorusable? && chorus_configuration.complete?
# ensure the columns exist in main list # ensure the columns exist in main list
# otherwise, they will be found by the find_column method
columns.filter { _1.id.in?(self.columns.map(&:id)) } columns.filter { _1.id.in?(self.columns.map(&:id)) }
end end
@ -48,6 +49,7 @@ module ColumnsConcern
columns.concat([groupe_instructeurs_id_column, followers_instructeurs_email_column]) columns.concat([groupe_instructeurs_id_column, followers_instructeurs_email_column])
# ensure the columns exist in main list # ensure the columns exist in main list
# otherwise, they will be found by the find_column method
columns.filter { _1.id.in?(self.columns.map(&:id)) } columns.filter { _1.id.in?(self.columns.map(&:id)) }
end end