Skip to content

Commit 4737032

Browse files
committed
Replace switch statement by a Hash lookup
1 parent cc44a08 commit 4737032

File tree

1 file changed

+11
-7
lines changed

1 file changed

+11
-7
lines changed

lib/ajax-datatables-rails/datatable/column.rb

Lines changed: 11 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,16 @@
33
module AjaxDatatablesRails
44
module Datatable
55
class Column
6+
7+
DB_ADAPTER_TYPE_CAST = {
8+
mysql: 'CHAR',
9+
mysql2: 'CHAR',
10+
sqlite: 'TEXT',
11+
sqlite3: 'TEXT',
12+
oracle: 'VARCHAR2(4000)',
13+
oracleenhanced: 'VARCHAR2(4000)',
14+
}.freeze
15+
616
attr_reader :datatable, :index, :options
717
attr_writer :search
818

@@ -127,13 +137,7 @@ def non_regex_search
127137
end
128138

129139
def typecast
130-
case config.db_adapter
131-
when :oracle, :oracleenhanced then 'VARCHAR2(4000)'
132-
when :mysql, :mysql2 then 'CHAR'
133-
when :sqlite, :sqlite3 then 'TEXT'
134-
else
135-
'VARCHAR'
136-
end
140+
DB_ADAPTER_TYPE_CAST[config.db_adapter] || 'VARCHAR'
137141
end
138142

139143
def casted_column

0 commit comments

Comments
 (0)