xlsxrb 0.1.14 → 0.1.16

data/lib/xlsxrb/elements/worksheet.rb

@@ -17,7 +17,7 @@ module Xlsxrb
     class Worksheet
       [Enumerable, CoordinateAccess].each { |m| include m }
 
-      attr_reader :name, :rows, :columns, :charts, :conditional_formatting, :data_validations, :unmapped_data, :errors, :state, :hyperlinks, :comments, :images, :styles, :date1904, :print_area, :print_titles, :sparkline_groups
+      attr_reader :name, :rows, :columns, :charts, :conditional_formatting, :data_validations, :unmapped_data, :errors, :state, :hyperlinks, :comments, :images, :styles, :date1904, :print_area, :print_titles, :sparkline_groups, :merged_cells, :auto_filter, :tables, :shapes, :freeze_pane, :page_setup, :page_margins, :header_footer, :print_options, :sheet_protection, :sheet_view, :sheet_properties, :sheet_format, :row_breaks, :col_breaks, :protected_ranges, :cell_watches, :selection
       attr_writer :dimension
       alias conditional_formats conditional_formatting
 
@@ -42,8 +42,26 @@ module Xlsxrb
       # @param print_area [String, nil] Sheet print area range (e.g. "A1:H50").
       # @param print_titles [Hash, nil] Repeating print title rows/cols (e.g. { rows: "1:2", cols: "A:B" }).
       # @param sparkline_groups [Array<Hash>] Sparkline groups in the sheet.
-      #: (name: String?, ?rows: Array[Elements::Row], ?columns: Array[Elements::Column], ?charts: Array[Hash[Symbol, untyped]], ?conditional_formatting: Array[Hash[Symbol, untyped]]?, ?data_validations: Array[Hash[Symbol, untyped]], ?unmapped_data: Hash[untyped, untyped], ?errors: Array[String]?, ?conditional_formats: Array[Hash[Symbol, untyped]]?, ?state: Symbol, ?hyperlinks: Hash[String, Hash[Symbol, untyped]], ?comments: Array[Hash[Symbol, untyped]], ?images: Array[Elements::Image], ?styles: Hash[untyped, untyped]?, ?date1904: bool, ?dimension: String?, ?trim_empty_rows: bool, ?pad_empty_rows: bool, ?pad_empty_cells: bool, ?print_area: String?, ?print_titles: Hash[Symbol, untyped]?, ?sparkline_groups: Array[Hash[Symbol, untyped]]) -> void
-      def initialize(name:, rows: [], columns: [], charts: [], conditional_formatting: nil, data_validations: [], unmapped_data: {}, errors: nil, conditional_formats: nil, state: :visible, hyperlinks: {}, comments: [], images: [], styles: nil, date1904: false, dimension: nil, trim_empty_rows: false, pad_empty_rows: false, pad_empty_cells: false, print_area: nil, print_titles: nil, sparkline_groups: [])
+      # @param merged_cells [Array<String>] Merged cell ranges (e.g. ["A1:B2"]).
+      # @param auto_filter [String, nil] AutoFilter range (e.g. "A1:E10").
+      # @param tables [Array<Hash>] Table definitions in the sheet.
+      # @param shapes [Array<Hash>] Shape definitions in the sheet.
+      # @param freeze_pane [Hash, nil] Freeze pane settings (e.g. { row: 1, col: 0 }).
+      # @param page_setup [Hash] Page setup properties (e.g. orientation, paper size).
+      # @param page_margins [Hash, nil] Page margins settings.
+      # @param header_footer [Hash] Header and footer configurations.
+      # @param print_options [Hash] Print options (e.g. gridlines, headings).
+      # @param sheet_protection [Hash, nil] Sheet protection settings.
+      # @param sheet_view [Hash] Sheet view configurations (e.g. zoom, gridlines).
+      # @param sheet_properties [Hash] Sheet properties (e.g. tabColor, outlinePr).
+      # @param sheet_format [Hash] Sheet format properties (e.g. defaultRowHeight, defaultColWidth).
+      # @param row_breaks [Array<Integer | Hash>] Explicit page row breaks.
+      # @param col_breaks [Array<Integer | Hash>] Explicit page column breaks.
+      # @param protected_ranges [Array<Hash>] Protected range definitions.
+      # @param cell_watches [Array<String>] Watched cell references.
+      # @param selection [Hash, Array<Hash>, nil] Active cell and selection settings.
+      #: (name: String?, ?rows: Array[Elements::Row], ?columns: Array[Elements::Column], ?charts: Array[Hash[Symbol, untyped]], ?conditional_formatting: Array[Hash[Symbol, untyped]]?, ?data_validations: Array[Hash[Symbol, untyped]], ?unmapped_data: Hash[untyped, untyped], ?errors: Array[String]?, ?conditional_formats: Array[Hash[Symbol, untyped]]?, ?state: Symbol, ?hyperlinks: Hash[String, untyped], ?comments: Array[Hash[Symbol, untyped]], ?images: Array[Elements::Image], ?styles: (Elements::Styles | Hash[untyped, untyped])?, ?date1904: bool, ?dimension: String?, ?trim_empty_rows: bool, ?pad_empty_rows: bool, ?pad_empty_cells: bool, ?print_area: String?, ?print_titles: Hash[Symbol, untyped]?, ?sparkline_groups: Array[Hash[Symbol, untyped]], ?merged_cells: Array[String], ?auto_filter: String?, ?tables: Array[Hash[Symbol, untyped]], ?shapes: Array[Hash[Symbol, untyped]], ?freeze_pane: Hash[Symbol, untyped]?, ?page_setup: Hash[Symbol, untyped], ?page_margins: Hash[Symbol, untyped]?, ?header_footer: Hash[Symbol, untyped], ?print_options: Hash[Symbol, untyped], ?sheet_protection: Hash[Symbol, untyped]?, ?sheet_view: Hash[Symbol, untyped], ?sheet_properties: Hash[Symbol, untyped], ?sheet_format: Hash[Symbol, untyped], ?row_breaks: Array[untyped], ?col_breaks: Array[untyped], ?protected_ranges: Array[Hash[Symbol, untyped]], ?cell_watches: Array[String], ?selection: (Hash[Symbol, untyped] | Array[Hash[Symbol, untyped]])?) -> void
+      def initialize(name:, rows: [], columns: [], charts: [], conditional_formatting: nil, data_validations: [], unmapped_data: {}, errors: nil, conditional_formats: nil, state: :visible, hyperlinks: {}, comments: [], images: [], styles: nil, date1904: false, dimension: nil, trim_empty_rows: false, pad_empty_rows: false, pad_empty_cells: false, print_area: nil, print_titles: nil, sparkline_groups: [], merged_cells: [], auto_filter: nil, tables: [], shapes: [], freeze_pane: nil, page_setup: {}, page_margins: nil, header_footer: {}, print_options: {}, sheet_protection: nil, sheet_view: {}, sheet_properties: {}, sheet_format: {}, row_breaks: [], col_breaks: [], protected_ranges: [], cell_watches: [], selection: nil)
         @name = name
         @rows = (rows || []).freeze
         @columns = (columns || []).freeze
