spreadbase 0.2.0 → 0.2.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 603c441b06a1eb85d926da95f9ce1c384fcd1642402e786fa7fc6106cd2e6def
-  data.tar.gz: 973d5b1f2908261266e760b028f7cc884e5c9e9ba651488d9c7396f9d39ec688
+  metadata.gz: c697a87553389d94f16ea79fbcb0552ddea9dd7e18b13404078cca8b0d13b32d
+  data.tar.gz: 05b540a328ee21d7263b4f3b619dd6799641d4ca04ef2d99b694ec6d75b243a5
 SHA512:
-  metadata.gz: 99e574f21843aa28699d5732942165e11f5b213a3dfe932f365b4f9bd0c0798fca7b4b49a84a755e2d9dbfabdfacab5cc3a840eb727ef09a9890d8f1255d470e
-  data.tar.gz: 1fbe613fa5c63cd02853ce36d8d423a2597ab536e560c6c14f79ca9e5c7b247fc9f643b7390561221c20346f54ac72954048b3d9e5f2e90b64f4b7f4c2718cfe
+  metadata.gz: e94b944ff92895cb2f090c7c8b4ca2b0ce84eee4ffd2c9cb855047f430992ed60bfa6dbf39f336414887a4f12b02c3ce1dc0857ae7293e071514a9360c7f2334
+  data.tar.gz: 32c1bf438e40c8a5b4a321160f3931c3afae5fe1f455946fe2912c1ea26c2f423ba57af0b0cb06b92f7470db12656bc047180eabed82416664bd6e0d176884c1

data/.github/workflows/ci.yml

@@ -0,0 +1,25 @@
+name: CI
+
+on: [push, pull_request]
+
+jobs:
+  rspec:
+    runs-on: ubuntu-latest
+
+    strategy:
+      matrix:
+        ruby: ['3.0', '2.7', '2.6', '2.5', '2.4', '2.3']
+
+    steps:
+      - name: Checkout spreadbase repository
+        uses: actions/checkout@v2
+
+      - name: Setup Ruby
+        uses: ruby/setup-ruby@v1
+        with:
+          ruby-version: ${{ matrix.ruby }}
+          bundler-cache: true
+
+      - name: Run RSpec
+        run: |
+          bundle exec rake spec

data/Gemfile

@@ -1,3 +1,7 @@
 source 'https://rubygems.org'
 
 gemspec
+
+if Gem.ruby_version >= Gem::Version.new('3.0.0')
+  gem 'rexml'
+end

data/README.md

