Merge pull request #34 from mvdamme/fix_presenter_bug

Fixed presenter bug related to combining html_safe cell content and td_options

lib/table_cloth/presenters/default.rb

@@ -41,7 +41,7 @@ def row_for_object(object)
       end
 
       def column_for_object(column, object)
-        td_options = column.options[:td_options] || {}
+        td_options = column.options[:td_options].try(:dup) || {}
         value = column.value(object, view_context, table)
 
         if value.is_a?(Array)

spec/lib/presenters/default_spec.rb

@@ -66,5 +66,20 @@
         end
       end
     end
+
+    context 'html_safe cell content with td_options' do
+      let(:objects) do
+        list = FactoryGirl.build_list(:dummy_model, 3, :name => nil)
+        list.first.name = 'robert'
+        list
+      end
+      let(:dummy_table) { Class.new(DummyTableWithHTMLSafe) }
+
+      it 'only outputs html_safe content in the correct column' do
+        tbody = Nokogiri::HTML(subject.tbody.to_s).at_xpath('//tbody')
+
+        expect( tbody.xpath('//td').map(&:content) ).to eq ['robert', '', '']
+      end
+    end
   end
 end

spec/support/dummy_table_with_html_safe.rb

@@ -0,0 +1,5 @@
+class DummyTableWithHTMLSafe < TableCloth::Base
+  column :name, td_options: { class: 'irrelevant' } do |object|
+    object.name.html_safe if object.name
+  end
+end