spreadsheet 1.1.5 → 1.3.0

data/lib/spreadsheet/format.rb

@@ -101,12 +101,12 @@ module Spreadsheet
       @pattern_fg_color = :border
       @pattern_bg_color = :pattern_bg
       @regexes = {
-        :date         => Regexp.new(client("[YMD]", 'UTF-8')),
+        :date         => Regexp.new(client("[YMD]|d{2}|m{3}|y{2}", 'UTF-8')),
         :date_or_time => Regexp.new(client("[hmsYMD]", 'UTF-8')),
         :datetime     => Regexp.new(client("([YMD].*[HS])|([HS].*[YMD])", 'UTF-8')),
         :time         => Regexp.new(client("[hms]", 'UTF-8')),
         :number       => Regexp.new(client("([\#]|0+)", 'UTF-8')),
-        :locale       => Regexp.new(client(/\A\[\$\-\d+\]/.to_s, 'UTF-8')),
+        :locale       => Regexp.new(client(/\A\[\$\-\S+\]/.to_s, 'UTF-8')),
       }
 
       # Temp code to prevent merged formats in non-merged cells.

data/lib/spreadsheet/version.rb

@@ -0,0 +1,7 @@
+# frozen_string_literal: true
+
+module Spreadsheet
+  ##
+  # The version of Spreadsheet you are using.
+  VERSION = '1.3.0'
+end

data/lib/spreadsheet/workbook.rb

@@ -76,7 +76,7 @@ module Spreadsheet
     # Returns the count of total worksheets present.
     # Takes no arguments. Just returns the length of @worksheets array.
     def sheet_count
-    @worksheets.length
+      @worksheets.length
     end
     ##
     # The Font at _idx_

data/lib/spreadsheet/worksheet.rb

@@ -49,7 +49,7 @@ module Spreadsheet
         :right => 0.75,
         :bottom => 1
       }
-      @name = opts[:name] || 'Worksheet'
+      @name = sanitize_invalid_characters(opts[:name] || 'Worksheet')
       @workbook = opts[:workbook]
       @rows = []
       @columns = []
@@ -153,9 +153,13 @@ module Spreadsheet
     # If the argument skip is given, #each iterates from that row until but
     # omitting the first unused Row, effectively skipping the first _skip_ Rows
     # from the top of the Worksheet.
-    def each skip=dimensions[0]
-      skip.upto(dimensions[1] - 1) do |idx|
-        yield row(idx)
+    def each(skip=dimensions[0], &block)
+      rows = skip.upto(dimensions[1] - 1).map { |index| row(index) }.to_enum
+
+      if block_given?
+        rows.each(&block)
+      else
+        rows
       end
     end
     def encoding # :nodoc:
@@ -228,6 +232,9 @@ module Spreadsheet
       updated_from idx
       res
     end
+    def << cells=[]
+      insert_row @rows.size, cells
+    end
     def inspect
       names = instance_variables
       names.delete '@rows'
@@ -314,7 +321,7 @@ module Spreadsheet
 
     def compact!
       recalculate_dimensions
-      
+
       # detect first non-nil non-empty row if given first row is empty or nil
       if row(@dimensions[0]).empty? || row(@dimensions[0]).compact.join('').empty?
         (@dimensions[0]...@dimensions[1]).each do |i|
@@ -325,19 +332,19 @@ module Spreadsheet
 
       # detect last non-nil non-empty row if given last row is empty or nil
       if row(@dimensions[1] - 1).empty? || row(@dimensions[1] - 1).compact.join('').empty?
-        i = @dimensions[1] - 1
+        row_size = @dimensions[1] - 1
         @dimensions[1] = @dimensions[0]
         # divide and conquer
-        while(i - @dimensions[1] > 1) do
-          if row(i).empty? || row(i).compact.join('').empty?
-            i = @dimensions[1] + (((i - @dimensions[1]) + 1) / 2).to_i
+        while(row_size - @dimensions[1] > 1) do
+          if row(row_size).empty? || row(row_size).compact.join('').empty?
+            row_size = @dimensions[1] + (((row_size - @dimensions[1]) + 1) / 2).to_i
           else
-            _i = ((i - @dimensions[1]) / 2).to_i + 1
-            @dimensions[1] = i
-            i = i + _i
+            _i = ((row_size - @dimensions[1]) / 2).to_i + 1
+            @dimensions[1] = row_size
+            row_size = row_size + _i
           end
         end
-        @dimensions[1] = i + 1
+        @dimensions[1] = row_size + 1
       end
 
       # detect first non-empty non-nil column if first column is empty or nil
@@ -359,6 +366,9 @@ module Spreadsheet
     end
 
     private
