spreadsheet 1.3.4 → 1.3.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 992768cae61da12d700181640e3968fd5e10110bc04d0cc575820b0f63d6030e
-  data.tar.gz: 682b9a07e067b830c9f78889c15f8c8d70c0f94caa2e91d5e5743beed731cb01
+  metadata.gz: f244d6b1694de3b2590286a2b047de226f36cd855dd534a2db47d221da62ec59
+  data.tar.gz: e7043746b2683555f602be54ad7e3b68495e5bf7e9abaae5df5c839c139ecc74
 SHA512:
-  metadata.gz: aad8b496a030e98f5e7493c98f80ae090f15a7b2f80925217d189ec045fdd0bb61881bd8780c913a21e4eb3c481a477119d7a639a2253bc7ca918f5c48bee21e
-  data.tar.gz: 96c1b47b1cb6bafdc0afd57e10e84df7d2aaf2eb5859f1557ffedfee135450d009db1f3805cf3c0fbf7c7bd2c1a1e2ab91cfc15b5af5e3445d315faa889ae5aa
+  metadata.gz: 97e1b3679d5693891b5a3caba131de68cbac2783067ebd4d318c457db924008b0100209a4d218ebd4357f80bf4cb70b8a1482ae5ea51fec1ec7fd23ccef89fd7
+  data.tar.gz: eee493446105db70f89be23210bafff8683c068cb1a0e319fe4ab356aa0af2ce4904d5e545e79eae4a15af898c9d5b69df9f6e0307a07f94ee5d3d7d09d9b083

data/bin/irb

