simple_xlsx_reader 2.0.1 → 3.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 20c74bef372629ffb807d50df274c90682e17d53eed296a1e05fbb99533d4a8e
-  data.tar.gz: 9c4311913e79ad139414a4fe064d89fb246a1d46f30b7736c87f049985801660
+  metadata.gz: 5449525d4e46a013f92e8406a2ec2d07b06bb795efc7c8d76b9ffbcace22a38f
+  data.tar.gz: 5c664baa8d88692767f5bb6d2879e24c27098206695b88da42f3bc0d30bb9bce
 SHA512:
-  metadata.gz: 18db8595a36d4d9bb0f1dfee5da58753b799a6358530dfb9436f9c7b72e8e06bb9101d86d8dc617669c075fced44db8c24fdca6c5a3b2cb6b908cc0cd645eeb1
-  data.tar.gz: 23d4b057060c5f66ad3d57d0d65fa6e127ed4c7faa622ec0d236b58dd439179b41dff041ec2dd955c2ec893d577d9660465d2dcea0fe9a9f3af5e911ac5aa8e7
+  metadata.gz: 90d5fedde0aa4cc2bcb8b4d9134859890bbf4357efbf6ba9aa0aaf3bd21ad1cd9f54a279d1126938a300dee36fbb8a10d63b66f945c9f7eeb8edc880bb23327a
+  data.tar.gz: ab684cc09075a0b9a1054c045bf4202718159b258c87f62b10e18aafa4faaa7a527ba95f8e765f7609ab7ba24c43a31176906ecefd612c85badbdefee9164184

data/CHANGELOG.md

@@ -1,3 +1,16 @@
+### 3.0.0
+
+* Change the way we typecast cells in the General format. This probably won't
+  break anything in your app, but it's a change in behavior that theoretically
+  could.
+
+  Previously, we were treating cells using General the format as strings, when
+  according to the Office XML standard, they should be treated as numbers. We
+  now attempt to cast such cells as numbers, and fall back to strings if number
+  casting fails.
+
+  Thanks @jrodrigosm
+
 ### 2.0.1
 
 * Restore ability to parse IO strings (@robbevp)

data/lib/simple_xlsx_reader/loader.rb

@@ -149,7 +149,13 @@ module SimpleXlsxReader
         # detected earlier and cast here by its standardized symbol
         ##
 
-        when :string, :unsupported
+        # no type encoded with the the General format defaults to a number type
+        when nil, :string
+          retval = Integer(value, exception: false)
+          retval ||= Float(value, exception: false)
+          retval ||= value
+          retval
+        when :unsupported
           value
         when :fixnum
           value.to_i

data/lib/simple_xlsx_reader/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module SimpleXlsxReader
-  VERSION = '2.0.1'
+  VERSION = '3.0.0'
 end

data/test/simple_xlsx_reader_test.rb

@@ -827,6 +827,10 @@ describe SimpleXlsxReader do
                 <c r='I1' s='0'>
                   <v>GUI-made hyperlink</v>
                 </c>
+
+                <c r='J1' s='0'>
+                  <v>1</v>
+                </c>
               </row>
             </sheetData>
 
@@ -925,6 +929,10 @@ describe SimpleXlsxReader do
         )
       )
     end
+    
+    it "reads 'Generic' cells with numbers as numbers" do
+      _(@row[9]).must_equal 1
+    end
   end
 
   describe 'parsing documents with blank rows' do
@@ -936,7 +944,7 @@ describe SimpleXlsxReader do
             <sheetData>
             <row r="2" spans="1:1">
               <c r="A2" s="0">
-                <v>0</v>
+                <v>a</v>
               </c>
             </row>
             <row r="4" spans="1:1">
@@ -967,13 +975,30 @@ describe SimpleXlsxReader do
     it 'reads row data despite gaps in row numbering' do
       _(@rows).must_equal [
         [nil, nil, nil, nil],
-        ['0', nil, nil, nil],
+        ['a', nil, nil, nil],
         [nil, nil, nil, nil],
-        [nil, '1', nil, nil],
-        [nil, nil, '2', nil],
+        [nil, 1, nil, nil],
+        [nil, nil, 2, nil],
         [nil, nil, nil, nil],
-        [nil, nil, nil, '3']
+        [nil, nil, nil, 3]
       ]
     end
   end
+
+  # https://support.microsoft.com/en-us/office/available-number-formats-in-excel-0afe8f52-97db-41f1-b972-4b46e9f1e8d2
+  describe 'numeric fields styled as "General"' do
+    let(:misc_numbers_path) do
+      File.join(File.dirname(__FILE__), 'misc_numbers.xlsx')
+    end
+
+    let(:sheet) { SimpleXlsxReader.open(misc_numbers_path).sheets[0] }
+
+    it 'reads medium sized integers as integers' do
+      _(sheet.rows.slurp[1][0]).must_equal 98070
+    end
+
+    it 'reads large (>12 char) integers as integers' do
+      _(sheet.rows.slurp[1][1]).must_equal 1234567890123
+    end
+  end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: simple_xlsx_reader
 version: !ruby/object:Gem::Version
-  version: 2.0.1
+  version: 3.0.0
 platform: ruby
 authors:
 - Woody Peterson
@@ -113,6 +113,7 @@ files:
 - test/gdocs_sheet_test.rb
 - test/lower_case_sharedstrings.xlsx
 - test/lower_case_sharedstrings_test.rb
+- test/misc_numbers.xlsx
 - test/performance_test.rb
 - test/sesame_street_blog.xlsx
 - test/shared_strings.xml
@@ -152,6 +153,7 @@ test_files:
 - test/gdocs_sheet_test.rb
 - test/lower_case_sharedstrings.xlsx
 - test/lower_case_sharedstrings_test.rb
+- test/misc_numbers.xlsx
 - test/performance_test.rb
 - test/sesame_street_blog.xlsx
 - test/shared_strings.xml