ooxml_parser 0.6.0 → 0.8.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: af046e45d945d773885e2c8c9c8becef406a7e0c50a1f71161e1252b73295441
-  data.tar.gz: 16e54721a7a8c6f066693343ad8950c88d136531b9f6a3a53d7d7154c199c699
+  metadata.gz: 07201a9a59d3d4b457115c2faf6c3e419eb9322064c25e8d793adb7439cd70ba
+  data.tar.gz: 18fe77d063dc4e6ef7554abcec151ab5987eb1754a6dcba80cbc258092e3a460
 SHA512:
-  metadata.gz: 216969571d18f07e04e10c7499628dc59d131a531a7cf1f8abdaf390bba04f04912b5f4f3bc268cd06b45d4853eb9fe3347a1ef136cfa56f2e8f6b88fccde7ab
-  data.tar.gz: a14b99b7e06bfd36c38fe2e52774e617cc60f94193314b5bad6200fe5d973cc854a8774aa7a43bdcd556bfec6a2f53c0dee59ff978442f49663bd8e3c43a5d2d
+  metadata.gz: a4583846a0c88a4dbb30c2ad727c3f444f15ef340904532cdc106b0cad70eaa2ee71ca5f8cdc6e26adac38e0caed12ca78519499b5a4391d7b0e9869a418488e
+  data.tar.gz: 35385dccda157dd0fef6d22c9ce0dde82630ab5861513eaee733f379532388722449fd993ff69327ca2b66f95b2219646d21bfa7b75439b6085db5cd1500fa22

data/lib/ooxml_parser.rb

@@ -29,6 +29,7 @@ module OoxmlParser
     @configuration ||= Configuration.new
   end
 
+  # Reset settings to default
   def self.reset
     @configuration = Configuration.new
   end

data/lib/ooxml_parser/common_parser/common_data/alternate_content/chart/chart.rb

@@ -41,6 +41,9 @@ module OoxmlParser
       @parent = parent
     end
 
+    # Parse properties of Chart
+    # @param chart_prop_node [Nokogiri::XML:Element] node to parse
+    # @return [void]
     def parse_properties(chart_prop_node)
       chart_prop_node.xpath('*').each do |chart_props_node_child|
         case chart_props_node_child.name
@@ -71,6 +74,9 @@ module OoxmlParser
     extend Gem::Deprecate
     deprecate :data, 'series points interface', 2020, 1
 
+    # Parse  Chart
+    # @param parent [OOOXMLDocumentObject] parent of chart
+    # @return [Chart] result of parsing
     def self.parse(parent: nil)
       chart = Chart.new(parent: parent)
       chart_xml = chart.parse_xml(OOXMLDocumentObject.current_xml)

data/lib/ooxml_parser/common_parser/common_data/alternate_content/chart/chart/series.rb

@@ -11,7 +11,7 @@ module OoxmlParser
     attr_reader :order
     # @return [SeriesText] text of series
     attr_accessor :text
-    # @return [Categories] categories of chart
+    # @return [SeriesText] categories of chart
     attr_accessor :categories
     # @return [DisplayLabelsProperties]
     attr_accessor :display_labels
@@ -35,7 +35,7 @@ module OoxmlParser
         when 'tx'
           @text = SeriesText.new(parent: self).parse(node_child)
         when 'cat'
-          @categories = Categories.new(parent: self).parse(node_child)
+          @categories = SeriesText.new(parent: self).parse(node_child)
         when 'dLbls'
           @display_labels = DisplayLabelsProperties.new(parent: self).parse(node_child)
         when 'val'

data/lib/ooxml_parser/common_parser/common_data/alternate_content/chart/chart/series/series_text.rb

@@ -20,6 +20,4 @@ module OoxmlParser
       self
     end
   end
-
-  Categories = SeriesText
 end

data/lib/ooxml_parser/common_parser/common_data/alternate_content/drawing/docx_drawing.rb

@@ -30,34 +30,15 @@ module OoxmlParser
         when 'inline'
           @type = :inline
         end
-        @properties.distance_from_text = DocxDrawingDistanceFromText.new(parent: self).parse(node_child)
-        @properties.wrap = DocxWrapDrawing.new(parent: self).parse(node_child)
-        node_child.attributes.each do |key, value|
-          case key
-          when 'relativeHeight'
-            @properties.relative_height = value.value.to_i
-          end
-        end
         node_child.xpath('*').each do |content_node_child|
           case content_node_child.name
