diff options
author | Rémy Coutable <remy@rymai.me> | 2018-05-30 11:43:40 +0000 |
---|---|---|
committer | Rémy Coutable <remy@rymai.me> | 2018-05-30 11:43:40 +0000 |
commit | 1058973fdc226746c2a693ce429f8bf8becb74c6 (patch) | |
tree | 047d9db8eae66df65f171fb1695ada891590b1f5 /app | |
parent | a51c57730fcd32d085fc196722b2a569d76f59c1 (diff) | |
parent | 884fbf1d058ac00bc5c107f327475fa6eeda1f9f (diff) | |
download | gitlab-ce-1058973fdc226746c2a693ce429f8bf8becb74c6.tar.gz |
Merge branch 'rails5-boolean-cast' into 'master'46973-job-failed-71275338
Fix boolean casting for nil value
Closes #46925
See merge request gitlab-org/gitlab-ce!19202
Diffstat (limited to 'app')
-rw-r--r-- | app/models/service.rb | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/app/models/service.rb b/app/models/service.rb index 831c2ea1141..1d259bcfec7 100644 --- a/app/models/service.rb +++ b/app/models/service.rb @@ -206,10 +206,11 @@ class Service < ActiveRecord::Base args.each do |arg| class_eval %{ def #{arg}? + # '!!' is used because nil or empty string is converted to nil if Gitlab.rails5? - !ActiveModel::Type::Boolean::FALSE_VALUES.include?(#{arg}) + !!ActiveRecord::Type::Boolean.new.cast(#{arg}) else - ActiveRecord::ConnectionAdapters::Column::TRUE_VALUES.include?(#{arg}) + !!ActiveRecord::Type::Boolean.new.type_cast_from_database(#{arg}) end end } |