uniword 1.0.10 → 1.2.4

data/lib/uniword/ooxml/relationships/package_relationships.rb

@@ -24,6 +24,13 @@ module Uniword
         # Generate package-level .rels file
         #
         # @return [PackageRelationships] Relationships object for _rels/.rels
+        def self.next_available_rid(relationships)
+          max = relationships.relationships.filter_map do |r|
+            r.id[/\ArId(\d+)\z/, 1]&.to_i
+          end.max || 0
+          "rId#{max + 1}"
+        end
+
         def self.generate_package_rels
           new(
             relationships: [

data/lib/uniword/presentationml/slide_id.rb

@@ -1,7 +1,6 @@
 # frozen_string_literal: true
 
 require "lutaml/model"
-require_relative "../properties/relationship_id"
 
 module Uniword
   module Presentationml

data/lib/uniword/presentationml/slide_master_id.rb

@@ -1,7 +1,6 @@
 # frozen_string_literal: true
 
 require "lutaml/model"
-require_relative "../properties/relationship_id"
 
 module Uniword
   module Presentationml

data/lib/uniword/properties/bold.rb

@@ -1,7 +1,5 @@
 # frozen_string_literal: true
 
-require_relative "boolean_element_factory"
-
 module Uniword
   module Properties
     BooleanElementFactory.define("b", "Bold")

data/lib/uniword/properties/boolean_formatting.rb

@@ -1,7 +1,5 @@
 # frozen_string_literal: true
 
-require_relative "boolean_element_factory"
-
 module Uniword
   module Properties
     # Run-level boolean formatting elements

data/lib/uniword/properties/borders.rb

@@ -28,8 +28,8 @@ module Uniword
         namespace Ooxml::Namespaces::WordProcessingML
 
         map_element "top", to: :top, render_nil: false
-        map_element "bottom", to: :bottom, render_nil: false
         map_element "left", to: :left, render_nil: false
+        map_element "bottom", to: :bottom, render_nil: false
         map_element "right", to: :right, render_nil: false
         map_element "between", to: :between, render_nil: false
         map_element "bar", to: :bar, render_nil: false

data/lib/uniword/properties/italic.rb

@@ -1,7 +1,5 @@
 # frozen_string_literal: true
 
-require_relative "boolean_element_factory"
-
 module Uniword
   module Properties
     BooleanElementFactory.define("i", "Italic")

data/lib/uniword/properties/numbering_properties.rb

@@ -17,8 +17,8 @@ module Uniword
         element "numPr"
         namespace Ooxml::Namespaces::WordProcessingML
 
-        map_element "numId", to: :num_id, render_nil: false
         map_element "ilvl", to: :ilvl, render_nil: false
+        map_element "numId", to: :num_id, render_nil: false
       end
     end
   end

data/lib/uniword/properties/style_reference.rb

@@ -17,6 +17,19 @@ module Uniword
         map_attribute "val", to: :value
       end
 
+      # Cast raw style-id strings into references (mirrors Text.cast), so
+      # `properties.style = "Heading1"` stores a proper StyleReference
+      def self.cast(value)
+        case value
+        when StyleReference, nil
+          value
+        when String
+          new(value: value)
+        else
+          super
+        end
+      end
+
       # Compare with another StyleReference or a string
       def ==(other)
         if other.is_a?(StyleReference)

data/lib/uniword/quality/rules/heading_hierarchy_rule.rb

@@ -72,7 +72,7 @@ module Uniword
       # @param paragraph [Paragraph] The paragraph to check
       # @return [Integer, nil] Heading level (1-9) or nil if not a heading
       def extract_heading_level(paragraph)
-        style_name = paragraph.properties&.style
+        style_name = paragraph.style
         return nil unless style_name
 
         match = style_name.to_s.match(HEADING_PATTERN)

data/lib/uniword/quality/rules/style_consistency_rule.rb

@@ -76,7 +76,7 @@ module Uniword
       # @param paragraph [Paragraph] The paragraph to check
       # @return [Boolean] true if has style
       def has_style?(paragraph)
-        style = paragraph.properties&.style
+        style = paragraph.style
         style&.value && !style.value.empty?
       end
 

data/lib/uniword/review/review_manager.rb

@@ -1,7 +1,5 @@
 # frozen_string_literal: true
 
-require_relative "accept_reject"
-
 module Uniword
   module Review
     # Orchestrator for comments and tracked changes in a document.

data/lib/uniword/template/helpers/loop_helper.rb

@@ -105,12 +105,12 @@ module Uniword
           # recursively process all text nodes
           return unless element.is_a?(Uniword::Wordprocessingml::Run)
 
-          text = element.text
+          text = element.text_string
           # Simple variable replacement ({{var}})
           text.scan(/\{\{([^@].+?)\}\}/).each do |match|
             var_name = match[0]
             value = resolver.resolve(var_name)
-            element.text = text.gsub("{{#{var_name}}}", value.to_s)
+            element.substitute("{{#{var_name}}}", value.to_s)
           end
         end
       end

data/lib/uniword/template/variable_resolver.rb

@@ -175,7 +175,20 @@ module Uniword
 
         return false if left_num.nil? || right_num.nil?
 
-        left_num.public_send(operator, right_num)
+        case operator
+        when :== then left_num == right_num
+        when :!= then left_num != right_num
+        when :>  then left_num > right_num
+        when :>= then left_num >= right_num
+        when :<  then left_num < right_num
+        when :<= then left_num <= right_num
+        when :+  then left_num + right_num
+        when :-  then left_num - right_num
+        when :*  then left_num * right_num
+        when :/  then left_num / right_num
+        else
+          raise ArgumentError, "Unsupported operator: #{operator.inspect}"
+        end
       end
 
       # Convert value to number

data/lib/uniword/theme/theme_xml_parser.rb

@@ -79,7 +79,7 @@ module Uniword
 
           # Set the color using the appropriate attribute
           attr_name = color_name == "folHlink" ? :fol_hlink : color_name.to_sym
-          scheme.public_send(:"#{attr_name}=", color_obj)
+          scheme.assign_color_by_name(attr_name, color_obj)
         end
 
         # Sync hash interface

data/lib/uniword/toc/toc_generator.rb

@@ -142,14 +142,7 @@ module Uniword
       # @param paragraph [Wordprocessingml::Paragraph] Paragraph to inspect
       # @return [String, nil] Style name or nil
       def resolve_style_name(paragraph)
-        style_ref = paragraph.properties&.style
-        return nil unless style_ref
-
-        if style_ref.is_a?(Uniword::Properties::StyleReference)
-          style_ref.value
-        else
-          style_ref.to_s
-        end
+        paragraph.style&.value
       end
 
       # Extract text content from a paragraph.
@@ -197,9 +190,7 @@ module Uniword
       def build_title_paragraph
         para = Wordprocessingml::Paragraph.new
         para.properties = Wordprocessingml::ParagraphProperties.new
-        para.properties.style = Uniword::Properties::StyleReference.new(
-          value: "TOCHeading",
-        )
+        para.properties.style = "TOCHeading"
 
         run = Wordprocessingml::Run.new
         run.text = "Table of Contents"
@@ -251,9 +242,7 @@ module Uniword
       def build_entry_paragraph(entry)
         para = Wordprocessingml::Paragraph.new
         para.properties = Wordprocessingml::ParagraphProperties.new
-        para.properties.style = Uniword::Properties::StyleReference.new(
-          value: "TOC#{entry.level}",
-        )
+        para.properties.style = "TOC#{entry.level}"
 
         run = Wordprocessingml::Run.new
         run.text = entry.text

data/lib/uniword/transformation/mhtml_element_renderer.rb

@@ -69,7 +69,8 @@ module Uniword
       # Convert OOXML Run to HTML
       def run_to_html(run)
         # Handle breaks and drawings first (these never get style wrapping)
-        return break_to_html(run.break) if run.break
+        # (parsed runs hold [] for absent collections, so check emptiness)
+        return break_to_html(run.break) if Array(run.break).any?
         if run.drawings && !run.drawings.empty?
           return drawing_to_html(run.drawings.first)
         end
@@ -105,7 +106,8 @@ module Uniword
             # Skip wrapping for internal OOXML styles that shouldn't produce
             # visible HTML class wrappers
             is_internal_style = %w[stem hyperlink].include?(style_id)
-            is_stem_spacer = (style_id == "stem") && run.text.to_s.strip.empty?
+            is_stem_spacer = (style_id == "stem") &&
+              run.text_string.strip.empty?
             unless is_internal_style || is_stem_spacer
               style_val = run_style_to_class(props.style.value)
               content = %(<span class="#{style_val}">#{content}</span>)
@@ -125,7 +127,7 @@ module Uniword
         elsif run.tab
           tab_to_html(run)
         else
-          text = run.text.to_s
+          text = run.text_string
           return "" if text.empty?
 
           escape_html(text)
@@ -142,13 +144,15 @@ module Uniword
         %(<span class="MsoEndnoteReference"><span style="mso-special-character:endnote"></span></span>)
       end
 
-      # Convert OOXML Break to HTML
-      def break_to_html(brk)
-        if brk.type == "page"
-          %(<br clear="all" style="mso-special-character:line-break;page-break-before:always" />)
-        else
-          %(<br />)
-        end
+      # Convert OOXML Breaks to HTML, one <br> per Break element
+      def break_to_html(breaks)
+        Array(breaks).map do |brk|
+          if brk.type == "page"
+            %(<br clear="all" style="mso-special-character:line-break;page-break-before:always" />)
+          else
+            %(<br />)
+          end
+        end.join
       end
 
       # Convert OOXML FieldChar to HTML span
@@ -272,7 +276,9 @@ module Uniword
         end
       end
 
-      # Convert OOXML OMath to HTML (wrapped in stem span)
+      # Convert an OOXML OMath model to HTML (wrapped in stem span).
+      # Accepts Omml::Models::CTOMath/OMath/OMathPara (the typed attribute
+      # values on Paragraph) — anything responding to `to_xml` works.
       def omath_to_html(o_math)
         xml = o_math.to_xml
         # Strip XML declaration and namespace prefixes for clean inline HTML
@@ -491,23 +497,87 @@ module Uniword
 
       private
 
-      # Get paragraph content including runs, hyperlinks, SDTs, and oMath
-      # Interleaves all element types in correct order using position markers.
+      # Render paragraph content (runs, hyperlinks, SDTs, oMath) in the
+      # document order captured by `paragraph.element_order`.
+      #
+      # Walking element_order (rather than `build_position_map` keyed on
+      # run_position) keeps MHTML positioning in lockstep with the DOCX
+      # serializer, which is the single source of truth for interleaving.
+      # It also handles inline math, which is now an Omml::Models::CTOMath
+      # instance without a run_position attribute.
       def paragraph_content_to_html(paragraph)
+        order = paragraph.element_order
+        return paragraph_content_to_html_legacy(paragraph) if order.nil? || order.empty?
+
+        run_idx = 0
+        hl_idx = 0
+        omath_idx = 0
+        omp_idx = 0
+        sdt_idx = 0
+
+        parts = []
+        order.each do |entry|
+          next unless entry.element?
+
+          case entry.name
+          when "r"
+            collect_run_at(paragraph, run_idx, parts)
+            run_idx += 1
+          when "hyperlink"
+            hl = paragraph.hyperlinks[hl_idx]
+            parts << hyperlink_to_html(hl) if hl
+            hl_idx += 1
+          when "oMath"
+            om = paragraph.o_maths[omath_idx]
+            parts << omath_to_html(om) if om
+            omath_idx += 1
+          when "oMathPara"
+            omp = paragraph.o_math_paras[omp_idx]
+            parts << omath_to_html(omp) if omp
+            omp_idx += 1
+          when "sdt"
+            sdt = paragraph.sdts[sdt_idx]
+            parts << sdt_to_inline_html(sdt) if sdt
+            sdt_idx += 1
+          end
+        end
+
+        # Defensive: emit any math/SDTs not represented in element_order.
+        paragraph.o_math_paras.each_with_index do |omp, i|
+          next if i < omp_idx
+
+          parts << omath_to_html(omp)
+        end
+        paragraph.sdts.each_with_index do |sdt, i|
+          next if i < sdt_idx
+
+          parts << sdt_to_inline_html(sdt)
+        end
+
+        parts.join
+      end
+
+      def collect_run_at(paragraph, idx, parts)
+        run = paragraph.runs[idx]
+        return unless run
+
+        parts << if run.is_a?(Uniword::Wordprocessingml::StructuredDocumentTag)
+                   sdt_to_inline_html(run)
+                 else
+                   run_to_html(run)
+                 end
+      end
+
+      # Fallback for Paragraphs constructed without element_order tracking
+      # (older code paths). Preserves the previous behavior of appending
+      # inline math/hyperlinks at the end.
+      def paragraph_content_to_html_legacy(paragraph)
         run_count = paragraph.runs.size
 
-        # Build position-indexed maps of hyperlinks and oMaths
         hl_by_pos = build_position_map(paragraph.hyperlinks, run_count)
-        omath_by_pos = build_position_map(paragraph.o_maths, run_count)
 
         parts = []
         paragraph.runs.each_with_index do |run, idx|
-          # Insert any oMaths that should appear at this position
-          if omath_by_pos.key?(idx)
-            omath_by_pos[idx].each { |om| parts << omath_to_html(om) }
-          end
-
-          # Insert any hyperlinks that should appear at this position
           if hl_by_pos.key?(idx)
             hl_by_pos[idx].each { |hl| parts << hyperlink_to_html(hl) }
           end
@@ -519,26 +589,13 @@ module Uniword
                    end
         end
 
-        # Append any remaining oMaths at the end
-        if omath_by_pos.key?(run_count)
-          omath_by_pos[run_count].each { |om| parts << omath_to_html(om) }
-        end
-
-        # Append any remaining hyperlinks at the end
         if hl_by_pos.key?(run_count)
           hl_by_pos[run_count].each { |hl| parts << hyperlink_to_html(hl) }
         end
 
-        paragraph.o_math_paras.each do |omp|
-          # Block math: render each oMath inside the oMathPara
-          omp.o_maths.each do |om|
-            parts << omath_to_html(om)
-          end
-        end
-
-        paragraph.sdts.each do |sdt|
-          parts << sdt_to_inline_html(sdt)
-        end
+        paragraph.o_maths.each { |om| parts << omath_to_html(om) }
+        paragraph.o_math_paras.each { |omp| parts << omath_to_html(omp) }
+        paragraph.sdts.each { |sdt| parts << sdt_to_inline_html(sdt) }
 
         parts.join
       end

data/lib/uniword/transformation/mhtml_metadata_builder.rb

@@ -184,7 +184,7 @@ module Uniword
           para_chars = 0
 
           element.runs.each do |run|
-            text = run.text.to_s
+            text = run.text_string
             characters += text.length
             para_chars += text.length
             words += text.split.length

data/lib/uniword/transformation/mhtml_style_builder.rb

@@ -697,6 +697,12 @@ module Uniword
         </style>
       LSB_STYLE
 
+      class << self
+        # Custom CSS override set by callers (e.g., ISO adapter).
+        # When set, #style_block returns this instead of the default.
+        attr_accessor :custom_style_block
+      end
+
       # Build the w:LatentStyles block.
       # @return [String] Static latent styles XML
       def self.latent_styles
@@ -704,9 +710,10 @@ module Uniword
       end
 
       # Build the CSS style block.
-      # @return [String] Static CSS style block
+      # Returns custom CSS if set via custom_style_block=, otherwise default.
+      # @return [String] CSS style block
       def self.style_block
-        STYLE_BLOCK
+        custom_style_block || STYLE_BLOCK
       end
     end
   end

data/lib/uniword/transformation/ooxml_to_html_converter.rb

@@ -54,7 +54,7 @@ module Uniword
       # @param run [Uniword::Wordprocessingml::Run]
       # @return [String] HTML text content with inline formatting
       def self.run_to_html(run)
-        text = escape_html(run.text || "")
+        text = escape_html(run.text_string)
         return text if text.empty?
 
         props = run.properties
@@ -135,12 +135,10 @@ module Uniword
       # @param paragraph [Uniword::Wordprocessingml::Paragraph]
       # @return [String] HTML class/style attribute or empty string
       def self.paragraph_style(paragraph)
-        return "" unless paragraph.properties
-
-        style = paragraph.properties.style
+        style = paragraph.style
         return "" unless style
 
-        " class=\"#{escape_html(style)}\""
+        " class=\"#{escape_html(style.to_s)}\""
       end
 
       # Convert OOXML font size (half-points) to HTML font size

data/lib/uniword/transformation/ooxml_to_mhtml_converter.rb

@@ -18,7 +18,8 @@ module Uniword
     #   output = Uniword::Infrastructure::MimePackager.new(mhtml_doc).build_mime_content
     #
     class OoxmlToMhtmlConverter
-      # Static MsoNormalTable CSS (used in wrap_html_document head)
+      # Static MsoNormalTable CSS (used in wrap_html_document head).
+      # Only used when MhtmlStyleBuilder does not provide custom CSS.
       MSO_NORMAL_TABLE_STYLE = <<~CSS
         <!--[if gte mso 10]>
         <style>
@@ -30,21 +31,12 @@ module Uniword
         	mso-style-noshow:yes;
         	mso-style-priority:99;
         	mso-style-parent:"";
-        	mso-padding-alt:0in 5.4pt 0in 5.4pt;
-        	mso-para-margin-top:0in;
-        	mso-para-margin-right:0in;
-        	mso-para-margin-bottom:8.0pt;
-        	mso-para-margin-left:0in;
-        	line-height:115%;
+        	mso-padding-alt:0cm 5.4pt 0cm 5.4pt;
+        	mso-para-margin:0cm;
+        	mso-para-margin-bottom:.0001pt;
         	mso-pagination:widow-orphan;
-        	font-size:12.0pt;
-        	font-family:"Aptos",sans-serif;
-        	mso-ascii-font-family:Aptos;
-        	mso-ascii-theme-font:minor-latin;
-        	mso-hansi-font-family:Aptos;
-        	mso-hansi-theme-font:minor-latin;
-        	mso-font-kerning:1.0pt;
-        	mso-ligatures:standardcontextual;}
+        	font-size:10pt;
+        	font-family:"Cambria",serif;}
         </style>
         <![endif]-->
       CSS