summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorRémy Coutable <remy@rymai.me>2018-05-30 11:43:40 +0000
committerRémy Coutable <remy@rymai.me>2018-05-30 11:43:40 +0000
commit1058973fdc226746c2a693ce429f8bf8becb74c6 (patch)
tree047d9db8eae66df65f171fb1695ada891590b1f5
parenta51c57730fcd32d085fc196722b2a569d76f59c1 (diff)
parent884fbf1d058ac00bc5c107f327475fa6eeda1f9f (diff)
downloadgitlab-ce-46973-job-failed-71275338.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
-rw-r--r--app/models/service.rb5
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
}