write_xlsx 1.09.4 → 1.09.5

data/lib/write_xlsx/worksheet/cell_data.rb

@@ -13,7 +13,7 @@ module Writexlsx
       # attributes for the <cell> element. This is the innermost loop so efficiency is
       # important where possible.
       #
-      def cell_attributes #:nodoc:
+      def cell_attributes # :nodoc:
         xf_index = xf ? xf.get_xf_index : 0
         attributes = [
           ['r', xl_rowcol_to_cell(row, col)]
@@ -40,7 +40,10 @@ module Writexlsx
     class NumberCellData < CellData # :nodoc:
       def initialize(worksheet, row, col, num, xf)
         @worksheet = worksheet
-        @row, @col, @token, @xf = row, col, num, xf
+        @row = row
+        @col = col
+        @token = num
+        @xf = xf
       end
 
       def data
@@ -57,14 +60,17 @@ module Writexlsx
     class StringCellData < CellData # :nodoc:
       def initialize(worksheet, row, col, index, xf)
         @worksheet = worksheet
-        @row, @col, @token, @xf = row, col, index, xf
+        @row = row
+        @col = col
+        @token = index
+        @xf = xf
       end
 
       def data
         { :sst_id => token }
       end
 
-      TYPE_STR_ATTRS = ['t', 's'].freeze
+      TYPE_STR_ATTRS = %w[t s].freeze
       def write_cell
         attributes = cell_attributes
         attributes << TYPE_STR_ATTRS
@@ -81,7 +87,11 @@ module Writexlsx
     class FormulaCellData < CellData # :nodoc:
       def initialize(worksheet, row, col, formula, xf, result)
         @worksheet = worksheet
-        @row, @col, @token, @xf, @result = row, col, formula, xf, result
+        @row = row
+        @col = col
+        @token = formula
+        @xf = xf
+        @result = result
       end
 
       def data
@@ -89,18 +99,18 @@ module Writexlsx
       end
 
       def write_cell
-        truefalse = {'TRUE' => 1, 'FALSE' => 0}
+        truefalse = { 'TRUE' => 1, 'FALSE' => 0 }
         error_code = ['#DIV/0!', '#N/A', '#NAME?', '#NULL!', '#NUM!', '#REF!', '#VALUE!']
 
         attributes = cell_attributes
-        if @result &&  !(@result.to_s =~ /^([+-]?)(?=\d|\.\d)\d*(\.\d*)?([Ee]([+-]?\d+))?$/)
+        if @result && !(@result.to_s =~ /^([+-]?)(?=\d|\.\d)\d*(\.\d*)?([Ee]([+-]?\d+))?$/)
           if truefalse[@result]
-            attributes << ['t', 'b']
+            attributes << %w[t b]
             @result = truefalse[@result]
           elsif error_code.include?(@result)
-            attributes << ['t', 'e']
+            attributes << %w[t e]
           else
-            attributes << ['t', 'str']
+            attributes << %w[t str]
           end
         end
         @worksheet.writer.tag_elements('c', attributes) do
@@ -113,7 +123,12 @@ module Writexlsx
     class FormulaArrayCellData < CellData # :nodoc:
       def initialize(worksheet, row, col, formula, xf, range, result)
         @worksheet = worksheet
-        @row, @col, @token, @xf, @range, @result = row, col, formula, xf, range, result
+        @row = row
+        @col = col
+        @token = formula
+        @xf = xf
+        @range = range
+        @result = result
       end
 
       def data
@@ -131,7 +146,12 @@ module Writexlsx
     class DynamicFormulaArrayCellData < CellData # :nodoc:
       def initialize(worksheet, row, col, formula, xf, range, result)
         @worksheet = worksheet
-        @row, @col, @token, @xf, @range, @result = row, col, formula, xf, range, result
+        @row = row
+        @col = col
+        @token = formula
+        @xf = xf
+        @range = range
+        @result = result
       end
 
       def data
@@ -141,7 +161,7 @@ module Writexlsx
       def write_cell
         # Add metadata linkage for dynamic array formulas.
         attributes = cell_attributes
-        attributes << ['cm', '1']
+        attributes << %w[cm 1]
 
         @worksheet.writer.tag_elements('c', attributes) do
           @worksheet.write_cell_array_formula(token, range)
@@ -153,7 +173,10 @@ module Writexlsx
     class BooleanCellData < CellData # :nodoc:
       def initialize(worksheet, row, col, val, xf)
         @worksheet = worksheet
-        @row, @col, @token, @xf = row, col, val, xf
+        @row = row
+        @col = col
+        @token = val
+        @xf = xf
       end
 
       def data
@@ -163,18 +186,19 @@ module Writexlsx
       def write_cell
         attributes = cell_attributes
 
-        attributes << ['t', 'b']
+        attributes << %w[t b]
         @worksheet.writer.tag_elements('c', attributes) do
           @worksheet.write_cell_value(token)
         end
       end
     end
 
-
     class BlankCellData < CellData # :nodoc:
       def initialize(worksheet, row, col, xf)
         @worksheet = worksheet
-        @row, @col, @xf = row, col, xf
+        @row = row
+        @col = col
+        @xf = xf
       end
 
       def data

data/lib/write_xlsx/worksheet/data_validation.rb

@@ -15,13 +15,15 @@ module Writexlsx
         row1, col1, row2, col2, options = row_col_notation(args)
         if row2.respond_to?(:keys)
           options_to_instance_variable(row2.dup)
-          row2, col2 = row1, col1
+          row2 = row1
+          col2 = col1
         elsif options.respond_to?(:keys)
           options_to_instance_variable(options.dup)
         else
           raise WriteXLSXInsufficientArgumentError
         end
         raise WriteXLSXInsufficientArgumentError if [row1, col1, row2, col2].include?(nil)
+
         check_for_valid_input_params
 
         check_dimensions(row1, col1)
@@ -42,7 +44,7 @@ module Writexlsx
 
         # The any, list and custom validations don't have a criteria
         # so we use a default of 'between'
-        if ['none', 'list', 'custom'].include?(@validate)
+        if %w[none list custom].include?(@validate)
           @criteria  = 'between'
           @maximum   = nil
         end
@@ -56,13 +58,10 @@ module Writexlsx
 
         convert_date_time_value_if_required
         # Check that the input title doesn't exceed the maximum length.
-        if @input_title && @input_title.length > 32
-          raise "Length of input title '#{@input_title}' exceeds Excel's limit of 32"
-        end
+        raise "Length of input title '#{@input_title}' exceeds Excel's limit of 32" if @input_title && @input_title.length > 32
         # Check that the input message doesn't exceed the maximum length.
-        if @input_message && @input_message.length > 255
-          raise "Length of input message '#{@input_message}' exceeds Excel's limit of 255"
-        end
+        raise "Length of input message '#{@input_message}' exceeds Excel's limit of 255" if @input_message && @input_message.length > 255
+
         set_some_defaults
 
         # A (for now) undocumented parameter to pass additional cell ranges.
@@ -76,7 +75,7 @@ module Writexlsx
       end
 
       def keys
-        self.instance_variables.collect { |v| v.to_s.sub(/@/, '').to_sym }
+        instance_variables.collect { |v| v.to_s.sub(/@/, '').to_sym }
       end
 
       def validate_none?
@@ -86,7 +85,7 @@ module Writexlsx
       #
       # Write the <dataValidation> element.
       #
-      def write_data_validation(writer) #:nodoc:
+      def write_data_validation(writer) # :nodoc:
         @writer = writer
         if @validate == 'none'
           @writer.empty_tag('dataValidation', attributes)
@@ -105,9 +104,9 @@ module Writexlsx
       #
       # Write the <formula1> element.
       #
-      def write_formula_1(formula) #:nodoc:
+      def write_formula_1(formula) # :nodoc:
         # Convert a list array ref into a comma separated string.
-        formula   = %!"#{formula.join(',')}"! if formula.kind_of?(Array)
+        formula   = %("#{formula.join(",")}") if formula.is_a?(Array)
 
         formula = formula.sub(/^=/, '') if formula.respond_to?(:sub)
 
@@ -117,7 +116,7 @@ module Writexlsx
       #
       # Write the <formula2> element.
       #
-      def write_formula_2(formula) #:nodoc:
+      def write_formula_2(formula) # :nodoc:
         formula = formula.sub(/^=/, '') if formula.respond_to?(:sub)
 
         @writer.data_element('formula2', formula)
@@ -147,8 +146,8 @@ module Writexlsx
         end
 
         if @error_type
-          attributes << ['errorStyle', 'warning'] if @error_type == 1
-          attributes << ['errorStyle', 'information'] if @error_type == 2
+          attributes << %w[errorStyle warning] if @error_type == 1
+          attributes << %w[errorStyle information] if @error_type == 2
         end
         attributes << ['allowBlank',       1] if @ignore_blank != 0
         attributes << ['showDropDown',     1] if @dropdown     == 0
@@ -176,9 +175,8 @@ module Writexlsx
       def check_for_valid_input_params
         check_parameter(self, valid_validation_parameter, 'data_validation')
 
-        unless has_key?(:validate)
-          raise WriteXLSXOptionParameterError, "Parameter :validate is required in data_validation()"
-        end
+        raise WriteXLSXOptionParameterError, "Parameter :validate is required in data_validation()" unless has_key?(:validate)
+
         unless valid_validation_type.has_key?(@validate.downcase)
           raise WriteXLSXOptionParameterError,
                 "Unknown validation type '#{@validate}' for parameter :validate in data_validation()"
@@ -190,9 +188,7 @@ module Writexlsx
       end
 
       def check_criteria_required
-        unless has_key?(:criteria)
-          raise WriteXLSXOptionParameterError, "Parameter :criteria is required in data_validation()"
-        end
+        raise WriteXLSXOptionParameterError, "Parameter :criteria is required in data_validation()" unless has_key?(:criteria)
       end
 
       def check_maximum_value_when_criteria_is_between_or_notbetween
@@ -216,81 +212,77 @@ module Writexlsx
       def convert_date_time_value_if_required
         if @validate == 'date' || @validate == 'time'
           date_time = convert_date_time(@value)
-          if date_time
-            @value = date_time
-          end
+          @value = date_time if date_time
 
           if @maximum
             date_time = convert_date_time(@maximum)
-            if date_time
-              @maximum = date_time
-            end
+            @maximum = date_time if date_time
           end
         end
       end
 
       def error_type_hash
-        {'stop' => 0, 'warning' => 1, 'information' => 2}
+        { 'stop' => 0, 'warning' => 1, 'information' => 2 }
       end
 
       def valid_validation_type # :nodoc:
         {
-          'any'             => 'none',
-          'any value'       => 'none',
-          'whole number'    => 'whole',
-          'whole'           => 'whole',
-          'integer'         => 'whole',
-          'decimal'         => 'decimal',
-          'list'            => 'list',
-          'date'            => 'date',
-          'time'            => 'time',
-          'text length'     => 'textLength',
-          'length'          => 'textLength',
-          'custom'          => 'custom'
+          'any'          => 'none',
+          'any value'    => 'none',
+          'whole number' => 'whole',
+          'whole'        => 'whole',
+          'integer'      => 'whole',
+          'decimal'      => 'decimal',
+          'list'         => 'list',
+          'date'         => 'date',
+          'time'         => 'time',
+          'text length'  => 'textLength',
+          'length'       => 'textLength',
+          'custom'       => 'custom'
         }
       end
 
       # List of valid input parameters.
       def valid_validation_parameter
-        [
-          :validate,
-          :criteria,
-          :value,
-          :source,
-          :minimum,
-          :maximum,
-          :ignore_blank,
-          :dropdown,
-          :show_input,
-          :input_title,
-          :input_message,
-          :show_error,
-          :error_title,
-          :error_message,
-          :error_type,
-          :other_cells
+        %i[
+          validate
+          criteria
+          value
+          source
+          minimum
+          maximum
+          ignore_blank
+          dropdown
+          show_input
+          input_title
+          input_message
+          show_error
+          error_title
+          error_message
+          error_type
+          other_cells
         ]
       end
 
       # List of valid criteria types.
       def valid_criteria_type  # :nodoc:
         {
-          'between'                     => 'between',
-          'not between'                 => 'notBetween',
-          'equal to'                    => 'equal',
-          '='                           => 'equal',
-          '=='                          => 'equal',
-          'not equal to'                => 'notEqual',
-          '!='                          => 'notEqual',
-          '<>'                          => 'notEqual',
-          'greater than'                => 'greaterThan',
-          '>'                           => 'greaterThan',
-          'less than'                   => 'lessThan',
-          '<'                           => 'lessThan',
-          'greater than or equal to'    => 'greaterThanOrEqual',
-          '>='                          => 'greaterThanOrEqual',
-          'less than or equal to'       => 'lessThanOrEqual',
-          '<='                          => 'lessThanOrEqual'
+          'between'                  => 'between',
+          'not between'              => 'notBetween',
+          'equal to'                 => 'equal',
+          '='                        => 'equal',
+          '=='                       => 'equal',
+          'not equal to'             => 'notEqual',
+          '!='                       => 'notEqual',
+          '<>'                       => 'notEqual',
+          'greater than'             => 'greaterThan',
+          '>'                        => 'greaterThan',
+          'less than'                => 'lessThan',
+          '<'                        => 'lessThan',
+          'greater than or equal to' => 'greaterThanOrEqual',
+          '>='                       => 'greaterThanOrEqual',
+          'less than or equal to'    => 'lessThanOrEqual',
+          '<='                       => 'lessThanOrEqual'
         }
       end
 

data/lib/write_xlsx/worksheet/hyperlink.rb

@@ -33,9 +33,7 @@ module Writexlsx
         url = escape_url(url)
 
         # Excel limits the escaped URL and location/anchor to 255 characters.
-        if url.bytesize > max_url_length || (!@url_str.nil? && @url_str.bytesize > max_url_length)
-          raise "Ignoring URL '#{url}' where link or anchor > #{max_url_length} characters since it exceeds Excel's limit for URLS. See LIMITATIONS section of the Excel::Writer::XLSX documentation."
-        end
+        raise "Ignoring URL '#{url}' where link or anchor > #{max_url_length} characters since it exceeds Excel's limit for URLS. See LIMITATIONS section of the Excel::Writer::XLSX documentation." if url.bytesize > max_url_length || (!@url_str.nil? && @url_str.bytesize > max_url_length)
 
         @url       = url
         @str       = normalized_str
@@ -45,7 +43,7 @@ module Writexlsx
       def attributes(row, col, id)
         ref = xl_rowcol_to_cell(row, col)
 
-        attr = [ ['ref', ref] ]
+        attr = [['ref', ref]]
         attr << r_id_attributes(id)
 
         attr << ['location', @url_str] if @url_str
@@ -78,14 +76,12 @@ module Writexlsx
         @url_str = @str.dup
 
         # Excel limits escaped URL to #{max_url_length} characters.
-        if @url.bytesize > max_url_length
-          raise "URL '#{@url}' > #{max_url_length} characters, it exceeds Excel's limit for URLS."
-        end
+        raise "URL '#{@url}' > #{max_url_length} characters, it exceeds Excel's limit for URLS." if @url.bytesize > max_url_length
 
         @tip = tip
       end
 
-      def attributes(row, col, dummy = nil)
+      def attributes(row, col, _dummy = nil)
         attr = [
           ['ref', xl_rowcol_to_cell(row, col)],
           ['location', @url]
@@ -102,8 +98,8 @@ module Writexlsx
         str ||= url.dup
 
         # For external links change the directory separator from Unix to Dos.
-        url = url.gsub(%r|/|, '\\')
-        str.gsub!(%r|/|, '\\')
+        url = url.gsub(%r{/}, '\\')
+        str.gsub!(%r{/}, '\\')
 
         # Strip the mailto header.
         str.sub!(/^mailto:/, '')
@@ -115,19 +111,17 @@ module Writexlsx
         url = escape_url(url)
 
         # Add the file:/// URI to the url if non-local.
-        if url =~ %r![:]! ||        # Windows style "C:/" link.
-           url =~ %r!^\\\\!        # Network share.
+        if url =~ /:/ ||        # Windows style "C:/" link.
+           url =~ /^\\\\/        # Network share.
           url = "file:///#{url}"
         end
 
         # Convert a ./dir/file.xlsx link to dir/file.xlsx.
-        url = url.sub(%r!^.\\!, '')
+        url = url.sub(/^.\\/, '')
         @url_str   = url_str
 
         # Excel limits the escaped URL and location/anchor to max_url_length characters.
-        if url.bytesize > max_url_length || (!@url_str.nil? && @url_str.bytesize > max_url_length)
-          raise "Ignoring URL '#{url}' where link or anchor > #{max_url_length} characters since it exceeds Excel's limit for URLS. See LIMITATIONS section of the Excel::Writer::XLSX documentation."
-        end
+        raise "Ignoring URL '#{url}' where link or anchor > #{max_url_length} characters since it exceeds Excel's limit for URLS. See LIMITATIONS section of the Excel::Writer::XLSX documentation." if url.bytesize > max_url_length || (!@url_str.nil? && @url_str.bytesize > max_url_length)
 
         @url       = url
         @str       = str

data/lib/write_xlsx/worksheet/page_setup.rb

@@ -67,7 +67,7 @@ module Writexlsx
 
       def hide_gridlines(option)
         if option == 0 || !option
-          @print_gridlines  = true
+          @print_gridlines = true
           @print_options_changed = true
         else
           @print_gridlines  = false
@@ -94,7 +94,7 @@ module Writexlsx
       #     r:id="rId1"
       # />
       #
-      def write_page_setup(writer) #:nodoc:
+      def write_page_setup(writer) # :nodoc:
         return unless @page_setup_changed
 
         attributes = []
@@ -102,15 +102,14 @@ module Writexlsx
         attributes << ['scale',           @scale]         if @scale != 100
         attributes << ['fitToWidth',      @fit_width]     if @fit_page && @fit_width != 1
         attributes << ['fitToHeight',     @fit_height]    if @fit_page && @fit_height != 1
-        attributes << ['pageOrder',       "overThenDown"] if @across
+        attributes << %w[pageOrder overThenDown] if @across
         attributes << ['firstPageNumber', @page_start]    if @page_start && @page_start > 1
         attributes << ['orientation',
                        if @orientation
                          'portrait'
                        else
                          'landscape'
-                       end
-                      ]
+                       end]
         attributes << ['blackAndWhite', 1]      if @black_white
         attributes << ['useFirstPageNumber', 1] if ptrue?(@page_start)
 
@@ -124,14 +123,14 @@ module Writexlsx
       #
       # Write the <pageMargins> element.
       #
-      def write_page_margins(writer) #:nodoc:
+      def write_page_margins(writer) # :nodoc:
         writer.empty_tag('pageMargins', margin_attributes)
       end
 
       #
       # Write the <printOptions> element.
       #
-      def write_print_options(writer) #:nodoc:
+      def write_print_options(writer) # :nodoc:
         return unless @print_options_changed
 
         attributes = []
@@ -145,7 +144,7 @@ module Writexlsx
       #
       # Write the <headerFooter> element.
       #
-      def write_header_footer(writer, excel2003_style) #:nodoc:
+      def write_header_footer(writer, excel2003_style) # :nodoc:
         tag = 'headerFooter'
         attributes = []
         attributes << ['scaleWithDoc', 0]     unless ptrue?(@header_footer_scales)
@@ -159,7 +158,6 @@ module Writexlsx
         elsif excel2003_style
           writer.empty_tag(tag, attributes)
         end
-
       end
 
       private
@@ -167,14 +165,14 @@ module Writexlsx
       #
       # Write the <oddHeader> element.
       #
-      def write_odd_header(writer) #:nodoc:
+      def write_odd_header(writer) # :nodoc:
         writer.data_element('oddHeader', @header)
       end
 
       #
       # Write the <oddFooter> element.
       #
-      def write_odd_footer(writer) #:nodoc:
+      def write_odd_footer(writer) # :nodoc:
         writer.data_element('oddFooter', @footer)
       end