fixed an xlsx format bug where formating containing [red] would get p…

…icked up as date
roo-rb · Mar 24, 2014 · 350157e · 350157e
1 parent 7d403e3
commit 350157e
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/lib/roo/excelx.rb b/lib/roo/excelx.rb
@@ -46,7 +46,7 @@ def to_type(format)
         type
       elsif format.include?('#')
         :float
-      elsif format.include?('d') || format.include?('y')
+      elsif !format.match(/d+(?![\]])/).nil? || format.include?('y')
         if format.include?('h') || format.include?('s')
           :datetime
         else

diff --git a/spec/lib/roo/excelx/format_spec.rb b/spec/lib/roo/excelx/format_spec.rb
@@ -40,7 +40,8 @@
       'dd/mmm/yy\\ hh:mm' => :datetime,
       'dd/mmm/yy' => :date, # 2011-05-21
       'yyyy-mm-dd' => :date, # 2011-09-16
-      'yyyy-mm-dd;@' => :date
+      'yyyy-mm-dd;@' => :date,
+      '#0_);[Red]\(0\)' => :float
     }.each do |format, type|
       it "translates #{format} to #{type}" do
         Roo::Excelx::Format.to_type(format).should == type