isodoc 1.6.0 → 1.6.5

data/lib/isodoc/function/section.rb

@@ -8,11 +8,11 @@ module IsoDoc::Function
       insert_tab(out, 1)
     end
 
-    def inline_header_title(out, node, title)
+    def inline_header_title(out, _node, title)
       out.span **{ class: "zzMoveToFollowing" } do |s|
         s.b do |b|
           title&.children&.each { |c2| parse(c2, b) }
-          clausedelimspace(out) if /\S/.match(title&.text)
+          clausedelimspace(out) if /\S/.match?(title&.text)
         end
       end
     end
@@ -20,12 +20,14 @@ module IsoDoc::Function
     # used for subclauses
     def clause_parse_title(node, div, title, out, header_class = {})
       return if title.nil?
+
       if node["inline-header"] == "true"
         inline_header_title(out, node, title)
       else
-        depth = (title && title["depth"]) ? title["depth"] :
-          node.ancestors("clause, annex, terms, references, definitions, "\
-                        "acknowledgements, introduction, foreword").size + 1
+        depth = if title && title["depth"] then title["depth"]
+                else node.ancestors("clause, annex, terms, references, definitions, "\
+                                    "acknowledgements, introduction, foreword").size + 1
+                end
         div.send "h#{depth}", **attr_code(header_class) do |h|
           title&.children&.each { |c2| parse(c2, h) }
         end
@@ -49,8 +51,11 @@ module IsoDoc::Function
     def clause_name(_num, title, div, header_class)
       header_class = {} if header_class.nil?
       div.h1 **attr_code(header_class) do |h1|
-        title.is_a?(String) ? h1 << title :
+        if title.is_a?(String)
+          h1 << title
+        else
           title&.children&.each { |c2| parse(c2, h1) }
+        end
       end
       div.parent.at(".//h1")
     end
@@ -66,8 +71,9 @@ module IsoDoc::Function
       end
     end
 
-    def annex_name(annex, name, div)
+    def annex_name(_annex, name, div)
       return if name.nil?
+
       div.h1 **{ class: "Annex" } do |t|
         name.children.each { |c2| parse(c2, t) }
       end
@@ -144,7 +150,6 @@ module IsoDoc::Function
 
     def introduction(isoxml, out)
       f = isoxml.at(ns("//introduction")) || return
-      title_attr = { class: "IntroTitle" }
       page_break(out)
       out.div **{ class: "Section3", id: f["id"] } do |div|
         clause_name(nil, f.at(ns("./title")), div, { class: "IntroTitle" })
@@ -160,7 +165,6 @@ module IsoDoc::Function
       out.div **attr_code(id: f["id"]) do |s|
         clause_name(nil, f.at(ns("./title")) || @i18n.foreword, s,
                     { class: "ForewordTitle" })
-        #s.h1(**{ class: "ForewordTitle" }) { |h1| h1 << @i18n.foreword }
         f.elements.each { |e| parse(e, s) unless e.name == "title" }
       end
     end
@@ -187,12 +191,11 @@ module IsoDoc::Function
     end
 
     def preface(isoxml, out)
-      title_attr = { class: "IntroTitle" }
       isoxml.xpath(ns("//preface/clause | //preface/references | "\
                       "//preface/definitions | //preface/terms")).each do |f|
         page_break(out)
         out.div **{ class: "Section3", id: f["id"] } do |div|
-          clause_name(nil, f&.at(ns("./title")), div, title_attr)
+          clause_name(nil, f&.at(ns("./title")), div, { class: "IntroTitle" })
           f.elements.each do |e|
             parse(e, div) unless e.name == "title"
           end
@@ -202,37 +205,46 @@ module IsoDoc::Function
 
     def is_clause?(name)
       %w(clause references definitions terms foreword introduction abstract
-      acknowledgements).include? name
+         acknowledgements).include? name
     end
 
     def preface_block(isoxml, out)
       p = isoxml.at(ns("//preface")) or return
       p.elements.each do |e|
         next if is_clause?(e.name)
+
         parse(e, out)
       end
     end
 
     def copyright_parse(node, out)
-      out.div **{class: "boilerplate-copyright"} do |div|
+      return if @bare
+
+      out.div **{ class: "boilerplate-copyright" } do |div|
         node.children.each { |n| parse(n, div) }
       end
     end
 
     def license_parse(node, out)