+    def sanitize_invalid_characters(name) # :nodoc:
+      name.gsub(Regexp.new('[\\\/\*\?\:\[\]]'.encode(Spreadsheet.client_encoding)), '_')
+    end
     def index_of_first ary # :nodoc:
       return unless ary
       ary.index(ary.find do |elm| elm end)

data/lib/spreadsheet.rb

@@ -25,6 +25,7 @@
 #             8006 Zürich
 ###           Switzerland
 
+require 'spreadsheet/version'
 require 'spreadsheet/errors'
 
 require 'spreadsheet/excel/workbook'
@@ -44,10 +45,6 @@ require 'spreadsheet/excel/rgb'
 #  sheet.each do |row| puts row[0] end
 module Spreadsheet
 
-  ##
-  # The version of Spreadsheet you are using.
-  VERSION = '1.1.5'
-
   ##
   # Default client Encoding. Change this value if your application uses a
   # different Encoding:

data/test/excel/reader.rb

@@ -25,6 +25,19 @@ module Spreadsheet
           reader.setup not_empty_io
         end
       end
+
+      def test_not_frozen_stream_error_on_setup
+        return if Gem::Version.new(RUBY_VERSION) < Gem::Version.new('2.3.0')
+
+        reader = Spreadsheet::Excel::Reader.new
+        data = File.expand_path File.join('test', 'data')
+        path = File.join data, 'test_empty.xls'
+        content_string = File.read path
+        frozen_io = StringIO.new(content_string.freeze)
+        assert_nothing_thrown do
+          reader.setup(frozen_io)
+        end
+      end
     end
   end
 end

data/test/format.rb

@@ -25,6 +25,8 @@ module Spreadsheet
       assert_equal false, @format.date?
       @format.number_format = "0.00;[RED]\\-0.00"
       assert_equal false, @format.date?
+      @format.number_format = "[$-C0A]dd\\-mmm\\-yy"
+      assert_equal true, @format.date?
     end
     def test_date_or_time?
       assert_equal false, @format.date_or_time?

data/test/integration.rb

@@ -36,7 +36,7 @@ module Spreadsheet
     end
     def teardown
       Spreadsheet.client_encoding = 'UTF-8'
-      FileUtils.rm_r @var
+      FileUtils.rm_rf @var
     end
     def test_copy__identical__file_paths
       path = File.join @data, 'test_copy.xls'
@@ -70,7 +70,7 @@ module Spreadsheet
     def test_missing_format
       path = File.join @data, 'test_missing_format.xls'
       assert_nothing_thrown do
-        workbook = Spreadsheet.open(path, "rb")
+        Spreadsheet.open(path, "rb")
       end
     end
     def test_version_excel97__excel2010__utf16
@@ -506,7 +506,7 @@ module Spreadsheet
       assert_equal 3, sheets.size
       sheet = book.worksheet 0
       assert_instance_of Excel::Worksheet, sheet
-      str = "S\000h\000e\000e\000t\0001\000"
+      str = "S\000h\000e\000e\000t\0001\000".dup
       if RUBY_VERSION >= '1.9'
         str.force_encoding 'UTF-16LE' if str.respond_to?(:force_encoding)
       end
@@ -730,7 +730,7 @@ module Spreadsheet
       book = Spreadsheet::Excel::Workbook.new
       sheet = book.create_worksheet :name => 'My Worksheet'
       sheet[0,0] = 'my cell'
-      data = StringIO.new ''
+      data = StringIO.new ''.dup
       assert_nothing_raised do
         book.write data
       end
@@ -749,7 +749,7 @@ module Spreadsheet
       sheet1 = book.create_worksheet
       str1 = 'My Shared String'
       str2 = 'Another Shared String'
-      assert_equal 1, (str1.size + str2.size) % 2, 
+      assert_equal 1, (str1.size + str2.size) % 2,
         "str3 should start at an odd offset to test splitting of wide strings"
       str3 = '–––––––––– ' * 1000
       str4 = '1234567890 ' * 1000
@@ -829,7 +829,7 @@ module Spreadsheet
       assert_equal 2, book.worksheets.size
       sheet = book.worksheets.first
       assert_instance_of Spreadsheet::Excel::Worksheet, sheet
-      name = "W\000o\000r\000k\000s\000h\000e\000e\000t\0001\000"
+      name = "W\000o\000r\000k\000s\000h\000e\000e\000t\0001\000".dup
       name.force_encoding 'UTF-16LE' if name.respond_to?(:force_encoding)
       assert_equal name, sheet.name
       assert_not_nil sheet.offset
@@ -926,7 +926,7 @@ module Spreadsheet
       assert_equal 40, sheet1.row(11).height
       assert_instance_of Spreadsheet::Excel::Worksheet, sheet
       sheet = book.worksheets.last