-          when 'simplePos'
-            @properties.simple_position = OOXMLCoordinates.parse(content_node_child)
-          when 'extent'
-            @properties.object_size = OOXMLCoordinates.parse(content_node_child, x_attr: 'cx', y_attr: 'cy', unit: :emu)
           when 'graphic'
             @graphic = DocxGraphic.new(parent: self).parse(content_node_child)
-          when 'positionV'
-            @properties.vertical_position = DocxDrawingPosition.new(parent: self).parse(content_node_child)
-          when 'positionH'
-            @properties.horizontal_position = DocxDrawingPosition.new(parent: self).parse(content_node_child)
-          when 'sizeRelH'
-            @properties.size_relative_horizontal = SizeRelativeHorizontal.new(parent: self).parse(content_node_child)
-          when 'sizeRelV'
-            @properties.size_relative_vertical = SizeRelativeVertical.new(parent: self).parse(content_node_child)
           when 'docPr'
             @doc_properties = DocProperties.new(parent: self).parse(content_node_child)
           end
         end
+        @properties.parse(node_child)
       end
       self
     end

data/lib/ooxml_parser/common_parser/common_data/alternate_content/drawing/drawing_properties/docx_drawing_distance_from_text.rb

@@ -5,6 +5,9 @@ module OoxmlParser
   class DocxDrawingDistanceFromText < OOXMLDocumentObject
     attr_accessor :top, :bottom, :left, :right
 
+    # Parse DocxDrawingDistanceFromText object
+    # @param node [Nokogiri::XML:Element] node to parse
+    # @return [DocxDrawingDistanceFromText] result of parsing
     def parse(node)
       node.attributes.each do |key, value|
         case key

data/lib/ooxml_parser/common_parser/common_data/alternate_content/drawing/drawing_properties/docx_drawing_properties.rb

@@ -8,9 +8,56 @@ require_relative 'docx_wrap_drawing'
 module OoxmlParser
   # Docx Drawing Properties
   class DocxDrawingProperties < OOXMLDocumentObject
-    attr_accessor :distance_from_text, :simple_position, :wrap, :object_size, :vertical_position, :horizontal_position,
-                  :relative_height
-    attr_accessor :size_relative_horizontal
-    attr_accessor :size_relative_vertical
+    # @return [DocxDrawingDistanceFromText] distance from text
+    attr_reader :distance_from_text
+    # @return [DocxWrapDrawing] wrap of drawing
+    attr_reader :wrap
+    # @return [Integer] relative height of object
+    attr_reader :relative_height
+    # @return [OOXMLCoordinates] simple position of object
+    attr_reader :simple_position
+    # @return [OOXMLCoordinates] size of object
+    attr_reader :object_size
+    # @return [DocxDrawingPosition] vertical position
+    attr_reader :vertical_position
+    # @return [DocxDrawingPosition] horizontal position
+    attr_reader :horizontal_position
+    # @return [SizeRelativeHorizontal] size of drawing relative to horizontal
+    attr_reader :size_relative_horizontal
+    # @return [SizeRelativeVertical] size of drawing relative to vertical
+    attr_reader :size_relative_vertical
+
+    # Parse DocxDrawingProperties
+    # @param [Nokogiri::XML:Node] node with DocxDrawingProperties
+    # @return [DocxDrawingProperties] result of parsing
+    def parse(node)
+      @distance_from_text = DocxDrawingDistanceFromText.new(parent: self).parse(node)
+      @wrap = DocxWrapDrawing.new(parent: self).parse(node)
+
+      node.attributes.each do |key, value|
+        case key
+        when 'relativeHeight'
+          @relative_height = value.value.to_i
+        end
+      end
+
+      node.xpath('*').each do |content_node_child|
+        case content_node_child.name
+        when 'simplePos'
+          @simple_position = OOXMLCoordinates.parse(content_node_child)
+        when 'extent'
+          @object_size = OOXMLCoordinates.parse(content_node_child, x_attr: 'cx', y_attr: 'cy', unit: :emu)
+        when 'positionV'
+          @vertical_position = DocxDrawingPosition.new(parent: self).parse(content_node_child)
+        when 'positionH'
+          @horizontal_position = DocxDrawingPosition.new(parent: self).parse(content_node_child)
+        when 'sizeRelH'
+          @size_relative_horizontal = SizeRelativeHorizontal.new(parent: self).parse(content_node_child)
+        when 'sizeRelV'
+          @size_relative_vertical = SizeRelativeVertical.new(parent: self).parse(content_node_child)
+        end
+      end
+      self
+    end
   end
 end