@@ -67,6 +85,24 @@ module Xlsxrb
         @trim_empty_rows = trim_empty_rows ? true : false
         @pad_empty_rows = pad_empty_rows ? true : false
         @pad_empty_cells = pad_empty_cells ? true : false
+        @merged_cells = (merged_cells || []).freeze
+        @auto_filter = auto_filter
+        @tables = (tables || []).freeze
+        @shapes = (shapes || []).freeze
+        @freeze_pane = freeze_pane&.freeze
+        @page_setup = (page_setup || {}).freeze
+        @page_margins = page_margins&.freeze
+        @header_footer = (header_footer || {}).freeze
+        @print_options = (print_options || {}).freeze
+        @sheet_protection = sheet_protection&.freeze
+        @sheet_view = (sheet_view || {}).freeze
+        @sheet_properties = (sheet_properties || {}).freeze
+        @sheet_format = (sheet_format || unmapped_data[:sheet_format] || unmapped_data.dig(:facade, :sheet_format) || {}).freeze
+        @row_breaks = (row_breaks || []).freeze
+        @col_breaks = (col_breaks || []).freeze
+        @protected_ranges = (protected_ranges || unmapped_data[:protected_ranges] || unmapped_data.dig(:facade, :protected_ranges) || []).freeze
+        @cell_watches = (cell_watches || unmapped_data[:cell_watches] || unmapped_data.dig(:facade, :cell_watches) || []).freeze
+        @selection = selection&.freeze
       end
 
       # Returns whether trailing empty rows are omitted during iteration.
@@ -328,6 +364,15 @@ module Xlsxrb
         @state == :visible
       end
 
