rubyXL 3.4.18 → 3.4.25

data/lib/rubyXL/objects/workbook.rb

@@ -1,4 +1,3 @@
-#require 'tmpdir'
 require 'date'
 require 'rubyXL/objects/ooxml_object'
 require 'rubyXL/objects/shared_strings'
@@ -14,7 +13,6 @@ require 'rubyXL/objects/extensions'
 require 'rubyXL/objects/external_links'
 
 module RubyXL
-
   # Eventually, the entire code for Workbook will be moved here.
 
   # http://www.datypic.com/sc/ooxml/e-ssml_fileVersion-1.html
@@ -37,7 +35,7 @@ module RubyXL
 
   # http://www.datypic.com/sc/ooxml/e-ssml_workbookPr-1.html
   class WorkbookProperties < OOXMLObject
-    define_attribute(:date1904,                   :bool,   :default => false)
+    define_attribute(:date1904,                   :bool, :default => false)
     define_attribute(:showObjects,                RubyXL::ST_Objects, :default => 'all')
     define_attribute(:showBorderUnselectedTables, :bool,   :default => true)
     define_attribute(:filterPrivacy,              :bool,   :default => false)
@@ -152,7 +150,7 @@ module RubyXL
 
   # http://www.datypic.com/sc/ooxml/e-ssml_pivotCache-1.html
   class PivotCache < OOXMLObject
-    define_attribute(:cacheId, :int,    :required => true)
+    define_attribute(:cacheId, :int, :required => true)
     define_relationship(:required => true)
     define_element_name 'pivotCache'
   end
@@ -186,7 +184,7 @@ module RubyXL
     define_attribute(:vml,              :bool,   :default => false)
     define_attribute(:allowPng,         :bool,   :default => false)
     define_attribute(:targetScreenSize, RubyXL::ST_TargetScreenSize, :default => '800x600')
-    define_attribute(:dpi,              :int,    :default => 96)
+    define_attribute(:dpi,              :int, :default => 96)
     define_attribute(:codePage,         :int)
     define_element_name 'webPublishing'
   end
@@ -195,7 +193,7 @@ module RubyXL
   class CalculationProperties < OOXMLObject
     define_attribute(:calcId,                :int)
     define_attribute(:calcMode,              RubyXL::ST_CalcMode, :default => 'auto')
-    define_attribute(:fullCalcOnLoad,        :bool,   :default => false)
+    define_attribute(:fullCalcOnLoad,        :bool, :default => false)
     define_attribute(:refMode,               RubyXL::ST_RefMode, :default => 'A1')
     define_attribute(:iterate,               :bool,   :default => false)
     define_attribute(:iterateCount,          :int,    :default => 100)
@@ -216,7 +214,7 @@ module RubyXL
     define_attribute(:sourceObject,    :string)
     define_attribute(:destinationFile, :string, :required => true)
     define_attribute(:title,           :string)
-    define_attribute(:autoRepublish,   :bool,   :default => false)
+    define_attribute(:autoRepublish,   :bool, :default => false)
     define_element_name 'webPublishObject'
   end
 
@@ -228,7 +226,7 @@ module RubyXL
 
   # http://www.datypic.com/sc/ooxml/e-ssml_smartTagPr-1.html
   class SmartTagProperties < OOXMLObject
-    define_attribute(:embed, :bool,   :default => false)
+    define_attribute(:embed, :bool, :default => false)
     define_attribute(:show,  RubyXL::ST_SmartTagShow, :default => 'all')
     define_element_name 'smartTagPr'
   end
@@ -280,7 +278,7 @@ module RubyXL
     define_attribute(:yWindow,              :int,    :default => 0)
     define_attribute(:windowWidth,          :int)
     define_attribute(:windowHeight,         :int)
-    define_attribute(:tabRatio,             :int,    :default => 600)
+    define_attribute(:tabRatio,             :int, :default => 600)
     define_attribute(:activeSheetId,        :int)
     define_attribute(:showFormulaBar,       :bool,   :default => true)
     define_attribute(:showStatusbar,        :bool,   :default => true)
@@ -298,15 +296,17 @@ module RubyXL
 
   # http://www.datypic.com/sc/ooxml/e-ssml_workbook.html
   class Workbook < OOXMLTopLevelObject