data/lib/ooxml_parser/common_parser/common_data/alternate_content/drawing/drawing_properties/ooxml_coordinates.rb

@@ -18,6 +18,7 @@ module OoxmlParser
            end
     end
 
+    # @return [String] result of convert of object to string
     def to_s
       '(' + @x.to_s + '; ' + @y.to_s + ')'
     end

data/lib/ooxml_parser/common_parser/common_data/alternate_content/drawing/drawing_properties/ooxml_size.rb

@@ -28,6 +28,9 @@ module OoxmlParser
       self
     end
 
+    # Compare this object to other
+    # @param other [Object] any other object
+    # @return [True, False] result of comparision
     def ==(other)
       (to_base_unit.value - other.to_base_unit.value).abs < 10**(OoxmlParser.configuration.accuracy + 2)
     end
@@ -40,7 +43,7 @@ module OoxmlParser
     # @return [String] string representation of size
     def to_s(unit = :centimeter)
       converted = to_unit(unit)
-      "#{converted.value} #{converted.unit}"
+      "#{converted.value.to_f} #{converted.unit}"
     end
 
     # Convert all values to one same base unit

data/lib/ooxml_parser/common_parser/common_data/alternate_content/drawing/graphic/picture/docx_blip.rb

@@ -10,6 +10,7 @@ module OoxmlParser
 
     alias path path_to_media_file
 
+    # @return [String] result of convert of object to string
     def to_str
       path_to_media_file
     end

data/lib/ooxml_parser/common_parser/common_data/alternate_content/drawing/graphic/picture/docx_blip/file_reference.rb

@@ -10,6 +10,9 @@ module OoxmlParser
     # @return [String] content of file
     attr_accessor :content
 
+    # Parse FileReference object
+    # @param node [Nokogiri::XML:Element] node to parse
+    # @return [FileReference] result of parsing
     def parse(node)
       node.attributes.each do |key, value|
         case key
@@ -26,6 +29,7 @@ module OoxmlParser
         return self
       end
       return self if @path == 'NULL'
+      return self if @path.match?(URI::DEFAULT_PARSER.make_regexp)
 
       full_path_to_file = OOXMLDocumentObject.path_to_folder + OOXMLDocumentObject.root_subfolder + @path.gsub('..', '')
       if File.exist?(full_path_to_file)

data/lib/ooxml_parser/common_parser/common_data/alternate_content/drawing/graphic/picture/docx_picture.rb

@@ -28,6 +28,4 @@ module OoxmlParser
       self
     end
   end
-
-  Picture = DocxPicture
 end

data/lib/ooxml_parser/common_parser/common_data/alternate_content/drawing/graphic/shape/docx_shape/ooxml_text_box.rb

@@ -3,7 +3,7 @@
 module OoxmlParser
   # Class for parsing `txbx` tags
   class OOXMLTextBox < OOXMLDocumentObject
-    attr_accessor :properties, :elements
+    attr_accessor :elements
 
     def initialize(parent: nil)
       @elements = []
@@ -21,8 +21,6 @@ module OoxmlParser
           @elements << DocxParagraph.new(parent: self).parse(node_child, index)
         when 'tbl'
           @elements << Table.new(parent: self).parse(node_child, index)
-        when 'bodyPr'
-          @properties = OOXMLShapeBodyProperties.new(parent: self).parse(node_child)
         end
       end
       self

data/lib/ooxml_parser/common_parser/common_data/alternate_content/drawing/graphic/shape/shape_properties/color/docx_color_scheme.rb

@@ -35,6 +35,7 @@ module OoxmlParser
       self
     end
 
+    # @return [String] result of convert of object to string
     def to_s
       "Color: #{@color}, type: #{type}"
     end

data/lib/ooxml_parser/common_parser/common_data/alternate_content/drawing/graphic/shape/shape_properties/custom_geometry/docx_custom_geometry/docx_shape_line_path/docx_shape_line_element.rb

@@ -23,8 +23,6 @@ module OoxmlParser
         @type = :arc
       when 'cubicBezTo'
         @type = :cubic_bezier
-      when 'quadBezTo'
-        @type = :quadratic_bezier
       when 'close'
         @type = :close
       end

