isodoc 1.6.6 → 1.7.1

data/lib/isodoc/html_function/sectionsplit.rb

@@ -99,7 +99,9 @@ module IsoDoc::HtmlFunction
       refs = eref_to_internal_eref(xml, key)
       refs += xref_to_internal_eref(xml, key)
       xml.root["type"] = key # to force recognition of internal refs
-      insert_indirect_biblio(xml, refs, key)
+      ins = new_hidden_ref(xml)
+      copy_repo_items_biblio(ins, xml)
+      insert_indirect_biblio(ins, refs, key)
     end
 
     def svg_preprocess(xml)
@@ -164,15 +166,27 @@ module IsoDoc::HtmlFunction
     def eref_to_internal_eref_select(xml)
       refs = xml.xpath(("//*/@bibitemid")).map { |x| x.text } # rubocop:disable Style/SymbolProc
       refs.uniq.reject do |x|
-        xml.at(ns("//bibitem[@id = '#{x}'][@type = 'internal']"))
+        xml.at(ns("//bibitem[@id = '#{x}'][@type = 'internal']")) ||
+          xml.at(ns("//bibitem[@id = '#{x}']"\
+                    "[docidentifier/@type = 'repository']"))
       end
     end
 
     # from standoc
-    def insert_indirect_biblio(xmldoc, refs, prefix)
+    def new_hidden_ref(xmldoc)
       ins = xmldoc.at("bibliography") or
         xmldoc.root << "<bibliography/>" and ins = xmldoc.at("bibliography")
-      ins = ins.add_child("<references hidden='true' normative='false'/>").first
+      ins.add_child("<references hidden='true' normative='false'/>").first
+    end
+
+    def copy_repo_items_biblio(ins, xml)
+      xml.xpath(ns("//references/bibitem[docidentifier/@type = 'repository']"))
+        .each do |b|
+        ins << b.dup
+      end
+    end
+
+    def insert_indirect_biblio(ins, refs, prefix)
       refs.each do |x|
         ins << <<~BIBENTRY
           <bibitem id="#{x}" type="internal">

data/lib/isodoc/presentation_function/bibdata.rb

@@ -1,6 +1,7 @@
 module IsoDoc
   class PresentationXMLConvert < ::IsoDoc::Convert
     def bibdata(docxml)
+      docid_prefixes(docxml)
       a = bibdata_current(docxml) or return
       address_precompose(a)
       bibdata_i18n(a)
@@ -9,6 +10,12 @@ module IsoDoc
         "</localized-strings>"
     end
 
+     def docid_prefixes(docxml)
+      docxml.xpath(ns("//references/bibitem/docidentifier")).each do |i|
+        i.children = @xrefs.klass.docid_prefix(i["type"], i.text)
+      end
+    end
+
     def address_precompose(bib)
       bib.xpath(ns("//bibdata//address")).each do |b|
         next if b.at(ns("./formattedAddress"))

data/lib/isodoc/presentation_function/block.rb

@@ -19,12 +19,15 @@ module IsoDoc
       end
     end
 
-    def svg_extract(f)
-      return unless %r{^data:image/svg\+xml;base64,}.match?(f["src"])
+    def svg_extract(elem)
+      return unless %r{^data:image/svg\+xml;base64,}.match?(elem["src"])
 
