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

Merge pull request #8872 from tchak/fix-champ-multiselect

Browse source 
fix(dossier): selecting blank option should not empty selection
This commit is contained in:
Paul Chavard 2023-04-06 14:32:35 +00:00 committed by GitHub
commit 79fee39888
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 18 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
app/models/champs/multiple_drop_down_list_champ.rb
View file

@ -98,7 +98,7 @@ class Champs::MultipleDropDownListChamp < Champ
end
def value=(value)
return super(nil) if value.blank?
return super(nil) if value.nil?
values = if value.is_a?(Array)
value

17
spec/models/champs/multiple_drop_down_list_champ_spec.rb
View file

@ -32,6 +32,23 @@ describe Champs::MultipleDropDownListChamp do
it { is_expected.not_to be_valid }
end
context 'set value' do
it {
subject.value = ["val1"]
expect(subject.value).to eq("[\"val1\"]")
subject.value = 'val2'
expect(subject.value).to eq("[\"val1\",\"val2\"]")
subject.value = ''
expect(subject.value).to eq("[\"val1\",\"val2\"]")
subject.value = nil
expect(subject.value).to be_nil
subject.value = ["val1"]
expect(subject.value).to eq("[\"val1\"]")
subject.value = []
expect(subject.value).to be_nil
}
end
end
end