-      name = "m\000y\000 \000n\000a\000m\000e\000"
+      name = "m\000y\000 \000n\000a\000m\000e\000".dup
       name.force_encoding 'UTF-16LE' if name.respond_to?(:force_encoding)
       assert_equal name, sheet.name
       assert_not_nil sheet.offset
@@ -1211,8 +1211,8 @@ module Spreadsheet
       book = Spreadsheet::Workbook.new
       sheet = book.create_worksheet
       (0..200).each { |i| sheet.row(i).push "ëçáéíóú" }
-      assert_nothing_raised do 
-        book.write StringIO.new("", "w+") 
+      assert_nothing_raised do
+        book.write StringIO.new("".dup, "w+")
       end
     end
 
@@ -1257,7 +1257,7 @@ module Spreadsheet
       str1 = "Frozen String.".freeze
       sheet1[0,0] = str1
       sheet1.row(0).push str1
-      assert_nothing_raised do 
+      assert_nothing_raised do
         book.write path
       end
     end
@@ -1290,7 +1290,7 @@ module Spreadsheet
       sheet1 = book.create_worksheet
       (sheet1.row(0).format 0).border = :hair
       (sheet1.row(0).format 0).border_color = :brown
-      assert_nothing_raised do 
+      assert_nothing_raised do
         book.write path
       end
       book2 = Spreadsheet.open path
@@ -1391,7 +1391,7 @@ module Spreadsheet
       path = File.join @var, 'test_write_worksheet_visibility.xls'
       sheet1 = book.create_worksheet
       sheet1.visibility = :hidden
-      sheet2 = book.create_worksheet
+      _sheet2 = book.create_worksheet
       assert_nothing_raised do
         book.write path
       end
@@ -1401,6 +1401,30 @@ module Spreadsheet
       assert_equal(:visible, book2.worksheet(1).visibility)
     end
 
+    def test_append_and_reopen
+      filename = path = File.join @var, 'test.xls'
+      sheet_name = 'Test Sheet'
+
+      # Create excel
+      excel = Spreadsheet::Workbook.new(filename)
+      sheet = excel.create_worksheet(name: sheet_name)
+      sheet.row(1).replace ['Data']
+      excel.write(filename)
+
+      # Append something
+      excel = Spreadsheet.open(filename, 'a+')
+      sheet = excel.worksheet(sheet_name)
+      sheet.row(2).replace ['Data2']
+      filename = "test2.xls"
+      excel.write(filename)
+
+      # Reopen
+      excel = Spreadsheet.open filename
+      sheet = excel.worksheet sheet_name
+    ensure
+      File.delete filename if File.exist? filename
+    end
+
     def test_text_drawing
       path = File.join @data, 'test_text_drawing.xls'
       book = Spreadsheet.open path

data/test/suite.rb

@@ -3,6 +3,8 @@
 require 'rubygems'
 require 'bundler'
 require 'find'
+require 'simplecov'
+SimpleCov.start
 
 $VERBOSE = true
 

data/test/worksheet.rb

@@ -108,18 +108,40 @@ module Spreadsheet
       assert_equal 6, @book.formats.length
 
     end
-    
+
     def test_freeze_panel!
       assert_equal 0, @sheet.froze_top
       assert_equal 0, @sheet.froze_left
       assert_equal false, @sheet.has_frozen_panel?
-      
+
       @sheet.freeze!(2, 3)
       assert_equal 2, @sheet.froze_top
       assert_equal 3, @sheet.froze_left
       assert_equal true, @sheet.has_frozen_panel?
-      
+
+    end
+
+    def test_each_with_skip
+      @sheet[0, 0] = 'foo'
+      @sheet[1, 0] = 'bar'
+
+      assert_equal @sheet.each(1).count, 1
+      assert_equal @sheet.each(1).first[0], 'bar'
     end
-    
+
+    def test_each_with_index
+      @sheet[0, 0] = 'foo'
+      @sheet[1, 0] = 'bar'
+
+      @sheet.each.with_index do |row, index|
+        assert_equal row[0], @sheet[index, 0]
+      end
+    end
+
+    def test_name
+      worksheet = Worksheet.new(name: '\a/b*c?d:e[f]')
+      assert_equal '_a_b_c_d_e_f_', worksheet.name
+    end
+
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: spreadsheet
 version: !ruby/object:Gem::Version
-  version: 1.1.5
+  version: 1.3.0
 platform: ruby
 authors:
-- Masaomi Hatakeyama, Zeno R.R. Davatz
+- Hannes F. Wyss, Masaomi Hatakeyama, Zeno R.R. Davatz
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-11-20 00:00:00.000000000 Z
+date: 2021-09-23 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: ruby-ole
@@ -16,72 +16,72 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '1.0'
+        version: '0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '1.0'
+        version: '0'
 - !ruby/object:Gem::Dependency
