Merge pull request #162 from rishijain/update_special_character

handled parsing when header has brackets in it

lib/roo/base.rb

@@ -403,7 +403,15 @@ def row_with(query, return_headers = false)
       # makes sure headers is the first part of wildcard search for priority
       # ex. if UPC and SKU exist for UPC*SKU search, UPC takes the cake
       headers = query.map do |q|
-        q.map { |i| row.grep(/#{i}/i)[0] }.compact[0]
+        q.map do |i|
+          #if header has brackets in it for ex: date(yyyy-mm-dd), 
+          #can add other special characterss to this list.
+          if i.include?('(')
+            row.grep(i)[0]
+          else
+            row.grep(/#{i}/i)[0]
+          end
+        end.compact[0]
       end.compact
 
       if headers.length == query.length

test/test_roo.rb

@@ -295,6 +295,15 @@ def test_formula_openoffice
     end
   end
 
+  def test_header_with_brackets_excelx
+    with_each_spreadsheet(:name => 'advanced_header', :format => :openoffice) do |oo|
+      parsed_head = oo.parse(:headers => true)
+      assert_equal "Date(yyyy-mm-dd)", oo.cell('A',1)
+      assert_equal parsed_head[0].keys, ["Date(yyyy-mm-dd)"]
+      assert_equal parsed_head[0].values, ["Date(yyyy-mm-dd)"]
+    end
+  end
+
   def test_formula_excelx
     with_each_spreadsheet(:name=>'formula', :format=>:excelx) do |oo|
       assert_equal 1, oo.cell('A',1)