-      out.div **{class: "boilerplate-license"} do |div|
+      return if @bare
+
+      out.div **{ class: "boilerplate-license" } do |div|
         node.children.each { |n| parse(n, div) }
       end
     end
 
     def legal_parse(node, out)
-      out.div **{class: "boilerplate-legal"} do |div|
+      return if @bare
+
+      out.div **{ class: "boilerplate-legal" } do |div|
         node.children.each { |n| parse(n, div) }
       end
     end
 
     def feedback_parse(node, out)
-      out.div **{class: "boilerplate-feedback"} do |div|
+      return if @bare
+
+      out.div **{ class: "boilerplate-feedback" } do |div|
         node.children.each { |n| parse(n, div) }
       end
     end

data/lib/isodoc/function/table.rb

@@ -1,17 +1,16 @@
 module IsoDoc::Function
   module Table
-
     def table_title_parse(node, out)
       name = node.at(ns("./name")) or return
       out.p **{ class: "TableTitle", style: "text-align:center;" } do |p|
-        name and name.children.each { |n| parse(n, p) }
+        name&.children&.each { |n| parse(n, p) }
       end
     end
 
-    def thead_parse(node, t)
+    def thead_parse(node, table)
       thead = node.at(ns("./thead"))
       if thead
-        t.thead do |h|
+        table.thead do |h|
           thead.element_children.each_with_index do |n, i|
             tr_parse(n, h, i, thead.element_children.size, true)
           end
@@ -19,19 +18,19 @@ module IsoDoc::Function
       end
     end
 
-    def tbody_parse(node, t)
+    def tbody_parse(node, table)
       tbody = node.at(ns("./tbody")) || return
-      t.tbody do |h|
+      table.tbody do |h|
         tbody.element_children.each_with_index do |n, i|
           tr_parse(n, h, i, tbody.element_children.size, false)
         end
       end
     end
 
-    def tfoot_parse(node, t)
+    def tfoot_parse(node, table)
       tfoot = node.at(ns("./tfoot"))
       if tfoot
-        t.tfoot do |h|
+        table.tfoot do |h|
           tfoot.element_children.each_with_index do |n, i|
             tr_parse(n, h, i, tfoot.element_children.size, false)
           end
@@ -45,22 +44,23 @@ module IsoDoc::Function
         id: node["id"],
         class: "MsoISOTable",
         style: "border-width:1px;border-spacing:0;#{width}#{keep_style(node)}",
-        title: node["alt"]
+        title: node["alt"],
       )
     end
 
-    def tcaption(node, t)
+    def tcaption(node, table)
       return unless node["summary"]
-      t.caption do |c|
+
+      table.caption do |c|
         c.span **{ style: "display:none" } do |s|
           s << node["summary"]
         end
       end
     end
 
-    def colgroup(node, t)
+    def colgroup(node, table)
       colgroup = node.at(ns("./colgroup")) or return
-      t.colgroup do |cg|
+      table.colgroup do |cg|
         colgroup.xpath(ns("./col")).each do |c|
           cg.col **{ style: "width: #{c['width']};" }
         end
@@ -89,19 +89,19 @@ module IsoDoc::Function
     # border-left:#{col.zero? ? "#{SW} 1.5pt;" : "none;"}
     # border-right:#{SW} #{col == totalcols && !header ? "1.5" : "1.0"}pt;
 
-    def make_tr_attr(td, row, totalrows, header)
-      style = td.name == "th" ? "font-weight:bold;" : ""
-      td["align"] and style += "text-align:#{td['align']};"
-      td["valign"] and style += "vertical-align:#{td['valign']};"
-      rowmax = td["rowspan"] ? row + td["rowspan"].to_i - 1 : row
+    def make_tr_attr(cell, row, totalrows, header)
+      style = cell.name == "th" ? "font-weight:bold;" : ""
+      cell["align"] and style += "text-align:#{cell['align']};"
+      cell["valign"] and style += "vertical-align:#{cell['valign']};"
+      rowmax = cell["rowspan"] ? row + cell["rowspan"].to_i - 1 : row
       style += <<~STYLE
         border-top:#{row.zero? ? "#{SW} 1.5pt;" : 'none;'}
         border-bottom:#{SW} #{rowmax == totalrows ? '1.5' : '1.0'}pt;
       STYLE