@@ -1,4 +1,4 @@
-[![Build Status][BS img]](https://travis-ci.org/saveriomiroddi/spreadbase)
+[![CI](https://github.com/saveriomiroddi/spreadbase/actions/workflows/ci.yml/badge.svg)](https://github.com/saveriomiroddi/spreadbase/actions/workflows/ci.yml)
 
 SpreadBase!!
 ============
@@ -154,5 +154,3 @@ Roadmap/Todo
 ------------
 
 https://github.com/saveriomiroddi/spreadbase/wiki/Todo-%28roadmap%29
-
-[BS img]: https://travis-ci.org/saveriomiroddi/spreadbase.svg?branch=master

data/lib/spreadbase/codecs/open_document_12_modules/decoding.rb

@@ -59,7 +59,7 @@ module SpreadBase # :nodoc:
           # A single column/row can represent multiple columns (table:number-(columns|rows)-repeated)
           #
           table.column_width_styles = column_nodes.inject([]) { | current_styles, node | current_styles + decode_column_width_style(node) }
-          table.data                = row_nodes.inject([]) { | current_rows, node | current_rows + decode_row_node(node, options) }
+          table.data                = decode_row_nodes(row_nodes, options)
 
           table
         end
@@ -73,23 +73,48 @@ module SpreadBase # :nodoc:
           make_array_from_repetitions(style_name, repetitions)
         end
 
+        def decode_row_nodes(row_nodes, options)
+          rows = []
+          row_nodes.inject(0) do |size, node|
+            row, repetitions = decode_row_node(node, options)
+            row.empty? || append_row(rows, size, row, repetitions)
+            size + repetitions
+          end
+          rows
+        end
+
         def decode_row_node(row_node, options)
           repetitions = (row_node.attributes['table:number-rows-repeated'] || '1').to_i
           cell_nodes  = row_node.elements.to_a('table:table-cell')
 
-          # Watch out the :flatten; a single cell can represent multiple cells (table:number-columns-repeated)
-          #
-          values = cell_nodes.map { | node | decode_cell_node(node, options) }.flatten
+          [decode_cell_nodes(cell_nodes, options), repetitions]
+        end
 
-          make_array_from_repetitions(values, repetitions)
+        def append_row(rows, size, row, repetitions)
+          (size - rows.size).times { rows << [] }
+          rows.concat(make_array_from_repetitions(row, repetitions))
         end
 
-        def decode_cell_node(cell_node, options)
-          value = decode_cell_value(cell_node, options)
+        def decode_cell_nodes(cell_nodes, options)
+          cells = []
+          cell_nodes.inject(0) do |size, node|
+            cell, repetitions = decode_cell_node(node, options)
+            cell.nil? || append_cell(cells, size, cell, repetitions)
+            size + repetitions
+          end
+          cells
+        end
 
-          repetitions = (cell_node.attributes['table:number-columns-repeated'] || '1').to_i
+        def decode_cell_node(cell_node, options)
+          [
+            decode_cell_value(cell_node, options),
+            (cell_node.attributes['table:number-columns-repeated'] || '1').to_i
+          ]
+        end
 
-          make_array_from_repetitions(value, repetitions)
+        def append_cell(cells, size, cell, repetitions)
+          cells[size - 1] = nil if size != cells.size
+          cells.concat(make_array_from_repetitions(cell, repetitions))
         end
 
         def decode_cell_value(cell_node, options)

data/lib/spreadbase/version.rb

@@ -1,3 +1,3 @@
 module SpreadBase
-  VERSION = "0.2.0"
+  VERSION = "0.2.1"
 end

data/spec/codecs/open_document_12_spec.rb

@@ -15,7 +15,7 @@ describe SpreadBase::Codecs::OpenDocument12 do
       'abc', [
         [1,      1.1,        T_BIGDECIMAL],
         [T_DATE, T_DATETIME, T_TIME],
-        [nil,    'a',        nil]
+        [nil,    nil,        'a']
       ]
     )
 
@@ -56,8 +56,8 @@ describe SpreadBase::Codecs::OpenDocument12 do
 
         assert_size(row_3, 3) do | value_1, value_2, value_3 |
           expect(value_1).to eq(nil)
-          expect(value_2).to eq('a')
-          expect(value_3).to eq(nil)
+          expect(value_2).to eq(nil)
+          expect(value_3).to eq('a')
         end
 
       end
@@ -112,4 +112,90 @@ describe SpreadBase::Codecs::OpenDocument12 do
     expect(value).to eq(T_BIGDECIMAL)
   end
 
+  context "when cells at the end of the row are empty" do
+    let(:document_archive) do
+      document = SpreadBase::Document.new
+
+      document.tables << SpreadBase::Table.new(
+        'abc', [
+          [nil],
+          [nil, nil],
+          [1  , nil],
+          [nil, 1  , nil]
+        ]
+      )
+
+      SpreadBase::Codecs::OpenDocument12.new.encode_to_archive(document)
+    end
+
+    it "should drop such cells" do
+      document = SpreadBase::Codecs::OpenDocument12.new.decode_archive(document_archive)
+      table = document.tables[0]
+
+      assert_size(table.data, 4) do |row_1, row_2, row_3, row_4|
+        assert_size(row_1, 0)
+
+        assert_size(row_2, 0)
+
+        assert_size(row_3, 1) do |value_1|
+          expect(value_1).to eq(1)
+        end
+
+        assert_size(row_4, 2) do |value_1, value_2|
+          expect(value_1).to be_nil
+          expect(value_2).to eq(1)
+        end
+      end
+    end
+  end
+
+  context "when cells of the last row are empty" do
+    let(:document_archive) do
+      document = SpreadBase::Document.new
+
+      document.tables << SpreadBase::Table.new(
+        'abc', [
+          []
+        ]
+      )
+
+      document.tables << SpreadBase::Table.new(
+        'def', [
+          [nil]
+        ]
+      )
+
+      document.tables << SpreadBase::Table.new(
+        'ghi', [
+          [nil, nil]
+        ]
+      )
+
+      document.tables << SpreadBase::Table.new(
+        'jkl', [
+          [nil],
+          [1],
+          [nil]
+        ]
+      )
+
+      SpreadBase::Codecs::OpenDocument12.new.encode_to_archive(document)
+    end
+
+    it "should drop such row" do
+      document = SpreadBase::Codecs::OpenDocument12.new.decode_archive(document_archive)
+      tables = document.tables
+
+      assert_size(tables, 4) do |table_1, table_2, table_3, table_4|
+        assert_size(table_1.data, 0)
+
+        assert_size(table_2.data, 0)
+
+        assert_size(table_3.data, 0)
+
+        assert_size(table_4.data, 2)
+      end
+    end
+  end
+
 end

data/spec/elements/document_spec.rb

@@ -109,4 +109,27 @@ abc:
     expect(@sample_document.to_s(with_headers: true)).to eq(expected_string)
   end
 
+  it "should parse a existing ODS file" do
+    ods_file = File.expand_path('../fixtures/test.ods', __dir__)
+    document = SpreadBase::Document.new(ods_file)
+
+    expect(document.tables[0].name).to eq('Sheet1')
+    expect(document.tables[0].data).to match([
+      match(['hoge'                                      ]),
+      match([nil   , 'fuga'                              ]),
+      match([nil   , nil   , 'piyo'                      ]),
+      match([nil   , nil   , nil   , 'hogera'            ]),
+      match([nil   , nil   , nil   , nil     , 'hogehoge'])
+    ])
+
+    expect(document.tables[1].name).to eq('Sheet2')
+    expect(document.tables[1].data).to match([
+      match([nil  , nil     , nil  , nil  , 'foo']),
+      match([nil  , nil     , nil  , 'bar'       ]),
+      match([nil  , nil     , 'baz'              ]),
+      match([nil  , 'foobar'                     ]),
+      match(['qux'                               ])
+    ])
+  end
+
 end

data/spreadbase.gemspec

@@ -10,7 +10,7 @@ Gem::Specification.new do |s|
   s.platform    = Gem::Platform::RUBY
   s.required_ruby_version = '>= 2.3.0'
   s.authors     = ["Saverio Miroddi"]
-  s.date        = '2020-07-15'
+  s.date        = '2021-03-05'
   s.email       = ["saverio.pub2@gmail.com"]
   s.homepage    = "https://github.com/saveriomiroddi/spreadbase"
   s.summary     = %q{Library for reading/writing OpenOffice Calc documents.}

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: spreadbase
 version: !ruby/object:Gem::Version
-  version: 0.2.0
+  version: 0.2.1
 platform: ruby
 authors:
 - Saverio Miroddi
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-07-15 00:00:00.000000000 Z
+date: 2021-03-05 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rubyzip
@@ -59,9 +59,9 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
+- ".github/workflows/ci.yml"
 - ".gitignore"
 - ".rspec"
-- ".travis.yml"
 - Gemfile
 - LICENSE
 - README.md
@@ -80,6 +80,7 @@ files:
 - spec/codecs/open_document_12_spec.rb
 - spec/elements/document_spec.rb
 - spec/elements/table_spec.rb
+- spec/fixtures/test.ods
 - spec/spec_helper.rb
 - spec/spec_helpers.rb
 - spreadbase.gemspec
@@ -107,7 +108,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.8
+rubygems_version: 3.2.3
 signing_key: 
 specification_version: 4
 summary: Library for reading/writing OpenOffice Calc documents.

data/.travis.yml

@@ -1,8 +0,0 @@
-dist: bionic
-language: ruby
-rvm:
-  - 2.3
-  - 2.4
-  - 2.5
-  - 2.6
-  - 2.7