diff options
author | Sean Griffin <sean@seantheprogrammer.com> | 2016-08-31 12:38:37 -0400 |
---|---|---|
committer | Sean Griffin <sean@seantheprogrammer.com> | 2016-08-31 12:38:37 -0400 |
commit | 84efde740dc357de3ace08b2f2fd7a4a67dbd9bd (patch) | |
tree | c858f06849f59885cb736027d49f954de2ca59b4 /activerecord/lib/active_record/relation/predicate_builder | |
parent | 7ba3a48df5bfdc5e98506bb829f937e03b55a5b3 (diff) | |
download | rails-84efde740dc357de3ace08b2f2fd7a4a67dbd9bd.tar.gz rails-84efde740dc357de3ace08b2f2fd7a4a67dbd9bd.tar.bz2 rails-84efde740dc357de3ace08b2f2fd7a4a67dbd9bd.zip |
Revert "Extract `PredicateBuilder::CaseSensitiveHandler`"
This reverts commit 3a1f6fe7b4a70bf0698b0684dd48ac712c6883b6.
This commit takes the code in a direction that I am looking to avoid.
The predicate builder should be purely concerned with AST construction
as it matters to methods like `where`. Things like case sensitivity
should continue to be handled elsewhere.
Diffstat (limited to 'activerecord/lib/active_record/relation/predicate_builder')
-rw-r--r-- | activerecord/lib/active_record/relation/predicate_builder/case_sensitive_handler.rb | 21 |
1 files changed, 0 insertions, 21 deletions
diff --git a/activerecord/lib/active_record/relation/predicate_builder/case_sensitive_handler.rb b/activerecord/lib/active_record/relation/predicate_builder/case_sensitive_handler.rb deleted file mode 100644 index acf0bbd829..0000000000 --- a/activerecord/lib/active_record/relation/predicate_builder/case_sensitive_handler.rb +++ /dev/null @@ -1,21 +0,0 @@ -module ActiveRecord - class PredicateBuilder - class CaseSensitiveHandler # :nodoc: - def call(attribute, value) - value.call(attribute) - end - - class Value < Struct.new(:value, :table, :case_sensitive?) # :nodoc: - def call(attribute) - klass = table.send(:klass) - column = klass.column_for_attribute(attribute.name) - if case_sensitive? - klass.connection.case_sensitive_comparison(attribute, column, value) - else - klass.connection.case_insensitive_comparison(attribute, column, value) - end - end - end - end - end -end |