+      # Returns whether the worksheet is very hidden (:very_hidden).
+      #
+      # @return [Boolean]
+      # @api public
+      #: () -> bool
+      def very_hidden?
+        @state == :very_hidden
+      end
+
       # Returns hyperlink metadata for the given cell reference, or nil.
       #
       # @param ref_or_row [String, Symbol, Integer] Cell reference (e.g. "A1") or 0-based row index.
@@ -488,6 +533,24 @@ module Xlsxrb
         new_print_titles = changes.key?(:print_titles) ? changes[:print_titles] : print_titles
         new_sparklines = changes.key?(:sparkline_groups) ? changes[:sparkline_groups] : sparkline_groups
         new_trim = changes.key?(:trim_empty_rows) ? changes[:trim_empty_rows] : @trim_empty_rows
+        new_merged = changes.key?(:merged_cells) ? changes[:merged_cells] : merged_cells
+        new_auto_filter = changes.key?(:auto_filter) ? changes[:auto_filter] : auto_filter
+        new_tables = changes.key?(:tables) ? changes[:tables] : tables
+        new_shapes = changes.key?(:shapes) ? changes[:shapes] : shapes
+        new_freeze_pane = changes.key?(:freeze_pane) ? changes[:freeze_pane] : freeze_pane
+        new_page_setup = changes.key?(:page_setup) ? changes[:page_setup] : page_setup
+        new_page_margins = changes.key?(:page_margins) ? changes[:page_margins] : page_margins
+        new_header_footer = changes.key?(:header_footer) ? changes[:header_footer] : header_footer
+        new_print_options = changes.key?(:print_options) ? changes[:print_options] : print_options
+        new_sheet_protection = changes.key?(:sheet_protection) ? changes[:sheet_protection] : sheet_protection
+        new_sheet_view = changes.key?(:sheet_view) ? changes[:sheet_view] : sheet_view
+        new_sheet_properties = changes.key?(:sheet_properties) ? changes[:sheet_properties] : sheet_properties
+        new_sheet_format = changes.key?(:sheet_format) ? changes[:sheet_format] : sheet_format
+        new_row_breaks = changes.key?(:row_breaks) ? changes[:row_breaks] : row_breaks
+        new_col_breaks = changes.key?(:col_breaks) ? changes[:col_breaks] : col_breaks
+        new_protected_ranges = changes.key?(:protected_ranges) ? changes[:protected_ranges] : protected_ranges
+        new_cell_watches = changes.key?(:cell_watches) ? changes[:cell_watches] : cell_watches
+        new_selection = changes.key?(:selection) ? changes[:selection] : selection
 
         self.class.new(
           name: new_name,
@@ -507,7 +570,25 @@ module Xlsxrb
           print_area: new_print_area,
           print_titles: new_print_titles,
           sparkline_groups: new_sparklines,
-          trim_empty_rows: new_trim
+          trim_empty_rows: new_trim,
+          merged_cells: new_merged,
+          auto_filter: new_auto_filter,
+          tables: new_tables,
+          shapes: new_shapes,
+          freeze_pane: new_freeze_pane,
+          page_setup: new_page_setup,
+          page_margins: new_page_margins,
+          header_footer: new_header_footer,
+          print_options: new_print_options,
+          sheet_protection: new_sheet_protection,
+          sheet_view: new_sheet_view,
+          sheet_properties: new_sheet_properties,
+          sheet_format: new_sheet_format,
+          row_breaks: new_row_breaks,
+          col_breaks: new_col_breaks,
+          protected_ranges: new_protected_ranges,
+          cell_watches: new_cell_watches,
+          selection: new_selection
         )
       end
 
@@ -533,7 +614,23 @@ module Xlsxrb
           print_area: print_area,
           print_titles: print_titles,
           sparkline_groups: sparkline_groups,
-          trim_empty_rows: @trim_empty_rows
+          trim_empty_rows: @trim_empty_rows,
+          merged_cells: merged_cells,
+          auto_filter: auto_filter,
+          tables: tables,
+          shapes: shapes,
+          freeze_pane: freeze_pane,
+          page_setup: page_setup,
+          page_margins: page_margins,
+          header_footer: header_footer,
+          print_options: print_options,
+          sheet_protection: sheet_protection,
+          sheet_view: sheet_view,
+          sheet_properties: sheet_properties,
+          sheet_format: sheet_format,
+          row_breaks: row_breaks,
+          col_breaks: col_breaks,
+          selection: selection
         }
       end
 