-    CONTENT_TYPE = 'application/vnd.openxmlformats-officedocument.spreadsheetml.sheet.main+xml'
-    CONTENT_TYPE_TEMPLATE = 'application/vnd.openxmlformats-officedocument.spreadsheetml.template.main+xml'
-    CONTENT_TYPE_TEMPLATE_WITH_MACROS = 'application/vnd.ms-excel.template.macroEnabled.main+xml'
-    CONTENT_TYPE_WITH_MACROS = 'application/vnd.ms-excel.sheet.macroEnabled.main+xml'
-    REL_TYPE     = 'http://schemas.openxmlformats.org/officeDocument/2006/relationships/officeDocument'
+    CONTENT_TYPE = 'application/vnd.openxmlformats-officedocument.spreadsheetml.sheet.main+xml'.freeze
+    CONTENT_TYPE_TEMPLATE = 'application/vnd.openxmlformats-officedocument.spreadsheetml.template.main+xml'.freeze
+    CONTENT_TYPE_TEMPLATE_WITH_MACROS = 'application/vnd.ms-excel.template.macroEnabled.main+xml'.freeze
+    CONTENT_TYPE_WITH_MACROS = 'application/vnd.ms-excel.sheet.macroEnabled.main+xml'.freeze
+    REL_TYPE = 'http://schemas.openxmlformats.org/officeDocument/2006/relationships/officeDocument'.freeze
 
     # http://www.accountingweb.com/technology/excel/seven-characters-you-cant-use-in-worksheet-names
-    SHEET_NAME_FORBIDDEN_CHARS = /[\/\\\*\[\]\:\?]/
-    #SHEET_NAME_FORBIDDEN_NAMES = [ 'History' ]
+    SHEET_NAME_FORBIDDEN_CHARS = /[\/\\*\[\]:?]/
+
+    # https://answers.microsoft.com/en-us/msoffice/forum/all/excel-history-is-a-reserved-name-help/f8a398a4-b72c-48e3-82da-4f132c305e91
+    SHEET_NAME_FORBIDDEN_NAMES = [ 'History' ]
 
     include RubyXL::RelationshipSupport
 
@@ -332,7 +332,7 @@ module RubyXL
     define_relationship(RubyXL::PivotCacheDefinitionFile)
     define_relationship(RubyXL::PivotCacheRecordsFile)
     define_relationship(RubyXL::CustomXMLFile)
-    define_relationship(RubyXL::MacrosFile,         :macros)
+    define_relationship(RubyXL::MacrosFile, :macros)
     define_relationship(RubyXL::SlicerCacheFile)
 
     define_child_node(RubyXL::FileVersion)
@@ -358,13 +358,12 @@ module RubyXL
     define_child_node(RubyXL::ExtensionStorageArea)
 
     define_element_name 'workbook'
-    set_namespaces('http://schemas.openxmlformats.org/spreadsheetml/2006/main' => nil,
+    set_namespaces('http://schemas.openxmlformats.org/spreadsheetml/2006/main'           => nil,
                    'http://schemas.openxmlformats.org/officeDocument/2006/relationships' => 'r',
-                   'http://schemas.openxmlformats.org/markup-compatibility/2006' => 'mc',
-                   'http://schemas.microsoft.com/office/spreadsheetml/2010/11/main' => 'x15')
+                   'http://schemas.openxmlformats.org/markup-compatibility/2006'         => 'mc',
+                   'http://schemas.microsoft.com/office/spreadsheetml/2010/11/main'      => 'x15')
 
-    attr_accessor :worksheets
-    attr_accessor :is_template
+    attr_accessor :worksheets, :is_template
 
     def before_write_xml
       max_sheet_id = worksheets.collect(&:sheet_id).compact.max || 0
@@ -375,6 +374,7 @@ module RubyXL
         rel = relationship_container.find_by_target(sheet.xlsx_path)
 
         raise "Worksheet name '#{sheet.sheet_name}' contains forbidden characters" if sheet.sheet_name =~ SHEET_NAME_FORBIDDEN_CHARS
