Skip to content

Commit ecc6d78

Browse files
committed
fix: properly handle tree min/max validations
1 parent 1498e6e commit ecc6d78

File tree

1 file changed

+6
-8
lines changed

1 file changed

+6
-8
lines changed

app/models/tree_field_type.rb

Lines changed: 6 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -36,19 +36,17 @@ def mapping_field_name
3636
end
3737

3838
def minimum
39-
unless @values.nil?
40-
if @values.length >= @validations[:minimum]
41-
true
42-
else
43-
errors.add(:minimum, "You have selected too few values.")
44-
false
45-
end
39+
if !@values.nil? && @values[:values].length >= @validations[:minimum]
40+
true
41+
else
42+
errors.add(:minimum, "You have selected too few values.")
43+
false
4644
end
4745
end
4846

4947
def maximum
5048
unless @values.nil?
51-
if @values.length <= @validations[:maximum]
49+
if @values[:values].length <= @validations[:maximum]
5250
true
5351
else
5452
errors.add(:maximum, "You have selected too many values.")

0 commit comments

Comments
 (0)