@@ -544,13 +641,26 @@ module Xlsxrb
 
         name == other.name && rows == other.rows && columns == other.columns && charts == other.charts &&
           conditional_formatting == other.conditional_formatting && data_validations == other.data_validations &&
-          state == other.state && hyperlinks == other.hyperlinks && comments == other.comments
+          state == other.state && hyperlinks == other.hyperlinks && comments == other.comments &&
+          merged_cells == other.merged_cells && auto_filter == other.auto_filter &&
+          tables == other.tables && shapes == other.shapes && freeze_pane == other.freeze_pane &&
+          page_setup == other.page_setup && page_margins == other.page_margins &&
+          header_footer == other.header_footer && print_options == other.print_options &&
+          sheet_protection == other.sheet_protection && sheet_view == other.sheet_view &&
+          sheet_properties == other.sheet_properties && sheet_format == other.sheet_format &&
+          row_breaks == other.row_breaks && col_breaks == other.col_breaks &&
+          selection == other.selection
       end
       alias eql? ==
 
       #: () -> Integer
       def hash
-        [self.class, name, rows, columns, charts, conditional_formatting, data_validations, state, hyperlinks, comments].hash
+        [
+          self.class, name, rows, columns, charts, conditional_formatting, data_validations,
+          state, hyperlinks, comments, merged_cells, auto_filter, tables, shapes, freeze_pane,
+          page_setup, page_margins, header_footer, print_options, sheet_protection, sheet_view,
+          sheet_properties, sheet_format, row_breaks, col_breaks, selection
+        ].hash
       end
 
       # Returns self when load is called on an already in-memory Worksheet.

data/lib/xlsxrb/ooxml/cfb.rb

@@ -100,9 +100,16 @@ module Xlsxrb
           raise Xlsxrb::Error, "CFB file is too small" if @data.bytesize < 512
 
           _, major_ver = @data[0x18, 4].unpack("v2")
+          raise Xlsxrb::Error, "Unsupported CFB major version: #{major_ver}" unless [3, 4].include?(major_ver)
+
           @major_version = major_ver
           @sector_shift = @data[0x1E, 2].unpack1("v")
           @mini_sector_shift = @data[0x20, 2].unpack1("v")
+
+          expected_shift = @major_version == 4 ? 12 : 9
+          raise Xlsxrb::Error, "Invalid CFB sector shift in header: #{@sector_shift}" unless @sector_shift == expected_shift
+          raise Xlsxrb::Error, "Invalid CFB mini sector shift in header: #{@mini_sector_shift}" unless @mini_sector_shift == 6
+
           @sector_size = 1 << @sector_shift
           @mini_sector_size = 1 << @mini_sector_shift
 
@@ -123,6 +130,9 @@ module Xlsxrb
         end
 
         def read_sector(sector_id)
+          max_sectors = (@data.bytesize - 512) / @sector_size
+          return "".b if sector_id.negative? || sector_id >= max_sectors
+
           offset = sector_offset(sector_id)
           @data[offset, @sector_size] || "".b
         end

data/lib/xlsxrb/ooxml/reader/listeners/core_listeners.rb

@@ -24,12 +24,15 @@ module Xlsxrb
           @runs = []
           @current_font = {}
           @has_runs = false
+          @inside_rph = false
         end
 
         def start_element(_uri, local_name, qname, attributes)
           name = element_name(local_name, qname)
 
           case name
+          when "rPh"
+            @inside_rph = true
           when "si"
             @inside_si = true
             @text_buffer = +""
@@ -74,7 +77,7 @@ module Xlsxrb
           when "scheme"
             @current_font[:scheme] = attributes["val"] if @inside_rpr
           when "t"
-            @inside_t = @inside_si
+            @inside_t = @inside_si && !@inside_rph
             @text_buffer = +"" if @inside_r
           end
         end
@@ -87,12 +90,16 @@ module Xlsxrb
           name = element_name(local_name, qname)
 
           case name
+          when "rPh"
+            @inside_rph = false
           when "t"
             @inside_t = false
           when "rPr"
             @inside_rpr = false
           when "r"
-            run = { text: @text_buffer.dup }
+            decoded_text = XmlBuilder.unescape(@text_buffer)
+            decoded_text = decoded_text.gsub("\r\n", "\n").tr("\r", "\n") if decoded_text.include?("\r")
+            run = { text: decoded_text }
             run[:font] = @current_font.dup unless @current_font.empty?
             @runs << run
             @inside_r = false
@@ -100,7 +107,9 @@ module Xlsxrb
             @strings << if @has_runs
                           Xlsxrb::Elements::RichText.new(runs: @runs)
                         else