-      header and scope = (td["colspan"] ? "colgroup" : "col")
-      !header and td.name == "th" and scope =
-        (td["rowspan"] ? "rowgroup" : "row")
-      { rowspan: td["rowspan"], colspan: td["colspan"],
+      header and scope = (cell["colspan"] ? "colgroup" : "col")
+      !header and cell.name == "th" and scope =
+                                          (cell["rowspan"] ? "rowgroup" : "row")
+      { rowspan: cell["rowspan"], colspan: cell["colspan"],
         style: style.gsub(/\n/, ""), scope: scope }
     end
 

data/lib/isodoc/function/terms.rb

@@ -11,27 +11,27 @@ module IsoDoc::Function
     end
 
     def deprecated_term_parse(node, out)
-      out.p **{ class: "DeprecatedTerms", style:"text-align:left;" } do |p|
+      out.p **{ class: "DeprecatedTerms", style: "text-align:left;" } do |p|
         p << l10n("#{@i18n.deprecated}: ")
         node.children.each { |c| parse(c, p) }
       end
     end
 
     def admitted_term_parse(node, out)
-      out.p **{ class: "AltTerms", style:"text-align:left;" } do |p|
+      out.p **{ class: "AltTerms", style: "text-align:left;" } do |p|
         node.children.each { |c| parse(c, p) }
       end
     end
 
     def term_parse(node, out)
-      out.p **{ class: "Terms", style:"text-align:left;" } do |p|
+      out.p **{ class: "Terms", style: "text-align:left;" } do |p|
         node.children.each { |c| parse(c, p) }
       end
     end
 
-    def para_then_remainder(first, node, p, div)
+    def para_then_remainder(first, node, para, div)
       if first.name == "p"
-        first.children.each { |n| parse(n, p) }
+        first.children.each { |n| parse(n, para) }
         node.elements.drop(1).each { |n| parse(n, div) }
       else
         node.elements.each { |n| parse(n, div) }
@@ -72,7 +72,6 @@ module IsoDoc::Function
       node.children.each { |n| parse(n, out) }
     end
 
-    def termdocsource_parse(_node, _out)
-    end
+    def termdocsource_parse(_node, _out); end
   end
 end

data/lib/isodoc/function/to_word_html.rb

@@ -18,19 +18,20 @@ module IsoDoc::Function
 
     def init_file(filename, debug)
       filepath = Pathname.new(filename)
-      filename = filepath.sub_ext('').sub(/\.presentation$/, "").to_s
+      filename = filepath.sub_ext("").sub(/\.presentation$/, "").to_s
       dir = init_dir(filename, debug)
       @filename = filename
-      @localdir = filepath.parent.to_s + '/'
+      @localdir = "#{filepath.parent}/"
       @sourcedir = @localdir
-      @sourcefilename and @sourcedir = Pathname.new(@sourcefilename).parent.to_s + '/'
+      @sourcefilename and
+        @sourcedir = "#{Pathname.new(@sourcefilename).parent}/"
       [filename, dir]
     end
 
     def init_dir(filename, debug)
       dir = "#{filename}_files"
       unless debug
-        Dir.mkdir(dir, 0777) unless File.exists?(dir)
+        Dir.mkdir(dir, 0o777) unless File.exists?(dir)
         FileUtils.rm_rf "#{dir}/*"
       end
       dir
@@ -47,13 +48,11 @@ module IsoDoc::Function
 
     # isodoc.css overrides any CSS injected by Html2Doc, which
     # is inserted before this CSS.
-    def define_head(head, filename, _dir)
+    def define_head(head, _filename, _dir)
       if @standardstylesheet
         head.style do |style|
           @standardstylesheet.open
-          stylesheet = @standardstylesheet.read.gsub(
-            "FILENAME", File.basename(filename).sub(/\.presentation$/, "")
-          )
+          stylesheet = @standardstylesheet.read
           style.comment "\n#{stylesheet}\n"
         end
       end
@@ -78,7 +77,7 @@ module IsoDoc::Function
       section_break(body)
     end
 
-    def make_body2(body, docxml)
+    def make_body2(body, _docxml)
       body.div **{ class: "prefatory-section" } do |div2|
         div2.p { |p| p << "&nbsp;" } # placeholder
       end
@@ -117,16 +116,6 @@ module IsoDoc::Function
       @meta.get
     end
 
-    def middle_title(_isoxml, out)
-      out.p(**{ class: "zzSTDTitle1" }) { |p| p << @meta.get[:doctitle] }
-    end
-
-    def middle_admonitions(isoxml, out)
-      isoxml.xpath(ns("//sections/note | //sections/admonition")).each do |x|
-        parse(x, out)
-      end
-    end
-
     def middle(isoxml, out)
       middle_title(isoxml, out)
       middle_admonitions(isoxml, out)
@@ -140,6 +129,8 @@ module IsoDoc::Function
     end
 
     def boilerplate(node, out)
+      return if @bare
+
       boilerplate = node.at(ns("//boilerplate")) or return
       out.div **{ class: "authority" } do |s|
         boilerplate.children.each do |n|
@@ -147,16 +138,14 @@ module IsoDoc::Function
             s.h1 do |h|
               n.children.each { |nn| parse(nn, h) }
             end
-          else
-            parse(n, s)
+          else parse(n, s)
           end
         end
       end
     end
 
     def parse(node, out)
-      if node.text?
-        text_parse(node, out)
+      if node.text? then text_parse(node, out)
       else
         case node.name
         when "em" then em_parse(node, out)
@@ -236,8 +225,13 @@ module IsoDoc::Function
         when "svg" then svg_parse(node, out) # in Presentation XML only
         when "add" then add_parse(node, out)
         when "del" then del_parse(node, out)
-        else
-          error_parse(node, out)
+        when "form" then form_parse(node, out)
+        when "input" then input_parse(node, out)
+        when "select" then select_parse(node, out)
+        when "label" then label_parse(node, out)
+        when "option" then option_parse(node, out)
+        when "textarea" then textarea_parse(node, out)
+        else error_parse(node, out)
         end
       end
     end

data/lib/isodoc/function/utils.rb

@@ -1,191 +1,211 @@
 # frozen_string_literal: true
 
-module IsoDoc::Function
-  module Utils
-    def date_range(date)
-      self.class.date_range(date)
-    end
+module IsoDoc
+  module Function
+    module Utils
+      def date_range(date)
+        self.class.date_range(date)
+      end
 
-    def ns(xpath)
-      self.class.ns(xpath)
-    end
+      def ns(xpath)
+        self.class.ns(xpath)
+      end
 
-    def insert_tab(out, n)
-      tab = %w(Hans Hant).include?(@script) ? "　" : "  "
-      [1..n].each { out << tab }
-    end
+      def insert_tab(out, count)
+        tab = %w(Hans Hant).include?(@script) ? "&#x3000;" : "&nbsp; "
+        [1..count].each { out << tab }
+      end
 
-    # add namespaces for Word fragments
-    NOKOHEAD = <<~HERE
-      <!DOCTYPE html SYSTEM
-      "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">
-      <html xmlns="http://www.w3.org/1999/xhtml">
-      <head> <title></title> <meta charset="UTF-8" /> </head>
-      <body> </body> </html>
-    HERE
-
-    # block for processing XML document fragments as XHTML,
-    # to allow for HTMLentities
-    def noko(&block)
-      doc = ::Nokogiri::XML.parse(NOKOHEAD)
-      fragment = doc.fragment('')
-      ::Nokogiri::XML::Builder.with fragment, &block
-      fragment.to_xml(encoding: 'US-ASCII').lines.map do |l|
-        l.gsub(/\s*\n/, '')
+      # add namespaces for Word fragments
+      NOKOHEAD = <<~HERE
+        <!DOCTYPE html SYSTEM
+        "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">
+        <html xmlns="http://www.w3.org/1999/xhtml">
+        <head> <title></title> <meta charset="UTF-8" /> </head>
+        <body> </body> </html>
+      HERE
+
+      # block for processing XML document fragments as XHTML,
+      # to allow for HTMLentities
+      def noko(&block)
+        doc = ::Nokogiri::XML.parse(NOKOHEAD)
+        fragment = doc.fragment("")
+        ::Nokogiri::XML::Builder.with fragment, &block
+        fragment.to_xml(encoding: "US-ASCII").lines.map do |l|
+          l.gsub(/\s*\n/, "")
+        end
       end
-    end
 
-    def attr_code(attributes)
-      attributes = attributes.reject { |_, val| val.nil? }.map
-      attributes.map do |k, v|
-        [k, v.is_a?(String) ? HTMLEntities.new.decode(v) : v]
-      end.to_h
-    end
+      def attr_code(attributes)
+        attributes = attributes.reject { |_, val| val.nil? }.map
+        attributes.map do |k, v|
+          [k, v.is_a?(String) ? HTMLEntities.new.decode(v) : v]
+        end.to_h
+      end
 
-    DOCTYPE_HDR = '<!DOCTYPE html SYSTEM '\
-      '"http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">'
-
-    def to_xhtml(xml)
-      xml.gsub!(/<\?xml[^>]*>/, '')
-      /<!DOCTYPE /.match(xml) || (xml = DOCTYPE_HDR + xml)
-      xml = xml.split(/(\&[^ \r\n\t#;]+;)/).map do |t|
-        /^(\&[^ \t\r\n#;]+;)/.match?(t) ?
-          HTMLEntities.new.encode(HTMLEntities.new.decode(t), :hexadecimal) : t
-      end.join('')
-      begin
-        Nokogiri::XML.parse(xml, &:strict)
-      rescue Nokogiri::XML::SyntaxError => e
-        File.open("#{@filename}.#{@format}.err", 'w:UTF-8') { |f| f.write xml }
-        abort "Malformed Output XML for #{@format}: #{e} (see #{@filename}.#{@format}.err)"
+      DOCTYPE_HDR = "<!DOCTYPE html SYSTEM "\
+        '"http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">'
+
+      def to_xhtml(xml)
+        xml = to_xhtml_prep(xml)
+        begin
+          Nokogiri::XML.parse(xml, &:strict)
+        rescue Nokogiri::XML::SyntaxError => e
+          File.open("#{@filename}.#{@format}.err", "w:UTF-8") do |f|
+            f.write xml
+          end
+          abort "Malformed Output XML for #{@format}: #{e} "\
+            "(see #{@filename}.#{@format}.err)"
+        end
       end
-    end
 
-    def to_xhtml_fragment(xml)
-      doc = ::Nokogiri::XML.parse(NOKOHEAD)
-      fragment = doc.fragment(xml)
-      fragment
-    end
+      def to_xhtml_prep(xml)
+        xml.gsub!(/<\?xml[^>]*>/, "")
+        /<!DOCTYPE /.match(xml) || (xml = DOCTYPE_HDR + xml)
+        xml.split(/(&[^ \r\n\t#;]+;)/).map do |t|
+          if /^(&[^ \t\r\n#;]+;)/.match?(t)
+            HTMLEntities.new.encode(HTMLEntities.new.decode(t), :hexadecimal)
+          else t
+          end
+        end.join("")
+      end
 
-    def from_xhtml(xml)
-      xml.to_xml.sub(%r{ xmlns="http://www.w3.org/1999/xhtml"}, '')
-    end
+      def to_xhtml_fragment(xml)
+        doc = ::Nokogiri::XML.parse(NOKOHEAD)
+        doc.fragment(xml)
+      end
 
-    CLAUSE_ANCESTOR =
-      ".//ancestor::*[local-name() = 'annex' or "\
-      "local-name() = 'acknowledgements' or local-name() = 'term' or "\
-      "local-name() = 'appendix' or local-name() = 'foreword' or "\
-      "local-name() = 'introduction' or local-name() = 'terms' or "\
-      "local-name() = 'clause' or local-name() = 'references']/@id"
+      def from_xhtml(xml)
+        xml.to_xml.sub(%r{ xmlns="http://www.w3.org/1999/xhtml"}, "")
+      end
 
-    def get_clause_id(node)
-      clause = node.xpath(CLAUSE_ANCESTOR)
-      clause&.last&.text || nil
-    end
+      CLAUSE_ANCESTOR =
+        ".//ancestor::*[local-name() = 'annex' or "\
+        "local-name() = 'definitions' or "\
+        "local-name() = 'acknowledgements' or local-name() = 'term' or "\
+        "local-name() = 'appendix' or local-name() = 'foreword' or "\
+        "local-name() = 'introduction' or local-name() = 'terms' or "\
+        "local-name() = 'clause' or local-name() = 'references']/@id"
+
+      def get_clause_id(node)
+        clause = node.xpath(CLAUSE_ANCESTOR)
+        clause&.last&.text || nil
+      end
 
-    NOTE_CONTAINER_ANCESTOR =
-      ".//ancestor::*[local-name() = 'annex' or "\
-      "local-name() = 'foreword' or local-name() = 'appendix' or "\
-      "local-name() = 'introduction' or local-name() = 'terms' or "\
-      "local-name() = 'acknowledgements' or local-name() = 'term' or "\
-      "local-name() = 'clause' or local-name() = 'references' or "\
-      "local-name() = 'figure' or local-name() = 'formula' or "\
-      "local-name() = 'table' or local-name() = 'example']/@id"
-
-    def get_note_container_id(node)
-      container = node.xpath(NOTE_CONTAINER_ANCESTOR)
-      container&.last&.text || nil
-    end
+      NOTE_CONTAINER_ANCESTOR =
+        ".//ancestor::*[local-name() = 'annex' or "\
+        "local-name() = 'foreword' or local-name() = 'appendix' or "\
+        "local-name() = 'introduction' or local-name() = 'terms' or "\
+        "local-name() = 'acknowledgements' or local-name() = 'term' or "\
+        "local-name() = 'clause' or local-name() = 'references' or "\
+        "local-name() = 'figure' or local-name() = 'formula' or "\
+        "local-name() = 'table' or local-name() = 'example']/@id"
+
+      def get_note_container_id(node)
+        container = node.xpath(NOTE_CONTAINER_ANCESTOR)
+        container&.last&.text || nil
+      end
 
-    def sentence_join(array)
-      return '' if array.nil? || array.empty?
-      if array.length == 1 then array[0]
-      else
-        @i18n.l10n("#{array[0..-2].join(', ')} "\
-                   "#{@i18n.and} #{array.last}",
-                                     @lang, @script)
+      def sentence_join(array)
+        return "" if array.nil? || array.empty?
+
+        if array.length == 1 then array[0]
+        else
+          @i18n.l10n("#{array[0..-2].join(', ')} "\
+                     "#{@i18n.and} #{array.last}",
+                     @lang, @script)
+        end
       end
-    end
 
-    # avoid `; avoid {{ (Liquid Templates); avoid [[ (Javascript)
-    def extract_delims(text)
-      @openmathdelim = '(#('
-      @closemathdelim = ')#)'
-      while text.include?(@openmathdelim) || text.include?(@closemathdelim)
-        @openmathdelim += '('
-        @closemathdelim += ')'
+      # avoid `; avoid {{ (Liquid Templates); avoid [[ (Javascript)
+      def extract_delims(text)
+        @openmathdelim = "(#("
+        @closemathdelim = ")#)"
+        while text.include?(@openmathdelim) || text.include?(@closemathdelim)
+          @openmathdelim += "("
+          @closemathdelim += ")"
+        end
+        [@openmathdelim, @closemathdelim]
       end
-      [@openmathdelim, @closemathdelim]
-    end
 
-    def header_strip(h)
-      h = h.to_s.gsub(%r{<br\s*/>}, ' ').gsub(/<\/?h[123456][^>]*>/, '')
-           .gsub(/<\/?b[^>]*>/, '')
-      h1 = to_xhtml_fragment(h.dup)
-      h1.traverse do |x|
-        x.replace(' ') if x.name == 'span' && /mso-tab-count/.match(x['style'])
-        x.remove if x.name == 'img'
-        x.remove if x.name == 'span' && x['class'] == 'MsoCommentReference'
-        x.remove if x.name == 'a' && x['class'] == 'FootnoteRef'
-        x.remove if x.name == 'span' && /mso-bookmark/.match(x['style'])
-        x.replace(x.children) if x.name == 'a'
-      end
-      from_xhtml(h1)
-    end
+      def header_strip(hdr)
+        h1 = to_xhtml_fragment(hdr.to_s.gsub(%r{<br\s*/>}, " ")
+          .gsub(/<\/?h[123456][^>]*>/, "").gsub(/<\/?b[^>]*>/, "").dup)
+        h1.traverse do |x|
+          if x.name == "span" && /mso-tab-count/.match(x["style"])
+            x.replace(" ")
+          elsif header_strip_elem?(x) then x.remove
+          elsif x.name == "a" then x.replace(x.children)
+          end
+        end
+        from_xhtml(h1)
+      end
 
-    def liquid(doc)
-      self.class.liquid(doc)
-    end
+      def header_strip_elem?(elem)
+        elem.name == "img" ||
+          elem.name == "span" && elem["class"] == "MsoCommentReference" ||
+          elem.name == "a" && elem["class"] == "FootnoteRef" ||
+          elem.name == "span" && /mso-bookmark/.match(elem["style"])
+      end
 
-    def liquid(doc)
-      # unescape HTML escapes in doc
-      doc = doc.split(%r<(\{%|%\})>).each_slice(4).map do |a|
-        a[2] = a[2].gsub(/\&lt;/, '<').gsub(/\&gt;/, '>') if a.size > 2
-        a.join('')
-      end.join('')
-      Liquid::Template.parse(doc)
-    end
+=begin
+      def liquid(doc)
+        self.class.liquid(doc)
+      end
+=end
+
+      def liquid(doc)
+        # unescape HTML escapes in doc
+        doc = doc.split(%r<(\{%|%\})>).each_slice(4).map do |a|
+          a[2] = a[2].gsub(/&lt;/, "<").gsub(/&gt;/, ">") if a.size > 2
+          a.join("")
+        end.join("")
+        Liquid::Template.parse(doc)
+      end
 
-    def empty2nil(v)
-      return nil if !v.nil? && v.is_a?(String) && v.empty?
-      v
-    end
+      def empty2nil(str)
+        return nil if !str.nil? && str.is_a?(String) && str.empty?
 
-    def populate_template(docxml, _format = nil)
-      meta = @meta
-             .get
-             .merge(@labels ? {labels: @labels} : {})
-             .merge(@meta.labels ? {labels: @meta.labels} : {})
-             .merge(fonts_options || {})
-      template = liquid(docxml)
-      template.render(meta.map { |k, v| [k.to_s, empty2nil(v)] }.to_h)
-              .gsub('&lt;', '&#x3c;').gsub('&gt;', '&#x3e;').gsub('&amp;', '&#x26;')
-    end
+        str
+      end
 
-    def save_dataimage(uri, _relative_dir = true)
-      %r{^data:(image|application)/(?<imgtype>[^;]+);base64,(?<imgdata>.+)$} =~ uri
-      imgtype.sub!(/\+[a-z0-9]+$/, '') # svg+xml
-      imgtype = 'png' unless /^[a-z0-9]+$/.match imgtype
-      Tempfile.open(['image', ".#{imgtype}"]) do |f|
-        f.binmode
-        f.write(Base64.strict_decode64(imgdata))
-        @tempfile_cache << f # persist to the end
-        f.path
+      def populate_template(docxml, _format = nil)
+        meta = @meta
+          .get
+          .merge(@labels ? { labels: @labels } : {})
+          .merge(@meta.labels ? { labels: @meta.labels } : {})
+          .merge(fonts_options || {})
+        template = liquid(docxml)
+        template.render(meta.map { |k, v| [k.to_s, empty2nil(v)] }.to_h)
+          .gsub("&lt;", "&#x3c;").gsub("&gt;", "&#x3e;").gsub("&amp;", "&#x26;")
       end
-    end
 
-    def image_localfile(i)
-      if /^data:/.match? i['src']
-        save_dataimage(i['src'], false)
-      elsif %r{^([A-Z]:)?/}.match? i['src']
-        i['src']
-      else
-        File.join(@localdir, i['src'])
+      def save_dataimage(uri, _relative_dir = true)
+        %r{^data:(image|application)/(?<imgtype>[^;]+);base64,(?<imgdata>.+)$} =~ uri
+        imgtype.sub!(/\+[a-z0-9]+$/, "") # svg+xml
+        imgtype = "png" unless /^[a-z0-9]+$/.match? imgtype
+        Tempfile.open(["image", ".#{imgtype}"]) do |f|
+          f.binmode
+          f.write(Base64.strict_decode64(imgdata))
+          @tempfile_cache << f # persist to the end
+          f.path
+        end
+      end
+
+      def image_localfile(img)
+        if /^data:/.match? img["src"]
+          save_dataimage(img["src"], false)
+        elsif %r{^([A-Z]:)?/}.match? img["src"]
+          img["src"]
+        else
+          File.join(@localdir, img["src"])
+        end
       end
-    end
 
-    def labelled_ancestor(node)
-      !node.ancestors('example, requirement, recommendation, permission, '\
-                             'note, table, figure, sourcecode').empty?
+      def labelled_ancestor(node)
+        !node.ancestors("example, requirement, recommendation, permission, "\
+                        "note, table, figure, sourcecode").empty?
+      end
     end
   end
 end