saxlsx 1.5.0 → 1.6.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 597206ffbcc440bed3d0972489b4027464993b91
-  data.tar.gz: 2286d59b75e1641f1ed47e927a25d612c7891111
+  metadata.gz: 912aea753226e55b3b97c74c094e36fff1a703b3
+  data.tar.gz: 71b3a09c85b6fbcb8dc5da74dfcb4134d4eb3ba6
 SHA512:
-  metadata.gz: e5d5d8e6b1e6e29e2fbcc28b40bd9ce91a9d10d56b40fc15a56ca73a6a002eca850e875f977c2d8a9f01ed4df11affafae1762205aac35a613762085515ee3a1
-  data.tar.gz: 9dffa50378a9f632982978496fdd1d959990e6c9836d280b7e7dff43980b4f8edbf43a1d5694690986b90b8c523d36b47388675234d640f7558e17cfed247894
+  metadata.gz: ee4a4b76abdf8d4d797a1704c892cd9359b2b2a646ac38267feeba182d7f349b7016c50788e0382ef9552cc767f2b3356ef5adbf462cc761fc44d24ad164b410
+  data.tar.gz: 1cd5883650144d961e7ff726af1900d5ca037f4c0a9d5efd2aeccdcf9312145ee108c21a6b35e3be8ccde8d942b2b578ed5e0be1c75fb077c8798c6ed947cbf6

data/lib/saxlsx/rows_collection_parser.rb

@@ -14,8 +14,8 @@ module Saxlsx
       9  => :percentage,     # 0%
       10 => :percentage,     # 0.00%
       11 => :bignum,         # 0.00E+00
-      12 => :unsupported,    # # ?/?
-      13 => :unsupported,    # # ??/??
+      12 => :rational,       # # ?/?
+      13 => :rational,       # # ??/??
       14 => :date,           # mm-dd-yy
       15 => :date,           # d-mmm-yy
       16 => :date,           # d-mmm
@@ -42,6 +42,7 @@ module Saxlsx
 
     def initialize(workbook, &block)
       @base_date      = workbook.base_date
+      @auto_format    = workbook.auto_format
       @shared_strings = workbook.shared_strings
       @number_formats = workbook.number_formats
       @block = block
@@ -109,17 +110,20 @@ module Saxlsx
           date = @base_date + Rational((Float(text) * SECONDS_IN_DAY).round, SECONDS_IN_DAY)
           DateTime.new(date.year, date.month, date.day, date.hour, date.minute, date.second)
         when :fixnum
-          Integer(text)
+          Integer(text, 10)
         when :float, :percentage
           Float(text)
+        when :rational
+          Rational(text)
         when :bignum
           Float(text) # raises ArgumentError if text is not a number
           BigDecimal(text) # doesn't raise ArgumentError
         else
           if @current_type == 'n'
             Float(text)
-          elsif text =~ /\A-?\d+(\.\d+(?:e[+-]\d+)?)?\Z/i # Auto convert numbers
-            $1 ? Float(text) : Integer(text)
+          elsif @auto_format && text =~ /\A-?\d+(\.\d+(?:e[+-]\d+)?)?\Z/i
+            # Auto convert numbers
+            $1 ? Float(text) : Integer(text, 10)
           else
             CGI.unescapeHTML(text)
           end

data/lib/saxlsx/version.rb

@@ -1,3 +1,3 @@
 module Saxlsx
-  VERSION = '1.5.0'
+  VERSION = '1.6.0'
 end

data/lib/saxlsx/workbook.rb

@@ -4,18 +4,20 @@ module Saxlsx
     DATE_SYSTEM_1904 = DateTime.new(1904, 1, 1)
 
     attr_accessor :date1904
+    attr_reader :auto_format
 
-    def self.open(filename)
+    def self.open(*args)
       begin
-        workbook = self.new(filename)
+        workbook = self.new(*args)
         yield workbook
       ensure
         workbook.close
       end
     end
 
-    def initialize(filename)
+    def initialize(filename, auto_format: true)
       @file_system = FileSystem.new filename
+      @auto_format = auto_format
     end
 
     def close

data/spec/sheet_spec.rb

@@ -121,10 +121,12 @@ describe Sheet do
     end
   end
 
-  context 'with number formats' do
+  context 'with number formats and auto format' do
     let(:filename) { "#{File.dirname(__FILE__)}/data/SpecNumberFormat.xlsx" }
 
     [ ["General",    "Test"],
+      ["General",    123],
+      ["General",    123.5],
       ["Fixnum",     123],
       ["Currency",   123.0],
       ["Date",       DateTime.new(1970, 1, 1)],
@@ -145,4 +147,31 @@ describe Sheet do
       end
     end
   end
+
+  context 'with number formats and without auto format' do
+    let(:filename) { "#{File.dirname(__FILE__)}/data/SpecNumberFormat.xlsx" }
+
+    [ ["General",    "Test"],
+      ["General",    "0123"],
+      ["General",    "0123.50"],
+      ["Fixnum",     123],
+      ["Currency",   123.0],
+      ["Date",       DateTime.new(1970, 1, 1)],
+      ["Time",       DateTime.new(2015, 2, 13, 12, 40, 5)],
+      ["Percentage", 0.9999],
+      ["Fraction",   0.5],
+      ["Scientific", BigDecimal.new('3.4028236692093801E+38')],
+      ["Custom",     123.0],
+    ].each.with_index do |row, i|
+      name, value = row
+
+      it "should typecast #{name}" do
+        Workbook.open filename, auto_format: false do |w|
+          w.sheets[0].tap do |s|
+            expect(s.rows[i+1]).to eq([name, value, "Test"])
+          end
+        end
+      end
+    end
+  end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: saxlsx
 version: !ruby/object:Gem::Version
-  version: 1.5.0
+  version: 1.6.0
 platform: ruby
 authors:
 - Edgars Beigarts
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-02-08 00:00:00.000000000 Z
+date: 2017-04-24 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rubyzip