Make custom_field? method public

Author: n-rodriguez

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

@@ -27,7 +27,7 @@ def initialize(datatable, index, options)
         @datatable   = datatable
         @index       = index
         @options     = options
-        @view_column = datatable.view_columns[options['data'].to_sym]
+        @view_column = datatable.view_columns[options[:data].to_sym]
       end
 
       def data
@@ -50,6 +50,10 @@ def field
         source.split('.').last.to_sym
       end
 
+      def custom_field?
+        !source.include?('.')
+      end
+
       # Add formater option to allow modification of the value
       # before passing it to the database
       def formater
@@ -62,10 +66,6 @@ def formated_value
 
       private
 
-      def custom_field?
-        !source.include?('.')
-      end
-
       def config
         @config ||= AjaxDatatablesRails.config
       end

spec/ajax-datatables-rails/datatable/column_spec.rb

@@ -29,19 +29,19 @@
       expect(column.data).to eq('username')
     end
 
-    describe 'data' do
+    describe '#data' do
       it 'should return the data from params' do
         expect(column.data).to eq 'username'
       end
     end
 
-    describe 'source' do
+    describe '#source' do
       it 'should return the data source from view_column' do
         expect(column.source).to eq 'User.username'
       end
     end
 
-    describe 'table' do
+    describe '#table' do
       context 'with ActiveRecord ORM' do
         it 'should return the corresponding AR table' do
           expect(column.table).to eq User.arel_table
@@ -55,18 +55,24 @@
       end
     end
 
-    describe 'model' do
+    describe '#model' do
       it 'should return the corresponding AR model' do
         expect(column.model).to eq User
       end
     end
 
-    describe 'field' do
+    describe '#field' do
       it 'should return the corresponding field in DB' do
         expect(column.field).to eq :username
       end
     end
 
+    describe '#custom_field?' do
+      it 'should return false if field is bound to an AR field' do
+        expect(column.custom_field?).to be false
+      end
+    end
+
     describe '#search' do
       it 'child class' do
         expect(column.search).to be_a(AjaxDatatablesRails::Datatable::SimpleSearch)