+        raise "Worksheet name '#{sheet.sheet_name}' is forbidden" if SHEET_NAME_FORBIDDEN_NAMES.include?(sheet.sheet_name)
 
         sheets << RubyXL::Sheet.new(:name     => sheet.sheet_name[0..30], # Max sheet name length is 31 char
                                     :sheet_id => sheet.sheet_id || (max_sheet_id += 1),
@@ -403,7 +403,7 @@ module RubyXL
         raise "Unsupported extension: #{extension} (only .xlsx, .xlsm, .xltx and .xltm files are supported)."
       end
 
-      File.open(dst_file_path, "wb") { |output_file| FileUtils.copy_stream(root.stream, output_file) }
+      File.open(dst_file_path, 'wb') { |output_file| FileUtils.copy_stream(root.stream, output_file) }
 
       return dst_file_path
     end
@@ -421,8 +421,8 @@ module RubyXL
 
     def date_to_num(date)
       case date
-      when Date, DateTime then (date.ajd - base_date().ajd).to_f
-      when Time then ((date.to_r - base_date().to_time.to_r) / 86400).to_f
+      when Date, DateTime then (date.ajd - base_date.ajd).to_f
+      when Time then ((date.to_r - base_date.to_time.to_r) / 86400).to_f
       end
     end
 
@@ -567,7 +567,5 @@ module RubyXL
       self.workbook_properties ||= RubyXL::WorkbookProperties.new
       workbook_properties.date1904 = v
     end
-
   end
-
 end

data/lib/rubyXL/objects/worksheet.rb

@@ -15,7 +15,6 @@ require 'rubyXL/objects/query_table'
 require 'rubyXL/worksheet'
 
 module RubyXL
-
   # Eventually, the entire code for Worksheet will be moved here. One small step at a time!
 
   # http://www.datypic.com/sc/ooxml/e-ssml_outlinePr-1.html
@@ -58,7 +57,7 @@ module RubyXL
   end
 
   class WorksheetFormatProperties < OOXMLObject
-    define_attribute(:baseColWidth,     :int,    :default => 8)
+    define_attribute(:baseColWidth,     :int, :default => 8)
     define_attribute(:defaultColWidth,  :double)
     define_attribute(:defaultRowHeight, :double, :required => true)
     define_attribute(:customHeight,     :bool,   :default => false)
@@ -83,7 +82,7 @@ module RubyXL
     define_attribute(:blackAndWhite,      :bool,   :default => false)
     define_attribute(:draft,              :bool,   :default => false)
     define_attribute(:cellComments,       RubyXL::ST_CellComments, :default => 'none')
-    define_attribute(:useFirstPageNumber, :bool,   :default => false)
+    define_attribute(:useFirstPageNumber, :bool, :default => false)
     define_attribute(:errors,             RubyXL::ST_PrintError, :default => 'displayed')
     define_attribute(:horizontalDpi,      :int,    :default => 600)
     define_attribute(:verticalDpi,        :int,    :default => 600)
@@ -166,7 +165,7 @@ module RubyXL
   class ConditionalFormatValue < OOXMLObject
     define_attribute(:type, RubyXL::ST_CfvoType, :required => true)
     define_attribute(:val,  :string)
-    define_attribute(:gte,  :bool,   :default => true)
+    define_attribute(:gte,  :bool, :default => true)
     define_child_node(RubyXL::ExtensionStorageArea)
     define_element_name 'cfvo'
   end
@@ -212,7 +211,7 @@ module RubyXL
     define_attribute(:timePeriod,   RubyXL::ST_TimePeriod)
     define_attribute(:rank,         :int)
     define_attribute(:stdDev,       :int)
-    define_attribute(:equalAverage, :bool,   :default  => false)
+    define_attribute(:equalAverage, :bool, :default => false)
     define_child_node(RubyXL::Formula, :collection => true, :node_name => :formula, :accessor => :formulas)
     define_child_node(RubyXL::ColorScale)
     define_child_node(RubyXL::DataBar)
@@ -301,7 +300,7 @@ module RubyXL
 
   # http://www.datypic.com/sc/ooxml/e-ssml_hyperlink-1.html
   class Hyperlink < OOXMLObject
-    define_attribute(:ref,      :ref,    :required => true)
+    define_attribute(:ref, :ref, :required => true)
     define_relationship
     define_attribute(:location, :string)
     define_attribute(:tooltip,  :string)
@@ -408,9 +407,9 @@ module RubyXL
     define_attribute(:hiddenRows,     :bool,   :default => false)
     define_attribute(:hiddenColumns,  :bool,   :default => false)
     define_attribute(:state,          RubyXL::ST_Visibility, :default => 'visible')
-    define_attribute(:filterUnique,   :bool,   :default => false)
+    define_attribute(:filterUnique,   :bool, :default => false)
     define_attribute(:view,           RubyXL::ST_SheetViewType, :default => 'normal')
-    define_attribute(:showRuler,      :bool,   :default => true)
+    define_attribute(:showRuler,      :bool, :default => true)
     define_attribute(:topLeftCell,    :ref)
     define_child_node(RubyXL::Pane)
     define_child_node(RubyXL::Selection)
@@ -433,9 +432,9 @@ module RubyXL
 
   # http://www.datypic.com/sc/ooxml/e-ssml_control-1.html
   class EmbeddedControl < OOXMLObject
-    define_attribute(:shapeId, :int,    :required => true)
+    define_attribute(:shapeId, :int, :required => true)
     define_relationship(:required => true)
-    define_attribute(:name,    :string)
+    define_attribute(:name, :string)
     define_element_name 'control'
   end
 
@@ -489,7 +488,7 @@ module RubyXL
 
   # http://www.datypic.com/sc/ooxml/e-ssml_customPr-1.html
   class CustomProperty < OOXMLObject
-    define_attribute(:name,   :string, :required => true)
+    define_attribute(:name, :string, :required => true)
     define_relationship(:required => true)
     define_element_name 'customPr'
   end
@@ -546,7 +545,7 @@ module RubyXL
     define_attribute(:cacheIndex,    :bool, :default => false)
     define_attribute(:outline,       :bool, :default => true)
     define_attribute(:offset,        :ref)
-    define_attribute(:collapsedLevelsAreSubtotals, :bool,   :default => false)
+    define_attribute(:collapsedLevelsAreSubtotals, :bool, :default => false)
     define_attribute(:axis,          RubyXL::ST_Axis)
     define_attribute(:fieldPosition, :int,  :default => 0)
     define_child_node(RubyXL::PivotReferences)
@@ -597,10 +596,10 @@ module RubyXL
     define_attribute(:zoomScaleNormal,          :uint, :default => 0)
     define_attribute(:zoomScaleSheetLayoutView, :uint, :default => 0)
     define_attribute(:zoomScalePageLayoutView,  :uint, :default => 0)
-    define_attribute(:workbookViewId,           :uint, :required => true, :default => 0 )
+    define_attribute(:workbookViewId,           :uint, :required => true, :default => 0)
     define_child_node(RubyXL::Pane)
     define_child_node(RubyXL::Selection, :collection => true, :accessor => :selections)
-    define_child_node(RubyXL::PivotTableSelection, :collection => true, :accessor => :pivot_table_selections )
+    define_child_node(RubyXL::PivotTableSelection, :collection => true, :accessor => :pivot_table_selections)
     define_child_node(RubyXL::ExtensionStorageArea)
     define_element_name 'sheetView'
   end
@@ -614,8 +613,8 @@ module RubyXL
 
   # http://www.datypic.com/sc/ooxml/e-ssml_worksheet.html
   class Worksheet < OOXMLTopLevelObject
-    CONTENT_TYPE = 'application/vnd.openxmlformats-officedocument.spreadsheetml.worksheet+xml'
-    REL_TYPE     = 'http://schemas.openxmlformats.org/officeDocument/2006/relationships/worksheet'
+    CONTENT_TYPE = 'application/vnd.openxmlformats-officedocument.spreadsheetml.worksheet+xml'.freeze
+    REL_TYPE     = 'http://schemas.openxmlformats.org/officeDocument/2006/relationships/worksheet'.freeze
 
     include RubyXL::RelationshipSupport
 
@@ -651,7 +650,7 @@ module RubyXL
     define_child_node(RubyXL::SortState)
     define_child_node(RubyXL::DataConsolidate)
     define_child_node(RubyXL::CustomSheetViews)
-    define_child_node(RubyXL::MergedCells,           :accessor => :merged_cells)
+    define_child_node(RubyXL::MergedCells, :accessor => :merged_cells)
     define_child_node(RubyXL::PhoneticProperties)
     define_child_node(RubyXL::ConditionalFormatting, :collection => [0..-1])
     define_child_node(RubyXL::DataValidations)
@@ -677,11 +676,11 @@ module RubyXL
     define_child_node(RubyXL::ExtensionStorageArea)
     define_child_node(RubyXL::AlternateContent)
     define_element_name 'worksheet'
-    set_namespaces('http://schemas.openxmlformats.org/spreadsheetml/2006/main' => nil,
+    set_namespaces('http://schemas.openxmlformats.org/spreadsheetml/2006/main'           => nil,
                    'http://schemas.openxmlformats.org/officeDocument/2006/relationships' => 'r',
-                   'http://schemas.openxmlformats.org/markup-compatibility/2006' => 'mc',
-                   'http://schemas.microsoft.com/office/spreadsheetml/2009/9/ac' => 'x14ac',
-                   'urn:schemas-microsoft-com:mac:vml' => 'mv')
+                   'http://schemas.openxmlformats.org/markup-compatibility/2006'         => 'mc',
+                   'http://schemas.microsoft.com/office/spreadsheetml/2009/9/ac'         => 'x14ac',
+                   'urn:schemas-microsoft-com:mac:vml'                                   => 'mv')
 
     attr_accessor :workbook, :state, :sheet_name, :sheet_id, :rels
 
@@ -752,7 +751,12 @@ module RubyXL
       workbook.stylesheet.cell_xfs[get_row_style(row)]
     end
 
+    def cell_at(ref)
+      reference = RubyXL::Reference.new(ref)
+      raise "Invalid reference: #{ref}" unless reference.valid? && reference.single_cell?
+      sheet_data&.rows&.[](reference.first_row)&.cells&.[](reference.first_col)
+    end
+
     include LegacyWorksheet
   end
-
 end

data/lib/rubyXL/parser.rb

@@ -1,6 +1,5 @@
 module RubyXL
   class Parser
-
     # Parse <tt>.xslx</tt> file by reading it from local disk.
     def self.parse(src_file_path)
       begin
@@ -18,12 +17,11 @@ module RubyXL
     def self.parse_buffer(buffer)
       root = nil # Zip::File.open_buffer somehow fails to return the value from the block :(
       begin
-        ::Zip::File.open_buffer(buffer) { |zip_file| root = RubyXL::WorkbookRoot.parse_zip_file(zip_file)  } 
+        ::Zip::File.open_buffer(buffer) { |zip_file| root = RubyXL::WorkbookRoot.parse_zip_file(zip_file) }
         root.workbook
       rescue ::Zip::Error => e
         raise e, "XLSX file format error: #{e}", e.backtrace
       end
     end
-
   end
 end

data/lib/rubyXL/worksheet.rb

@@ -1,98 +1,97 @@
 module RubyXL
-module LegacyWorksheet
-  include Enumerable
-
-  def initialize(params = {})
-    super
-    self.workbook   = params[:workbook]
-    self.sheet_name = params[:sheet_name]
-    self.sheet_id   = params[:sheet_id]
-    self.sheet_data = RubyXL::SheetData.new
-    self.cols = RubyXL::ColumnRanges.new
-    @comments = [] # Do not optimize! These are arrays, so they will share the pointer!
-    @printer_settings = []
-    @generic_storage = []
-  end
-
-  # allows for easier access to sheet_data
-  def [](row = 0)
-    sheet_data[row]
-  end
-
-  def each
-    sheet_data.rows.each { |row| yield(row) }
-  end
-
-  def add_row(row_index = 0, params = {})
-    new_row = RubyXL::Row.new(params)
-    new_row.worksheet = self
-    sheet_data.rows[row_index] = new_row
-  end
-
-  def add_cell(row_index = 0, column_index = 0, data = '', formula = nil, overwrite = true)
-    validate_workbook
-    validate_nonnegative(row_index)
-    validate_nonnegative(column_index)
-    row = sheet_data.rows[row_index] || add_row(row_index)
-
-    c = row.cells[column_index]
-
-    if overwrite || c.nil?
-      c = RubyXL::Cell.new
-      c.worksheet = self
-      c.row = row_index
-      c.column = column_index
-
-      if formula then
-        c.formula = RubyXL::Formula.new(:expression => formula)
-        c.raw_value = data
-      else
-        case data
-        when Numeric          then c.raw_value = data
-        when String           then
+  module LegacyWorksheet
+    include Enumerable
+
+    def initialize(params = {})
+      super
+      self.workbook   = params[:workbook]
+      self.sheet_name = params[:sheet_name]
+      self.sheet_id   = params[:sheet_id]
+      self.sheet_data = RubyXL::SheetData.new
+      self.cols = RubyXL::ColumnRanges.new
+      @comments = [] # Do not optimize! These are arrays, so they will share the pointer!
+      @printer_settings = []
+      @generic_storage = []
+    end
+
+    # allows for easier access to sheet_data
+    def [](row = 0)
+      sheet_data[row]
+    end
+
+    def each
+      sheet_data.rows.each { |row| yield(row) }
+    end
+
+    def add_row(row_index = 0, params = {})
+      new_row = RubyXL::Row.new(params)
+      new_row.worksheet = self
+      sheet_data.rows[row_index] = new_row
+    end
+
+    def add_cell(row_index = 0, column_index = 0, data = '', formula = nil, overwrite = true)
+      validate_workbook
+      validate_nonnegative(row_index)
+      validate_nonnegative(column_index)
+      row = sheet_data.rows[row_index] || add_row(row_index)
+
+      c = row.cells[column_index]
+
+      if overwrite || c.nil?
+        c = RubyXL::Cell.new
+        c.worksheet = self
+        c.row = row_index
+        c.column = column_index
+
+        if formula then
+          c.formula = RubyXL::Formula.new(:expression => formula)
           c.raw_value = data
-          c.datatype = RubyXL::DataType::RAW_STRING
-        when RubyXL::RichText then
-          c.is = data
-          c.datatype = RubyXL::DataType::INLINE_STRING
-        when Time, Date, DateTime then
-          c.raw_value = workbook.date_to_num(data)
-        when NilClass         then nil
+        else
+          case data
+          when Numeric          then c.raw_value = data
+          when String           then
+            c.raw_value = data
+            c.datatype = RubyXL::DataType::RAW_STRING
+          when RubyXL::RichText then
+            c.is = data
+            c.datatype = RubyXL::DataType::INLINE_STRING
+          when Time, Date, DateTime then
+            c.raw_value = workbook.date_to_num(data)
+          when NilClass then nil
+          end
         end
+
+        range = cols && cols.locate_range(column_index)
+        c.style_index = row.style_index || (range && range.style_index) || 0
+        row.cells[column_index] = c
       end
 
-      range = cols && cols.locate_range(column_index)
-      c.style_index = row.style_index || (range && range.style_index) || 0
-      row.cells[column_index] = c
+      c
     end
 
-    c
-  end
+    private
 
-  private
+    # validates Workbook, ensures that this worksheet is in @workbook
+    def validate_workbook
+      unless @workbook.nil? || @workbook.worksheets.nil?
+        return if @workbook.worksheets.any? { |sheet| sheet.equal?(self) }
+      end
 
-  #validates Workbook, ensures that this worksheet is in @workbook
-  def validate_workbook()
-    unless @workbook.nil? || @workbook.worksheets.nil?
-      return if @workbook.worksheets.any? { |sheet| sheet.equal?(self) }
+      raise "This worksheet #{self} is not in workbook #{@workbook}"
     end
 
-    raise "This worksheet #{self} is not in workbook #{@workbook}"
-  end
+    # Ensures that storage space for a cell with +row_index+ and +column_index+
+    # exists in +sheet_data+ arrays, growing them up if necessary.
+    def ensure_cell_exists(row_index, column_index = 0)
+      validate_nonnegative(row_index)
+      validate_nonnegative(column_index)
 
-  # Ensures that storage space for a cell with +row_index+ and +column_index+
-  # exists in +sheet_data+ arrays, growing them up if necessary.
-  def ensure_cell_exists(row_index, column_index = 0)
-    validate_nonnegative(row_index)
-    validate_nonnegative(column_index)
-
-    sheet_data.rows[row_index] || add_row(row_index)
-  end
-
-  def validate_nonnegative(row_or_col)
-    raise 'Row and Column arguments must be nonnegative' if row_or_col < 0
-  end
-  private :validate_nonnegative
+      sheet_data.rows[row_index] || add_row(row_index)
+    end
 
-end #end class
+    def validate_nonnegative(row_or_col)
+      raise 'Row and Column arguments must be nonnegative' if row_or_col < 0
+    end
+    private :validate_nonnegative
+  end # end class
 end

data/lib/rubyXL.rb

@@ -9,5 +9,4 @@ module RubyXL
     return path unless path.absolute?
     path.relative_path_from(OOXMLTopLevelObject::ROOT)
   end
-
 end