data/lib/ooxml_parser/common_parser/common_data/alternate_content/drawing/graphic/shape/shape_properties/docx_shape_line.rb

@@ -16,6 +16,7 @@ module OoxmlParser
       @parent = parent
     end
 
+    # @return [Integer] stroke size of object
     def stroke_size
       if @color_scheme.nil? || @color_scheme.color == :none
         0

data/lib/ooxml_parser/common_parser/common_data/alternate_content/drawing/graphic/shape/shape_properties/docx_shape_properties.rb

@@ -10,8 +10,18 @@ require_relative 'custom_geometry/ooxml_custom_geometry'
 module OoxmlParser
   # DOCX Shape Properties
   class DocxShapeProperties < OOXMLDocumentObject
-    attr_accessor :shape_size, :preset_geometry, :fill_color, :text_box, :line, :custom_geometry
-    attr_accessor :blip_fill
+    # @return [DocxShapeSize] size of shape
+    attr_reader :shape_size
+    # @return [PresetGeometry] preset geometry of object
+    attr_reader :preset_geometry
+    # @return [DocxColor] color of object
+    attr_reader :fill_color
+    # @return [DocxShapeLine] line info
+    attr_reader :line
+    # @return [PresetGeometry] is some geometry custom
+    attr_reader :custom_geometry
+    # @return [BlipFill] BlipFill data
+    attr_reader :blip_fill
 
     alias transform shape_size
     alias fill fill_color
@@ -33,8 +43,6 @@ module OoxmlParser
           @shape_size = DocxShapeSize.new(parent: self).parse(node_child)
         when 'prstGeom'
           @preset_geometry = PresetGeometry.new(parent: self).parse(node_child)
-        when 'txbx'
-          @text_box = TextBox.parse_list(node_child)
         when 'ln'
           @line = DocxShapeLine.new(parent: self).parse(node_child)
         when 'blipFill'

data/lib/ooxml_parser/common_parser/common_data/alternate_content/drawing/graphic/shape/shape_properties/docx_shape_size.rb

@@ -3,9 +3,16 @@
 module OoxmlParser
   # Docx Shape Size
   class DocxShapeSize < OOXMLDocumentObject
-    attr_accessor :rotation, :offset, :extent
-    attr_accessor :child_offset
-    attr_accessor :child_extent
+    # @return [OoxmlSize] rotation info
+    attr_reader :rotation
+    # @return [OOXMLCoordinates] offset info
+    attr_reader :offset
+    # @return [OOXMLCoordinates] extent info
+    attr_reader :extent
+    # @return [OOXMLCoordinates] child offset
+    attr_reader :child_offset
+    # @return [OOXMLCoordinates] child extent
+    attr_reader :child_extent
     # @return [True, False] is image flipped horizontally
     attr_reader :flip_horizontal
     # @return [True, False] is image flipped vertically

data/lib/ooxml_parser/common_parser/common_data/alternate_content/picture/old_docx_picture.rb

@@ -6,7 +6,7 @@ require_relative 'group/old_docx_group'
 module OoxmlParser
   # Fallback DOCX Picture
   class OldDocxPicture < OOXMLDocumentObject
-    attr_accessor :data, :type, :style_number
+    attr_accessor :data, :type
 
     # Parse OldDocxPicture object
     # @param node [Nokogiri::XML:Element] node to parse
@@ -20,8 +20,6 @@ module OoxmlParser
         when 'group'
           @type = :group
           @data = OldDocxGroup.new(parent: self).parse(node_child)
-        when 'style'
-          @style_number = node_child.attribute('val').value.to_i
         end
       end
       self

data/lib/ooxml_parser/common_parser/common_data/alternate_content/picture/shape/old_docx_shape_properties.rb

@@ -3,7 +3,7 @@
 module OoxmlParser
   # Fallback DOCX Shape Properties
   class OldDocxShapeProperties < OOXMLDocumentObject
-    attr_accessor :fill_color, :opacity, :stroke_color, :stroke_weight
+    attr_accessor :fill_color, :stroke_color, :stroke_weight
 
     # Parse OldDocxShapeProperties object
     # @param node [Nokogiri::XML:Element] node to parse
@@ -13,8 +13,6 @@ module OoxmlParser
         case key
         when 'fillcolor'
           @fill_color = Color.new(parent: self).parse_hex_string(value.value.delete('#'))
