concat: accept ActiveSupport::Multibyte::Chars

Faveod / arel-extensions

Extending Arel

MIT License

142 stars 24 forks source link

concat: accept ActiveSupport::Multibyte::Chars #27

Closed akimd closed 4 years ago

akimd commented 4 years ago

Currently if we try to pass the result of "fôobâr".mb_chars to concat, it ends with

ActiveSupport::Multibyte::Chars can not be converted to CONCAT arg
.../gems/2.5.0/gems/arel_extensions-1.2.12/lib/arel_extensions/nodes/function.rb:77:in `convert_to_node'

akimd commented 4 years ago

Review says: don't depend on ActiveSupport.