-  name: rdoc
+  name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: test-unit
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '4.0'
+        version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '4.0'
+        version: '0'
 - !ruby/object:Gem::Dependency
-  name: hoe
+  name: simplecov
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '3.14'
+        version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '3.14'
-description: |-
-  The Spreadsheet Library is designed to read and write Spreadsheet Documents.
-  As of version 0.6.0, only Microsoft Excel compatible spreadsheets are
-  supported. Spreadsheet is a combination/complete rewrite of the
-  Spreadsheet::Excel Library by Daniel J. Berger and the ParseExcel Library by
-  Hannes Wyss. Spreadsheet can read, write and modify Spreadsheet Documents.
-email: zdavatz@ywesee.com
+        version: '0'
+description: As of version 0.6.0, only Microsoft Excel compatible spreadsheets are
+  supported
+email: hannes.wyss@gmail.com, mhatakeyama@ywesee.com, zdavatz@ywesee.com
 executables:
 - xlsopcodes
 extensions: []
-extra_rdoc_files:
-- GUIDE.md
-- History.md
-- LICENSE.txt
-- Manifest.txt
-- README.md
+extra_rdoc_files: []
 files:
-- ".gitignore"
-- ".travis.yml"
-- Excel97-2007BinaryFileFormatSpecification.pdf
-- GUIDE.md
-- Gemfile
-- Gemfile.lock
-- History.md
 - LICENSE.txt
 - Manifest.txt
-- README.md
-- Rakefile
+- bin/bundle
+- bin/bundler
+- bin/oletool
+- bin/rake
+- bin/sow
 - bin/xlsopcodes
-- excelfileformat.pdf
 - lib/parseexcel.rb
 - lib/parseexcel/parseexcel.rb
 - lib/parseexcel/parser.rb
@@ -120,14 +120,16 @@ files:
 - lib/spreadsheet/note.rb
 - lib/spreadsheet/noteObject.rb
 - lib/spreadsheet/row.rb
+- lib/spreadsheet/version.rb
 - lib/spreadsheet/workbook.rb
 - lib/spreadsheet/worksheet.rb
 - lib/spreadsheet/writer.rb
-- spreadsheet.gemspec
 - test/data/test_adding_data_to_existing_file.xls
 - test/data/test_borders.xls
 - test/data/test_changes.xls
 - test/data/test_comment.xls
+- test/data/test_compact_format_date.xls
+- test/data/test_compact_many_rows.xls
 - test/data/test_copy.xls
 - test/data/test_datetime.xls
 - test/data/test_empty.xls
@@ -136,8 +138,10 @@ files:
 - test/data/test_margin.xls
 - test/data/test_merged_and_protected.xls
 - test/data/test_merged_cells.xls
+- test/data/test_missing_format.xls
 - test/data/test_missing_row.xls
 - test/data/test_pagesetup.xls
+- test/data/test_sizes.xls
 - test/data/test_text_drawing.xls
 - test/data/test_version_excel5.xls
 - test/data/test_version_excel95.xls
@@ -156,14 +160,12 @@ files:
 - test/workbook.rb
 - test/workbook_protection.rb
 - test/worksheet.rb
-homepage: https://github.com/zdavatz/spreadsheet
+homepage: https://github.com/zdavatz/spreadsheet/
 licenses:
 - GPL-3.0
 metadata: {}
 post_install_message: 
-rdoc_options:
-- "--main"
-- README.md
+rdoc_options: []
 require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
@@ -177,9 +179,9 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.6.8
+rubygems_version: 3.2.3
 signing_key: 
 specification_version: 4
 summary: The Spreadsheet Library is designed to read and write Spreadsheet Documents
-test_files: []
+test_files:
+- test/suite.rb

data/.gitignore

@@ -1,2 +0,0 @@
-tags
-*.swp

data/.travis.yml

@@ -1,36 +0,0 @@
-language: ruby
-sudo: false
-cache: bundler
-before_install:
-  - gem install bundler
-bundler_args: --binstubs
-script: "bundle exec ruby test/suite.rb"
-rvm:
-  - ruby-head
-  - 2.3.3
-  - 2.2.6
-  - 2.1.10
-  - 2.0.0
-  - 1.9.3
-  - 1.9.2
-  - 1.8.7
-  - rbx-19mode
-  - rbx-18mode
-  - jruby-head
-  - jruby-19mode
-  - jruby-18mode
-  - ree
-matrix:
-  allow_failures:
-    - rvm: ruby-head
-    - rvm: rbx-19mode
-    - rvm: rbx-18mode
-    - rvm: jruby-head
-    - rvm: jruby-19mode
-    - rvm: jruby-18mode
-    - rvm: ree
-notifications:
-  email:
-    recipients:
-      - yasaka@ywesee.com
-      - zdavatz@ywesee.com