-        when 'opacity'
-          @opacity = value.value.to_f
         when 'strokecolor'
           @stroke_color = Color.new(parent: self).parse_hex_string(value.value.delete('#'))
         when 'strokeweight'

data/lib/ooxml_parser/common_parser/common_data/borders_properties.rb

@@ -27,12 +27,15 @@ module OoxmlParser
       size.zero? && val == :none
     end
 
+    # @return [String] result of convert of object to string
     def to_s
       return '' if nil?
 
       "borders color: #{@color}, size: #{size}, space: #{@space}, value: #{@val}"
     end
 
+    # Method to copy object
+    # @return [BordersProperties] copied object
     def copy
       BordersProperties.new(@color, size, @val, @space)
     end

data/lib/ooxml_parser/common_parser/common_data/color.rb

@@ -12,79 +12,6 @@ module OoxmlParser
   # Class for Color in RGB
   class Color < OOXMLDocumentObject
     include ColorHelper
-    # @return [Array] Deprecated Indexed colors
-    # List of color duplicated from `OpenXML Sdk IndexedColors` class
-    # See https://msdn.microsoft.com/en-us/library/documentformat.openxml.spreadsheet.indexedcolors.aspx
-    COLOR_INDEXED =
-      %w[
-        000000
-        FFFFFF
-        FF0000
-        00FF00
-        0000FF
-        FFFF00
-        FF00FF
-        00FFFF
-        000000
-        FFFFFF
-        FF0000
-        00FF00
-        0000FF
-        FFFF00
-        FF00FF
-        00FFFF
-        800000
-        008000
-        000080
-        808000
-        800080
-        008080
-        C0C0C0
-        808080
-        9999FF
-        993366
-        FFFFCC
-        CCFFFF
-        660066
-        FF8080
-        0066CC
-        CCCCFF
-        000080
-        FF00FF
-        FFFF00
-        00FFFF
-        800080
-        800000
-        008080
-        0000FF
-        00CCFF
-        CCFFFF
-        CCFFCC
-        FFFF99
-        99CCFF
-        FF99CC
-        CC99FF
-        FFCC99
-        3366FF
-        33CCCC
-        99CC00
-        FFCC00
-        FF9900
-        FF6600
-        666699
-        969696
-        003366
-        339966
-        003300
-        333300
-        993300
-        993366
-        333399
-        333333
-        n/a
-        n/a
-      ].freeze
-
     # @return [Integer] Value of Red Part
     attr_accessor :red
     # @return [Integer] Value of Green Part
@@ -103,7 +30,7 @@ module OoxmlParser
 
     alias set_alpha_channel alpha_channel=
 
-    attr_accessor :position
+    # @return [ColorProperties] properties of color
     attr_accessor :properties
 
     # Value of color if non selected
@@ -119,6 +46,7 @@ module OoxmlParser
       @parent = parent
     end
 
+    # @return [String] result of convert of object to string
     def to_s
       if @red == VALUE_FOR_NONE_COLOR && @green == VALUE_FOR_NONE_COLOR && @blue == VALUE_FOR_NONE_COLOR
         'none'
@@ -127,33 +55,43 @@ module OoxmlParser
       end
     end
 
+    # @return [String] inspect of object for debug means
     def inspect
       to_s
     end
 
+    # @return [String] color in hex value
     def to_hex
       (@red.to_s(16).rjust(2, '0') + @green.to_s(16).rjust(2, '0') + @blue.to_s(16).rjust(2, '0')).upcase
     end
 
     alias to_int16 to_hex
 
+    # @return [True, False] is color default
     def none?
       (@red == VALUE_FOR_NONE_COLOR) && (@green == VALUE_FOR_NONE_COLOR) && (@blue == VALUE_FOR_NONE_COLOR) ||
         (style == :nil)
     end
 
+    # @return [True, False] is color not default
     def any?
       !none?
     end
 
+    # @return [True, False] is color white
     def white?
       (@red == 255) && (@green == 255) && (@blue == 255)
     end
 
+    # Method to copy object
+    # @return [Color] copied object
     def copy
       Color.new(@red, @green, @blue)
     end
 
+    # Compare this object to other
+    # @param other [Object] any other object
+    # @return [True, False] result of comparision
     def ==(other)
       if other.is_a?(Color)
         if (@red == other.red) && (@green == other.green) && (@blue == other.blue)