-                          @text_buffer.dup
+                          decoded_text = XmlBuilder.unescape(@text_buffer)
+                          decoded_text = decoded_text.gsub("\r\n", "\n").tr("\r", "\n") if decoded_text.include?("\r")
+                          decoded_text
                         end
             @inside_si = false
             @text_buffer = +""
@@ -145,18 +154,23 @@ module Xlsxrb
           @is_has_runs = false
           @is_current_font = {}
           @is_run_text = +""
+          @inside_rph = false
+          @has_inline_text = false
         end
 
         def start_element(_uri, local_name, qname, attributes)
           name = element_name(local_name, qname)
 
           case name
+          when "rPh"
+            @inside_rph = true
           when "row"
             parse_row_attributes(attributes)
           when "c"
             @current_cell_ref = attributes["r"]
             @current_cell_type = attributes["t"]
             @current_cell_ph = %w[1 true].include?(attributes["ph"])
+            @has_inline_text = false
             @value_buffer = +""
             @inline_text_buffer = +""
             @formula_buffer = +""
@@ -188,6 +202,7 @@ module Xlsxrb
             @formula_r1 = attributes["r1"] if attributes["r1"]
             @formula_r2 = attributes["r2"] if attributes["r2"]
           when "is"
+            @has_inline_text = true
             @inside_is = true if @current_cell_type == "inlineStr"
           when "r"
             if @inside_is
@@ -231,11 +246,15 @@ module Xlsxrb
           when "scheme"
             @is_current_font[:scheme] = attributes["val"] if @inside_is_rpr
           when "t"
-            if @inside_is_r
-              @is_run_text = +""
-              @inside_inline_text = true
-            elsif @inside_is || (@current_cell_type == "inlineStr" && !@current_cell_ref.nil?)
-              @inside_inline_text = true
+            unless @inside_rph
+              if @inside_is_r
+                @is_run_text = +""
+                @inside_inline_text = true
+                @has_inline_text = true
+              elsif @inside_is || (@current_cell_type == "inlineStr" && !@current_cell_ref.nil?)
+                @inside_inline_text = true
+                @has_inline_text = true
+              end
             end
           end
         end
@@ -256,6 +275,8 @@ module Xlsxrb
           name = element_name(local_name, qname)
 
           case name
+          when "rPh"
+            @inside_rph = false
           when "v"
             @inside_value = false
           when "f"
@@ -309,7 +330,14 @@ module Xlsxrb
           return if @current_cell_ref.nil?
 
           unless @formula_buffer.empty? && @formula_type.nil?
