From 5b81bb1567a826d32f7013db42372a91c3753b35 Mon Sep 17 00:00:00 2001 From: Bart de Water Date: Sat, 28 Jul 2018 18:40:12 -0400 Subject: Fix test failures due to Performance/RegexpMatch correction `5 =~ /\d/` returns nil, but Integer doesn't have a `match?` method. --- .../lib/active_record/connection_adapters/abstract_adapter.rb | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) (limited to 'activerecord') diff --git a/activerecord/lib/active_record/connection_adapters/abstract_adapter.rb b/activerecord/lib/active_record/connection_adapters/abstract_adapter.rb index baf33e2d37..a4748dbeda 100644 --- a/activerecord/lib/active_record/connection_adapters/abstract_adapter.rb +++ b/activerecord/lib/active_record/connection_adapters/abstract_adapter.rb @@ -81,7 +81,9 @@ module ActiveRecord alias :in_use? :owner def self.type_cast_config_to_integer(config) - if SIMPLE_INT.match?(config) + if config.is_a?(Integer) + config + elsif SIMPLE_INT.match?(config) config.to_i else config -- cgit v1.2.3