@@ -0,0 +1,29 @@
+#!/usr/bin/env ruby
+# frozen_string_literal: true
+
+#
+# This file was generated by Bundler.
+#
+# The application 'irb' is installed as part of a gem, and
+# this file is here to facilitate running it.
+#
+
+require "pathname"
+ENV["BUNDLE_GEMFILE"] ||= File.expand_path("../../Gemfile",
+  Pathname.new(__FILE__).realpath)
+
+bundle_binstub = File.expand_path("../bundle", __FILE__)
+
+if File.file?(bundle_binstub)
+  if File.read(bundle_binstub, 300) =~ /This file was generated by Bundler/
+    load(bundle_binstub)
+  else
+    abort("Your `bin/bundle` was not generated by Bundler, so this binstub cannot run.
+Replace `bin/bundle` by running `bundle binstubs bundler --force`, then run this command again.")
+  end
+end
+
+require "rubygems"
+require "bundler/setup"
+
+load Gem.bin_path("irb", "irb")

data/bin/racc

@@ -0,0 +1,29 @@
+#!/usr/bin/env ruby
+# frozen_string_literal: true
+
+#
+# This file was generated by Bundler.
+#
+# The application 'racc' is installed as part of a gem, and
+# this file is here to facilitate running it.
+#
+
+require "pathname"
+ENV["BUNDLE_GEMFILE"] ||= File.expand_path("../../Gemfile",
+  Pathname.new(__FILE__).realpath)
+
+bundle_binstub = File.expand_path("../bundle", __FILE__)
+
+if File.file?(bundle_binstub)
+  if File.read(bundle_binstub, 300) =~ /This file was generated by Bundler/
+    load(bundle_binstub)
+  else
+    abort("Your `bin/bundle` was not generated by Bundler, so this binstub cannot run.
+Replace `bin/bundle` by running `bundle binstubs bundler --force`, then run this command again.")
+  end
+end
+
+require "rubygems"
+require "bundler/setup"
+
+load Gem.bin_path("racc", "racc")

data/bin/rdbg

@@ -0,0 +1,29 @@
+#!/usr/bin/env ruby
+# frozen_string_literal: true
+
+#
+# This file was generated by Bundler.
+#
+# The application 'rdbg' is installed as part of a gem, and
+# this file is here to facilitate running it.
+#
+
+require "pathname"
+ENV["BUNDLE_GEMFILE"] ||= File.expand_path("../../Gemfile",
+  Pathname.new(__FILE__).realpath)
+
+bundle_binstub = File.expand_path("../bundle", __FILE__)
+
+if File.file?(bundle_binstub)
+  if File.read(bundle_binstub, 300) =~ /This file was generated by Bundler/
+    load(bundle_binstub)
+  else
+    abort("Your `bin/bundle` was not generated by Bundler, so this binstub cannot run.
+Replace `bin/bundle` by running `bundle binstubs bundler --force`, then run this command again.")
+  end
+end
+
+require "rubygems"
+require "bundler/setup"
+
+load Gem.bin_path("debug", "rdbg")

data/bin/rdoc

@@ -0,0 +1,29 @@
+#!/usr/bin/env ruby
+# frozen_string_literal: true
+
+#
+# This file was generated by Bundler.
+#
+# The application 'rdoc' is installed as part of a gem, and
+# this file is here to facilitate running it.
+#
+
+require "pathname"
+ENV["BUNDLE_GEMFILE"] ||= File.expand_path("../../Gemfile",
+  Pathname.new(__FILE__).realpath)
+
+bundle_binstub = File.expand_path("../bundle", __FILE__)
+
+if File.file?(bundle_binstub)
+  if File.read(bundle_binstub, 300) =~ /This file was generated by Bundler/
+    load(bundle_binstub)
+  else
+    abort("Your `bin/bundle` was not generated by Bundler, so this binstub cannot run.
+Replace `bin/bundle` by running `bundle binstubs bundler --force`, then run this command again.")
+  end
+end
+
+require "rubygems"
+require "bundler/setup"
+
+load Gem.bin_path("rdoc", "rdoc")

data/bin/ri

@@ -0,0 +1,29 @@
+#!/usr/bin/env ruby
+# frozen_string_literal: true
+
+#
+# This file was generated by Bundler.
+#
+# The application 'ri' is installed as part of a gem, and
+# this file is here to facilitate running it.
+#
+
+require "pathname"
+ENV["BUNDLE_GEMFILE"] ||= File.expand_path("../../Gemfile",
+  Pathname.new(__FILE__).realpath)
+
+bundle_binstub = File.expand_path("../bundle", __FILE__)
+
+if File.file?(bundle_binstub)
+  if File.read(bundle_binstub, 300) =~ /This file was generated by Bundler/
+    load(bundle_binstub)
+  else
+    abort("Your `bin/bundle` was not generated by Bundler, so this binstub cannot run.
+Replace `bin/bundle` by running `bundle binstubs bundler --force`, then run this command again.")
+  end
+end
+
+require "rubygems"
+require "bundler/setup"
+
+load Gem.bin_path("rdoc", "ri")

data/bin/rubocop

@@ -0,0 +1,29 @@
+#!/usr/bin/env ruby
+# frozen_string_literal: true
+
+#
+# This file was generated by Bundler.
+#
+# The application 'rubocop' is installed as part of a gem, and
+# this file is here to facilitate running it.
+#
+
+require "pathname"
+ENV["BUNDLE_GEMFILE"] ||= File.expand_path("../../Gemfile",
+  Pathname.new(__FILE__).realpath)
+
+bundle_binstub = File.expand_path("../bundle", __FILE__)
+
+if File.file?(bundle_binstub)
+  if File.read(bundle_binstub, 300) =~ /This file was generated by Bundler/
+    load(bundle_binstub)
+  else
+    abort("Your `bin/bundle` was not generated by Bundler, so this binstub cannot run.
+Replace `bin/bundle` by running `bundle binstubs bundler --force`, then run this command again.")
+  end
+end
+
+require "rubygems"
+require "bundler/setup"
+
+load Gem.bin_path("rubocop", "rubocop")

data/bin/ruby-parse

@@ -0,0 +1,29 @@
+#!/usr/bin/env ruby
+# frozen_string_literal: true
+
+#
+# This file was generated by Bundler.
+#
+# The application 'ruby-parse' is installed as part of a gem, and
+# this file is here to facilitate running it.
+#
+
+require "pathname"
+ENV["BUNDLE_GEMFILE"] ||= File.expand_path("../../Gemfile",
+  Pathname.new(__FILE__).realpath)
+
+bundle_binstub = File.expand_path("../bundle", __FILE__)
+
+if File.file?(bundle_binstub)
+  if File.read(bundle_binstub, 300) =~ /This file was generated by Bundler/
+    load(bundle_binstub)
+  else
+    abort("Your `bin/bundle` was not generated by Bundler, so this binstub cannot run.
+Replace `bin/bundle` by running `bundle binstubs bundler --force`, then run this command again.")
+  end
+end
+
+require "rubygems"
+require "bundler/setup"
+
+load Gem.bin_path("parser", "ruby-parse")

data/bin/ruby-rewrite

@@ -0,0 +1,29 @@
+#!/usr/bin/env ruby
+# frozen_string_literal: true
+
+#
+# This file was generated by Bundler.
+#
+# The application 'ruby-rewrite' is installed as part of a gem, and
+# this file is here to facilitate running it.
+#
+
+require "pathname"
+ENV["BUNDLE_GEMFILE"] ||= File.expand_path("../../Gemfile",
+  Pathname.new(__FILE__).realpath)
+
+bundle_binstub = File.expand_path("../bundle", __FILE__)
+
+if File.file?(bundle_binstub)
+  if File.read(bundle_binstub, 300) =~ /This file was generated by Bundler/
+    load(bundle_binstub)
+  else
+    abort("Your `bin/bundle` was not generated by Bundler, so this binstub cannot run.
+Replace `bin/bundle` by running `bundle binstubs bundler --force`, then run this command again.")
+  end
+end
+
+require "rubygems"
+require "bundler/setup"
+
+load Gem.bin_path("parser", "ruby-rewrite")

data/bin/standardrb

@@ -0,0 +1,29 @@
+#!/usr/bin/env ruby
+# frozen_string_literal: true
+
+#
+# This file was generated by Bundler.
+#
+# The application 'standardrb' is installed as part of a gem, and
+# this file is here to facilitate running it.
+#
+
+require "pathname"
+ENV["BUNDLE_GEMFILE"] ||= File.expand_path("../../Gemfile",
+  Pathname.new(__FILE__).realpath)
+
+bundle_binstub = File.expand_path("../bundle", __FILE__)
+
+if File.file?(bundle_binstub)
+  if File.read(bundle_binstub, 300) =~ /This file was generated by Bundler/
+    load(bundle_binstub)
+  else
+    abort("Your `bin/bundle` was not generated by Bundler, so this binstub cannot run.
+Replace `bin/bundle` by running `bundle binstubs bundler --force`, then run this command again.")
+  end
+end
+
+require "rubygems"
+require "bundler/setup"
+
+load Gem.bin_path("standard", "standardrb")

data/bin/test-unit

@@ -0,0 +1,29 @@
+#!/usr/bin/env ruby
+# frozen_string_literal: true
+
+#
+# This file was generated by Bundler.
+#
+# The application 'test-unit' is installed as part of a gem, and
+# this file is here to facilitate running it.
+#
+
+require "pathname"
+ENV["BUNDLE_GEMFILE"] ||= File.expand_path("../../Gemfile",
+  Pathname.new(__FILE__).realpath)
+
+bundle_binstub = File.expand_path("../bundle", __FILE__)
+
+if File.file?(bundle_binstub)
+  if File.read(bundle_binstub, 300) =~ /This file was generated by Bundler/
+    load(bundle_binstub)
+  else
+    abort("Your `bin/bundle` was not generated by Bundler, so this binstub cannot run.
+Replace `bin/bundle` by running `bundle binstubs bundler --force`, then run this command again.")
+  end
+end
+
+require "rubygems"
+require "bundler/setup"
+
+load Gem.bin_path("test-unit", "test-unit")

data/lib/spreadsheet/column.rb

@@ -20,7 +20,7 @@ module Spreadsheet
     class << self
       def updater *keys
         keys.each do |key|
-          unless instance_methods.include? "unupdated_#{key}="
+          unless method_defined?("unupdated_#{key}=")
             alias_method :"unupdated_#{key}=", :"#{key}="
             define_method :"#{key}=" do |value|
               send :"unupdated_#{key}=", value
@@ -33,6 +33,7 @@ module Spreadsheet
     end
     include Datatypes
     include Enumerable
+
     attr_accessor :width, :worksheet
     attr_reader :default_format, :idx
     boolean :hidden, :collapsed

data/lib/spreadsheet/datatypes.rb

@@ -6,6 +6,7 @@ module Spreadsheet
   # attributes (boolean, colors and enumerations)
   module Datatypes
     include Compatibility
+
     def self.append_features mod
       super
       mod.module_eval do
@@ -142,7 +143,7 @@ module Spreadsheet
             define_method :"#{key}=" do |arg|
               if arg
                 arg = aliases.fetch arg do
-                  aliases.fetch arg.to_s.downcase.gsub(/[ \-]/, "_").to_sym, arg
+                  aliases.fetch arg.to_s.downcase.gsub(/[ -]/, "_").to_sym, arg
                 end
                 if values.any? { |val| val === arg }
                   instance_variable_set(ivar_name(key), arg)

data/lib/spreadsheet/excel/offset.rb

@@ -7,6 +7,7 @@ module Spreadsheet
     # Considered internal and subject to change without notice.
     module Offset
       include Compatibility
+
       attr_reader :changes, :offsets
       def initialize *args
         super
@@ -19,9 +20,10 @@ module Spreadsheet
         mod.module_eval do
           class << self
             include Compatibility
+
             def offset *keys
               keys.each do |key|
-                attr_reader key unless instance_methods.include? method_name(key)
+                attr_reader key unless method_defined?(method_name(key))
                 define_method :"#{key}=" do |value|
                   @changes.store key, true
                   instance_variable_set ivar_name(key), value

data/lib/spreadsheet/excel/reader/biff8.rb

@@ -7,6 +7,7 @@ module Spreadsheet
       # of Excel grows and methods get moved here for disambiguation.
       module Biff8
         include Spreadsheet::Excel::Internals
+
         ##
         # When a String is too long for one Opcode, it is continued in a Continue
         # Opcode. Excel may reconsider compressing the remainder of the string.

data/lib/spreadsheet/excel/reader.rb

@@ -18,6 +18,7 @@ module Spreadsheet
     class Reader
       include Spreadsheet::Encodings
       include Spreadsheet::Excel::Internals
+
       ROW_BLOCK_OPS = {
         blank: true, boolerr: true, dbcell: true, formula: true,
         label: true, labelsst: true, mulblank: true, mulrk: true,
@@ -1308,6 +1309,15 @@ module Spreadsheet
           xf = (flags & 0x0fff0000) >> 16
           attrs.store :default_format, @workbook.format(xf)
         end
+        # When a ROW record claims no cells (`first_used` == `first_unused`) but
+        # `set_missing_row_address` already recorded a valid offset from cell
+        # records found earlier in the stream, preserve the original offset.
+        # Some XLS writers emit ROW records after cell data with zeroed column
+        # ranges; without this fix, `read_row` would seek to the wrong position.
+        if first_used == first_unused && (existing = worksheet.row_addresses[index])
+          attrs[:offset] = existing[:offset]
+          attrs[:row_block] = existing[:row_block]
+        end
         # TODO: Row spacing
         worksheet.set_row_address index, attrs
       end
@@ -1345,6 +1355,7 @@ module Spreadsheet
       def extend_internals version
         require "spreadsheet/excel/internals/biff%i" % version
         extend Internals.const_get("Biff%i" % version)
+
         ## spreadsheets may not include a codepage record.
         @workbook.encoding = encoding 850 if version < 8
       rescue LoadError

data/lib/spreadsheet/excel/sst_entry.rb

@@ -6,6 +6,7 @@ module Spreadsheet
     # Shared String Table Entry
     class SstEntry
       include Spreadsheet::Encodings
+
       attr_accessor :chars, :phonetic, :richtext, :flags, :available,
         :continued_chars, :wide
       def initialize opts = {}

data/lib/spreadsheet/excel/workbook.rb

@@ -11,6 +11,7 @@ module Spreadsheet
     class Workbook < Spreadsheet::Workbook
       include Spreadsheet::Encodings
       include Spreadsheet::Excel::Offset
+
       BIFF_VERSIONS = {
         0x000 => 2,
         0x007 => 2,

data/lib/spreadsheet/excel/worksheet.rb

@@ -10,8 +10,9 @@ module Spreadsheet
     # to use any of these.
     class Worksheet < Spreadsheet::Worksheet
       include Spreadsheet::Excel::Offset
+
       offset :dimensions
-      attr_reader :offset, :ole, :links, :guts, :notes
+      attr_reader :offset, :ole, :links, :guts, :notes, :row_addresses
       def initialize opts = {}
         @row_addresses = nil
         super

data/lib/spreadsheet/excel/writer/biff8.rb

@@ -9,6 +9,7 @@ module Spreadsheet
       # of Excel grows and methods get moved here for disambiguation.
       module Biff8
         include Spreadsheet::Encodings
+
         ##
         # Check whether the string _data_ can be compressed (i.e. every second byte
         # is a Null-byte) and perform compression.

data/lib/spreadsheet/excel/writer/format.rb

@@ -9,6 +9,7 @@ module Spreadsheet
       # This class encapsulates everything that is needed to write an XF record.
       class Format < DelegateClass Spreadsheet::Format
         include Spreadsheet::Excel::Internals
+
         def self.boolean *args
           args.each do |key|
             define_method key do

data/lib/spreadsheet/excel/writer/n_worksheet.rb

@@ -14,6 +14,7 @@ module Spreadsheet
         include Spreadsheet::Excel::Writer::Biff8
         include Spreadsheet::Excel::Internals
         include Spreadsheet::Excel::Internals::Biff8
+
         attr_reader :worksheet
         def initialize workbook, worksheet
           @workbook = workbook

data/lib/spreadsheet/excel/writer/workbook.rb

@@ -17,6 +17,7 @@ module Spreadsheet
       class Workbook < Spreadsheet::Writer
         include Spreadsheet::Excel::Writer::Biff8
         include Spreadsheet::Excel::Internals
+
         attr_reader :fonts, :date_base
         def initialize *args
           super

data/lib/spreadsheet/excel/writer/worksheet.rb

@@ -15,6 +15,7 @@ module Spreadsheet
         include Spreadsheet::Excel::Writer::Biff8
         include Spreadsheet::Excel::Internals
         include Spreadsheet::Excel::Internals::Biff8
+
         attr_reader :worksheet
         def initialize workbook, worksheet
           @workbook = workbook

data/lib/spreadsheet/excel.rb

@@ -32,7 +32,7 @@ module Spreadsheet
   end
 
   class Worksheet
-    unless instance_methods.include? "new_format_column"
+    unless method_defined?(:new_format_column)
       alias_method :new_format_column, :format_column
       def format_column column, width = nil, format = nil
         if width.is_a? Format

data/lib/spreadsheet/font.rb

@@ -7,6 +7,7 @@ module Spreadsheet
   class Font
     include Spreadsheet::Datatypes
     include Spreadsheet::Encodings
+
     attr_accessor :name
     ##
     # You can set the following boolean Font attributes

data/lib/spreadsheet/format.rb

@@ -8,6 +8,7 @@ module Spreadsheet
   class Format
     include Spreadsheet::Datatypes
     include Spreadsheet::Encodings
+
     ##
     # You can set the following boolean attributes:
     # #cross_down::       Draws a Line from the top-left to the bottom-right

data/lib/spreadsheet/link.rb

@@ -19,6 +19,7 @@ module Spreadsheet
   #                  but if you do, here is where you can find it.
   class Link < String
     include Encodings
+
     attr_accessor :target_frame, :url, :dos, :fragment
     def initialize url = "", description = url, fragment = nil
       super(description)

data/lib/spreadsheet/note.rb

@@ -10,6 +10,7 @@ module Spreadsheet
   # #author  :: The name of the author who wrote the note
   class Note
     include Encodings
+
     attr_accessor :author, :length, :obj_id, :row, :col, :text
     def initialize
       @author = nil

data/lib/spreadsheet/noteObject.rb

@@ -8,6 +8,7 @@ module Spreadsheet
   #
   class NoteObject
     include Encodings
+
     attr_accessor :obj_id, :text
     def initialize
       @obj_id = -1

data/lib/spreadsheet/row.rb

@@ -18,10 +18,11 @@ module Spreadsheet
   # #height::         The height of this Row in points (defaults to 12).
   class Row < Array
     include Datatypes
+
     class << self
       def format_updater *keys
         keys.each do |key|
-          unless instance_methods.include? "unupdated_#{key}="
+          unless method_defined?("unupdated_#{key}=")
             alias_method :"unupdated_#{key}=", :"#{key}="
             define_method :"#{key}=" do |value|
               send :"unupdated_#{key}=", value

data/lib/spreadsheet/version.rb

@@ -3,5 +3,5 @@
 module Spreadsheet
   ##
   # The version of Spreadsheet you are using.
-  VERSION = "1.3.4"
+  VERSION = "1.3.5"
 end

data/lib/spreadsheet/workbook.rb

@@ -12,6 +12,7 @@ module Spreadsheet
   #                   Row#default_format or Worksheet#default_format.
   class Workbook
     include Spreadsheet::Encodings
+
     attr_reader :io, :worksheets, :formats, :fonts, :palette
     attr_accessor :active_worksheet, :encoding, :default_format, :version
     def initialize io = nil, opts = {default_format: Format.new}

data/lib/spreadsheet/worksheet.rb

@@ -28,6 +28,7 @@ module Spreadsheet
     include Spreadsheet::Encodings
     include Spreadsheet::Datatypes
     include Enumerable
+
     attr_accessor :name, :selected, :workbook, :password_hash
     attr_reader :rows, :columns, :merged_cells, :margins, :pagesetup
     attr_reader :froze_top, :froze_left

data/test/integration.rb

@@ -1217,6 +1217,20 @@ module Spreadsheet
       assert_not_nil sheet[2, 1]
     end
 
+    def test_row_record_with_empty_cell_range
+      # Some XLS writers emit ROW records with `first_used` == `first_unused` (claiming
+      # no cells) even though cell records (e.g. `LABELSST`) exist for that row
+      # earlier in the stream. Previously this caused `read_row` to seek to the wrong
+      # offset, returning an empty row despite valid cell data being present.
+      path = File.join @data, "test_row_record_empty_range.xls"
+      book = Spreadsheet.open path
+      sheet = book.worksheet 0
+      row0 = sheet.row(0).to_a.compact
+      assert_operator row0.length, :>, 0, "Row 0 should not be empty"
+      assert_equal "Name", row0[0]
+      assert_equal ["Name", "Code", "Description", "Reference", "Date", "Quantity"], row0
+    end
+
     def test_changes
       path = File.join @data, "test_changes.xls"
       book = Spreadsheet.open path

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: spreadsheet
 version: !ruby/object:Gem::Version
-  version: 1.3.4
+  version: 1.3.5
 platform: ruby
 authors:
 - Hannes F. Wyss, Masaomi Hatakeyama, Zeno R.R. Davatz
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2025-03-29 00:00:00.000000000 Z
+date: 2026-04-21 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: logger
@@ -134,9 +134,19 @@ files:
 - Manifest.txt
 - bin/bundle
 - bin/bundler
+- bin/irb
 - bin/oletool
+- bin/racc
 - bin/rake
+- bin/rdbg
+- bin/rdoc
+- bin/ri
+- bin/rubocop
+- bin/ruby-parse
+- bin/ruby-rewrite
 - bin/sow
+- bin/standardrb
+- bin/test-unit
 - bin/xlsopcodes
 - lib/parseexcel.rb
 - lib/parseexcel/parseexcel.rb
@@ -197,6 +207,7 @@ files:
 - test/data/test_missing_format.xls
 - test/data/test_missing_row.xls
 - test/data/test_pagesetup.xls
+- test/data/test_row_record_empty_range.xls
 - test/data/test_sizes.xls
 - test/data/test_text_drawing.xls
 - test/data/test_version_excel5.xls