-      svg = Base64.strict_decode64(f["src"]
+      svg = Base64.strict_decode64(elem["src"]
         .sub(%r{^data:image/svg\+xml;base64,}, ""))
-      f.replace(svg.sub(/<\?xml[^>]*>/, ""))
+      x = Nokogiri::XML.fragment(svg.sub(/<\?xml[^>]*>/, "")) do |config|
+        config.huge
+      end
+      elem.replace(x)
     end
 
     def figure1(f)

data/lib/isodoc/presentation_function/inline.rb

@@ -129,10 +129,8 @@ module IsoDoc
 
       loc = @i18n.locality[type] || type.sub(/^locality:/, "")
       loc = case node["case"]
-            when "capital" then loc.capitalize
             when "lowercase" then loc.downcase
-            else
-              loc.capitalize
+            else loc.capitalize
             end
       " #{loc}"
     end
@@ -162,15 +160,34 @@ module IsoDoc
     end
 
     def concept1(node)
-      content = node.first_element_child.children.reject do |c|
-        %w{locality localityStack}.include? c.name
-      end.select { |c| !c.text? || /\S/.match(c) }
-      n = if content.empty?
-            @i18n.term_defined_in.sub(/%/, node.first_element_child.to_xml)
-          else
-            "<em>#{node.children.to_xml}</em>"
-          end
-      node.replace(n)
+      xref = node&.at(ns("./xref/@target"))&.text or
+        return concept_render(node, node["ital"] || "true",
+                              node["ref"] || "true")
+      if node.at(ns("//definitions//dt[@id = '#{xref}']"))
+        concept_render(node, node["ital"] || "false", node["ref"] || "false")
+      else concept_render(node, node["ital"] || "true", node["ref"] || "true")
+      end
+    end
+
+    def concept_render(node, ital, ref)
+      node&.at(ns("./refterm"))&.remove
+      r = node.at(ns("./renderterm"))
+      r&.next = " "
+      if ital == "true" then r&.name = "em"
+      else r&.replace(r&.children)
+      end
+      concept1_ref(node, ref)
+      node.replace(node.children)
+    end
+
+    def concept1_ref(node, ref)
+      r = node.at(ns("./xref | ./eref | ./termref")) or return
+      return r.remove if ref == "false"
+
+      if non_locality_elems(r).select { |c| !c.text? || /\S/.match(c) }.empty?
+        r.replace(@i18n.term_defined_in.sub(/%/, r.to_xml))
+      else r.replace("[#{r.to_xml}]")
+      end
     end
 
     def variant(docxml)

data/lib/isodoc/presentation_function/section.rb

@@ -44,9 +44,7 @@ module IsoDoc
       prefix_name(elem, "", "#{lbl}#{clausedelim}", "name")
     end
 
-    def references(docxml)
-      docid_prefixes(docxml)
-    end
+    def references(docxml); end
 
     def docid_prefixes(docxml)
       docxml.xpath(ns("//references/bibitem/docidentifier")).each do |i|

data/lib/isodoc/presentation_xml_convert.rb

@@ -22,19 +22,19 @@ module IsoDoc
 
     def conversions(docxml)
       bibdata docxml
+      @xrefs.parse docxml
       section docxml
       block docxml
-      @xrefs.parse docxml
       inline docxml
     end
 
     def section(docxml)
-      display_order docxml
       clause docxml
       annex docxml
       term docxml
       references docxml
       index docxml
+      display_order docxml
     end
 
     def block(docxml)
@@ -53,10 +53,10 @@ module IsoDoc
     end
 
     def inline(docxml)
+      concept docxml
       xref docxml
       eref docxml
       origin docxml
-      concept docxml
       quotesource docxml
       mathml docxml
       variant docxml

data/lib/isodoc/version.rb

@@ -1,3 +1,3 @@
 module IsoDoc
-  VERSION = "1.6.6".freeze
+  VERSION = "1.7.1".freeze
 end

data/lib/isodoc/xref.rb

@@ -28,16 +28,16 @@ module IsoDoc
       @anchors
     end
 
-    def anchor(id, lbl, warning = true)
-      return nil if id.nil? || id.empty?
+    def anchor(ident, lbl, warning = true)
+      return nil if ident.nil? || ident.empty?
 
-      if warning && !@anchors[id]
+      if warning && !@anchors[ident]
         @seen ||= Seen_Anchor.instance
-        @seen.seen(id) or warn "No label has been processed for ID #{id}"
-        @seen.add(id)
-        return "[#{id}]"
+        @seen.seen(ident) or warn "No label has been processed for ID #{ident}"
+        @seen.add(ident)
+        return "[#{ident}]"
       end
-      @anchors.dig(id, lbl)
+      @anchors.dig(ident, lbl)
     end
 
     # extract names for all anchors, xref and label
@@ -50,6 +50,7 @@ module IsoDoc
       note_anchor_names(docxml.xpath(ns(SECTIONS_XPATH)))
       example_anchor_names(docxml.xpath(ns(SECTIONS_XPATH)))
       list_anchor_names(docxml.xpath(ns(SECTIONS_XPATH)))
+      deflist_anchor_names(docxml.xpath(ns(SECTIONS_XPATH)))
       bookmark_anchor_names(docxml)
     end
 

data/lib/isodoc/xref/xref_anchor.rb

@@ -1,61 +1,62 @@
 require "singleton"
 
-module IsoDoc::XrefGen
-  module Anchor
-    class Seen_Anchor
-      include Singleton
+module IsoDoc
+  module XrefGen
+    module Anchor
+      class Seen_Anchor
+        include Singleton
+
+        def initialize
+          @seen = {}
+        end
+
+        def seen(elem)
+          @seen.has_key?(elem)
+        end
+
+        def add(elem)
+          @seen[elem] = true
+        end
+      end
 
       def initialize
-        @seen = {}
+        @anchors = {}
       end
 
-      def seen(x)
-        @seen.has_key?(x)
+      def get_anchors
+        @anchors
       end
 
-      def add(x)
-        @seen[x] = true
+      def anchor_struct_label(lbl, elem)
+        case elem
+        when @labels["appendix"] then l10n("#{elem} #{lbl}")
+        else
+          lbl.to_s
+        end
       end
-    end
-
-    def initialize()
-      @anchors = {}
-    end
-
-    def get_anchors
-      @anchors
-    end
 
-    def anchor_struct_label(lbl, elem)
-      case elem
-      when @labels["appendix"] then l10n("#{elem} #{lbl}")
-      else
-        lbl.to_s
+      def anchor_struct_xref(lbl, elem)
+        l10n("#{elem} #{anchor_struct_value(lbl, elem)}")
       end
-    end
-
-    def anchor_struct_xref(lbl, elem)
-      l10n("#{elem} #{anchor_struct_value(lbl, elem)}")
-    end
 
-    def anchor_struct_value(lbl, elem)
-      case elem
-      when @labels["formula"] then "(#{lbl})"
-      when @labels["inequality"] then "(#{lbl})"
-      else
-        lbl
+      def anchor_struct_value(lbl, elem)
+        case elem
+        when @labels["formula"], @labels["inequality"] then "(#{lbl})"
+        else
+          lbl
+        end
       end
-    end
 
-    def anchor_struct(lbl, container, elem, type, unnumbered = false)
-      ret = {}
-      ret[:label] = unnumbered == "true" ? nil : anchor_struct_label(lbl, elem)
-      ret[:xref] = anchor_struct_xref(unnumbered == "true" ? "(??)" : lbl, elem)
-      ret[:xref].gsub!(/ $/, "")
-      ret[:container] = @klass.get_clause_id(container) unless container.nil?
-      ret[:type] = type
-      ret[:value] = anchor_struct_value(lbl, elem)
-      ret
+      def anchor_struct(lbl, container, elem, type, unnumb = false)
+        ret = {}
+        ret[:label] = unnumb == "true" ? nil : anchor_struct_label(lbl, elem)
+        ret[:xref] = anchor_struct_xref(unnumb == "true" ? "(??)" : lbl, elem)
+        ret[:xref].gsub!(/ $/, "")
+        ret[:container] = @klass.get_clause_id(container) unless container.nil?
+        ret[:type] = type
+        ret[:value] = anchor_struct_value(lbl, elem)
+        ret
+      end
     end
   end
 end

data/lib/isodoc/xref/xref_counter.rb

@@ -1,33 +1,37 @@
 require "roman-numerals"
 
-module IsoDoc::XrefGen
-  class Counter
-    def initialize(num = 0, opts = { numerals: :arabic })
-      @num = num
-      @letter = ""
-      @subseq = ""
-      @letter_override = nil
-      @number_override = nil
-      @style = opts[:numerals]
-      @skip_i = opts[:skip_i]
-      @base = ""
-      if num.is_a? String
-        if /^\d+$/.match?(num)
-          @num = num.to_i
-        else
-          @num = nil
-          @base = num[0..-2]
-          @letter = num[-1]
+module IsoDoc
+  module XrefGen
+    class Counter
+      def initialize(num = 0, opts = { numerals: :arabic })
+        @num = num
+        @letter = ""
+        @subseq = ""
+        @letter_override = nil
+        @number_override = nil
+        @style = opts[:numerals]
+        @skip_i = opts[:skip_i]
+        @base = ""
+        if num.is_a? String
+          if /^\d+$/.match?(num)
+            @num = num.to_i
+          else
+            @num = nil
+            @base = num[0..-2]
+            @letter = num[-1]
+          end
         end
       end
-    end
 
-    def new_subseq_increment(node)
-      @subseq = node["subsequence"]
-      @num += 1 unless @num.nil?
-      @letter = node["subsequence"] ? "a" : ""
-      @base = ""
-      if node["number"]
+      def new_subseq_increment(node)
+        @subseq = node["subsequence"]
+        @num += 1 unless @num.nil?
+        @letter = node["subsequence"] ? "a" : ""
+        @base = ""
+        new_subseq_increment1(node) if node["number"]
+      end
+
+      def new_subseq_increment1(node)
         /^(?<b>.*?)(?<n>\d*)(?<a>[a-zA-Z]*)$/ =~ node["number"]
         if !n.empty? || !a.empty?
           @letter_override = @letter = a unless a.empty?
@@ -38,108 +42,114 @@ module IsoDoc::XrefGen
           @letter = @letter_override if /^[a-zA-Z]$/.match?(@letter_override)
         end
       end
-    end
 
-    def sequence_increment(node)
-      if node["number"]
-        @base = @letter_override = @number_override = ""
-        /^(?<b>.*?)(?<n>\d+)$/ =~ node["number"]
-        if blank?(n)
-          @num = nil
-          @base = node["number"][0..-2]
-          @letter = @letter_override = node["number"][-1]
-        else
-          @number_override = @num = n.to_i
-          @base = b
-          @letter = ""
+      def sequence_increment(node)
+        if node["number"]
+          @base = @letter_override = @number_override = ""
+          /^(?<b>.*?)(?<n>\d+)$/ =~ node["number"]
+          if blank?(n)
+            @num = nil
+            @base = node["number"][0..-2]
+            @letter = @letter_override = node["number"][-1]
+          else
+            @number_override = @num = n.to_i
+            @base = b
+            @letter = ""
+          end
+        else @num += 1
         end
-      else
-        @num += 1
       end
-    end
 
-    def subsequence_increment(node)
-      if node["number"]
+      def subsequence_increment(node)
+        return increment_letter unless node["number"]
+
         @base = ""
         @letter_override = node["number"]
         /^(?<b>.*?)(?<n>\d*)(?<a>[a-zA-Z])$/ =~ node["number"]
-        if blank?(a)
-          if /^\d+$/.match?(node["number"])
-            @letter_override = @letter = ""
-            @number_override = @num = node["number"].to_i
-          else
-            /^(?<b>.*)(?<a>[a-zA-Z])$/ =~ node["number"]
-            unless blank?(a)
-              @letter = @letter_override = a
-              @base = b
-            end
-          end
+        if blank?(a) then subsequence_increment_no_letter(node)
         else
           @letter_override = @letter = a
           @base = b
           @number_override = @num = n.to_i unless n.empty?
         end
-      else
-        increment_letter
       end
-    end
 
-    def increment_letter
-      case @letter
-      when "Z"
-        @letter = "@"
-        @base = @base.empty? ? "A" : @base[0..-2] + (@base[-1].ord + 1).chr.to_s
-      when "z"
-        @letter = "`"
-        @base = @base.empty? ? "a" : @base[0..-2] + (@base[-1].ord + 1).chr.to_s
+      def subsequence_increment_no_letter(node)
+        if /^\d+$/.match?(node["number"])
+          @letter_override = @letter = ""
+          @number_override = @num = node["number"].to_i
+        else
+          /^(?<b>.*)(?<a>[a-zA-Z])$/ =~ node["number"]
+          unless blank?(a)
+            @letter = @letter_override = a
+            @base = b
+          end
+        end
+      end
+
+      def string_inc(str, start)
+        return start if str.empty?
+
+        str[0..-2] + (str[-1].ord + 1).chr.to_s
       end
-      @letter = (@letter.ord + 1).chr.to_s
-      @skip_i && %w(i I).include?(@letter) and
+
+      def increment_letter
+        case @letter
+        when "Z"
+          @letter = "@"
+          @base = string_inc(@base, "A")
+        when "z"
+          @letter = "`"
+          @base = string_inc(@base, "a")
+        end
         @letter = (@letter.ord + 1).chr.to_s
-    end
+        @skip_i && %w(i I).include?(@letter) and
+          @letter = (@letter.ord + 1).chr.to_s
+      end
 
-    def blank?(str)
-      str.nil? || str.empty?
-    end
+      def blank?(str)
+        str.nil? || str.empty?
+      end
 
-    def increment(node)
-      return self if node["unnumbered"]
+      def increment(node)
+        return self if node["unnumbered"]
 
-      @letter_override = nil
-      @number_override = nil
-      if node["subsequence"] != @subseq &&
-          !(blank?(node["subsequence"]) && blank?(@subseq))
-        new_subseq_increment(node)
-      elsif @letter.empty? then sequence_increment(node)
-      else subsequence_increment(node)
+        @letter_override = nil
+        @number_override = nil
+        if node["subsequence"] != @subseq &&
+            !(blank?(node["subsequence"]) && blank?(@subseq))
+          new_subseq_increment(node)
+        elsif @letter.empty? then sequence_increment(node)
+        else subsequence_increment(node)
+        end
+        self
       end
-      self
-    end
 
-    def print
-      num = @number_override || @num
-      out = @style == :roman && !num.nil? ? RomanNumerals.to_roman(num) : num
-      "#{@base}#{out}#{@letter_override || @letter}"
-    end
+      def print
+        num = @number_override || @num
+        out = @style == :roman && !num.nil? ? RomanNumerals.to_roman(num) : num
+        "#{@base}#{out}#{@letter_override || @letter}"
+      end
 
-    def ol_type(list, depth)
-      return list["type"].to_sym if list["type"]
-      return :arabic if [2, 7].include? depth
-      return :alphabet if [1, 6].include? depth
-      return :alphabet_upper if [4, 9].include? depth
-      return :roman if [3, 8].include? depth
-      return :roman_upper if [5, 10].include? depth
+      def ol_type(list, depth)
+        return list["type"].to_sym if list["type"]
+        return :arabic if [2, 7].include? depth
+        return :alphabet if [1, 6].include? depth
+        return :alphabet_upper if [4, 9].include? depth
+        return :roman if [3, 8].include? depth
+        return :roman_upper if [5, 10].include? depth
 
-      :arabic
-    end
+        :arabic
+      end
 
-    def listlabel(list, depth)
-      case ol_type(list, depth)
-      when :arabic then @num.to_s
-      when :alphabet then (96 + @num).chr.to_s
-      when :alphabet_upper then (64 + @num).chr.to_s
-      when :roman then RomanNumerals.to_roman(@num).downcase
-      when :roman_upper then RomanNumerals.to_roman(@num).upcase
+      def listlabel(list, depth)
+        case ol_type(list, depth)
+        when :arabic then @num.to_s
+        when :alphabet then (96 + @num).chr.to_s
+        when :alphabet_upper then (64 + @num).chr.to_s
+        when :roman then RomanNumerals.to_roman(@num).downcase
+        when :roman_upper then RomanNumerals.to_roman(@num).upcase
+        end
       end
     end
   end