-            cached = @value_buffer.empty? ? nil : @value_buffer.dup
+            cached = if @value_buffer.empty?
+                       nil
+                     elsif @current_cell_type == "b"
+                       v = @value_buffer.strip
+                       v == "1" || v.casecmp?("true")
+                     else
+                       @value_buffer.dup
+                     end
             f_type = { "shared" => :shared, "array" => :array, "dataTable" => :data_table }[@formula_type]
             @cells[@current_cell_ref] = Xlsxrb::Elements::Formula.new(
               expression: @formula_buffer.dup,
@@ -328,39 +356,29 @@ module Xlsxrb
             return
           end
 
-          case @current_cell_type
-          when "inlineStr"
-            @cells[@current_cell_ref] = if @is_has_runs
-                                          Xlsxrb::Elements::RichText.new(runs: @is_runs.map(&:dup))
-                                        else
-                                          @inline_text_buffer.dup
-                                        end
-          when "s"
-            index = @value_buffer.to_i
-            @cells[@current_cell_ref] = @shared_strings[index] || ""
-          when "e"
-            code = @value_buffer.dup
-            @cells[@current_cell_ref] = if Xlsxrb::Elements::VALID_ERROR_CODES.include?(code)
-                                          Xlsxrb::Elements::CellError.new(code:)
-                                        else
-                                          code
-                                        end
-          when "b"
-            @cells[@current_cell_ref] = @value_buffer.strip == "1"
-          when nil, "", "n"
-            return if @value_buffer.empty?
-
-            raw = @value_buffer.dup
-            @cells[@current_cell_ref] = numeric_value(raw)
+          if @current_cell_type == "inlineStr" && @is_has_runs
+            @cells[@current_cell_ref] = Xlsxrb::Elements::RichText.new(runs: @is_runs.map(&:dup))
+            return
           end
+
+          v = if @current_cell_type == "inlineStr"
+                @has_inline_text ? @inline_text_buffer : nil
+              else
+                @value_buffer.empty? ? nil : @value_buffer
+              end
+
+          val = Xlsxrb::Ooxml::WorksheetParser.resolve_cell_value(@current_cell_type, v, nil, @shared_strings)
+          return if val.nil? && (@current_cell_type.nil? || @current_cell_type == "" || @current_cell_type == "n" || @current_cell_type == "s" || @current_cell_type == "inlineStr")
+
+          @cells[@current_cell_ref] = if @current_cell_type == "e" && val && Xlsxrb::Elements::VALID_ERROR_CODES.include?(val)
+                                        Xlsxrb::Elements::CellError.new(code: val)
+                                      else
+                                        val
+                                      end
         end
 
         def numeric_value(raw)
-          if raw.include?(".")
-            raw.to_f
-          else
-            raw.to_i
-          end
+          Xlsxrb::Ooxml::WorksheetParser.numeric_cell_value(raw)
         end
 
         def element_name(local_name, qname)

data/lib/xlsxrb/ooxml/reader/listeners/feature_listeners.rb

@@ -204,7 +204,7 @@ module Xlsxrb
         def start_element(_uri, local_name, qname, _attributes)
           name = element_name(local_name, qname)
           case name
-          when "Application", "AppVersion"
+          when "Application", "AppVersion", "Company", "Manager"
             @current_field = name
             @text_buffer = +""
           when "HeadingPairs"
@@ -232,6 +232,12 @@ module Xlsxrb
           when "AppVersion"
             @properties[:app_version] = @text_buffer.dup
             @current_field = nil
+          when "Company"
+            @properties[:company] = @text_buffer.dup
+            @current_field = nil
+          when "Manager"
+            @properties[:manager] = @text_buffer.dup
+            @current_field = nil
           when "lpstr"
             if @inside_titles_of_parts
               @titles_of_parts << @text_buffer.dup
@@ -956,11 +962,14 @@ module Xlsxrb
           when "formula2"
             @inside_formula2 = true
             @text_buffer = +""
+          when "sqref"
+            @inside_sqref = true
+            @text_buffer = +""
           end
         end
 
         def characters(text)
-          @text_buffer << text if @inside_formula1 || @inside_formula2
+          @text_buffer << text if @inside_formula1 || @inside_formula2 || @inside_sqref
         end
 
         def end_element(_uri, local_name, qname)
@@ -972,6 +981,9 @@ module Xlsxrb
           when "formula2"
             @current_dv[:formula2] = @text_buffer.dup if @current_dv
             @inside_formula2 = false
+          when "sqref"
+            @current_dv[:sqref] = @text_buffer.strip if @current_dv && (@current_dv[:sqref].nil? || @current_dv[:sqref].empty?)
+            @inside_sqref = false
           when "dataValidation"
             @validations << @current_dv if @current_dv
             @current_dv = nil

data/lib/xlsxrb/ooxml/reader/listeners/style_listeners.rb

@@ -135,19 +135,19 @@ module Xlsxrb
           when "font"
             @current_font = {} if @inside_fonts || @inside_dxfs
           when "b"
-            @current_font[:bold] = true if @current_font
+            @current_font[:bold] = truthy_val?(attributes) if @current_font
           when "i"
-            @current_font[:italic] = true if @current_font
+            @current_font[:italic] = truthy_val?(attributes) if @current_font
           when "strike"
-            @current_font[:strike] = true if @current_font
+            @current_font[:strike] = truthy_val?(attributes) if @current_font
           when "shadow"
-            @current_font[:shadow] = true if @current_font
+            @current_font[:shadow] = truthy_val?(attributes) if @current_font
           when "outline"
-            @current_font[:outline] = true if @current_font
+            @current_font[:outline] = truthy_val?(attributes) if @current_font
           when "condense"
-            @current_font[:condense] = true if @current_font
+            @current_font[:condense] = truthy_val?(attributes) if @current_font
           when "extend"
-            @current_font[:extend] = true if @current_font
+            @current_font[:extend] = truthy_val?(attributes) if @current_font
           when "u"
             if @current_font
               val = attributes["val"]
@@ -303,6 +303,11 @@ module Xlsxrb
 
         private
 
+        def truthy_val?(attributes)
+          val = attributes["val"]
+          val.nil? || (val != "0" && val.downcase != "false")
+        end
+
         def parse_color(attributes)
           if @inside_mru_colors
             c = {}