@@ -190,18 +128,27 @@ module OoxmlParser
       (self.red - red).abs < delta && (self.green - green).abs < delta && (self.blue - blue).abs < delta ? true : false
     end
 
+    # Apply tint to color
+    # @param tint [Integer] tint to apply
+    # @return [void]
     def calculate_with_tint!(tint)
       @red += (tint.to_f * (255 - @red)).to_i
       @green += (tint.to_f * (255 - @green)).to_i
       @blue += (tint.to_f * (255 - @blue)).to_i
     end
 
+    # Apply shade to color
+    # @param shade [Integer] shade to apply
+    # @return [void]
     def calculate_with_shade!(shade)
       @red = (@red * shade.to_f).to_i
       @green = (@green * shade.to_f).to_i
       @blue = (@blue * shade.to_f).to_i
     end
 
+    # Parse color scheme data
+    # @param scheme_color_node [Nokogiri::XML:Element] node to parse
+    # @return [Color] result of parsing
     def parse_scheme_color(scheme_color_node)
       color = root_object.theme.color_scheme[scheme_color_node.attribute('val').value.to_sym].color
       hls = HSLColor.rgb_to_hsl(color)
@@ -229,6 +176,9 @@ module OoxmlParser
       self
     end
 
+    # Parse color model data
+    # @param color_model_parent_node [Nokogiri::XML:Element] node to parse
+    # @return [Color] result of parsing
     def parse_color_model(color_model_parent_node)
       color = nil
       tint = nil
@@ -240,9 +190,6 @@ module OoxmlParser
           end
         end
         case color_model_node.name
-        when 'scrgbClr'
-          color = Color.new(color_model_node.attribute('r').value.to_i, color_model_node.attribute('g').value.to_i, color_model_node.attribute('b').value.to_i)
-          color.alpha_channel = ColorAlphaChannel.parse(color_model_node)
         when 'srgbClr'
           color = Color.new.parse_hex_string(color_model_node.attribute('val').value)
           color.alpha_channel = ColorAlphaChannel.parse(color_model_node)
@@ -261,6 +208,9 @@ module OoxmlParser
       self
     end
 
+    # Parse color data
+    # @param color_node [Nokogiri::XML:Element] node to parse
+    # @return [Color] result of parsing
     def parse_color(color_node)
       case color_node.name
       when 'srgbClr'
@@ -280,6 +230,7 @@ module OoxmlParser
     end
 
     class << self
+      # @return [Color] random color
       def generate_random_color
         Color.new(rand(256), rand(256), rand(256))
       end
@@ -293,19 +244,31 @@ module OoxmlParser
         const_array_name.map { |current_color| Color.parse_string(current_color) }
       end
 
+      # @param index [Integer] index to get
+      # @return [Color] color by it's index
       def get_rgb_by_color_index(index)
-        color_by_index = COLOR_INDEXED[index]
+        color_by_index = color_indexes[index]
         return :unknown if color_by_index.nil?
 
         color_by_index == 'n/a' ? Color.new : Color.new.parse_hex_string(color_by_index)
       end
 
+      # @return [Array] Deprecated Indexed colors
+      #   List of color duplicated from `OpenXML Sdk IndexedColors` class
+      #   See https://msdn.microsoft.com/en-us/library/documentformat.openxml.spreadsheet.indexedcolors.aspx
+      def color_indexes
+        @color_indexes ||= File.readlines("#{__dir__}/color/color_indexes.list", chomp: true)
+      end
+
+      # Parse color from string
+      # @param str [String] string to parse
+      # @return [Color] result of parsing
       def parse_string(str)
         return str if str.is_a?(Color)
         return Color.new(VALUE_FOR_NONE_COLOR, VALUE_FOR_NONE_COLOR, VALUE_FOR_NONE_COLOR) if str == 'none' || str == '' || str == 'transparent' || str.nil?
 
         split = if str.include?('RGB (') || str.include?('rgb(')
-                  str.gsub(/[(RGBrgb\(\) )]/, '').split(',')
+                  str.gsub(/[(RGBrgb() )]/, '').split(',')
                 elsif str.include?('RGB ') || str.include?('rgb')
                   str.gsub(/RGB |rgb/, '').split(', ')
                 else
@@ -317,6 +280,9 @@ module OoxmlParser
 
       alias parse parse_string
 
+      # Convert other object type to Color
+      # @param something [Object] object to convert
+      # @return [Color] result of conversion
       def to_color(something)
         case something
         when SchemeColor