isodoc 3.1.1 → 3.1.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 60e7a916df30b4032fbe9f1b2dcc6cec2563759b21525f9a7aa932f1f0dd2a24
-  data.tar.gz: 4f4add705922211d501cafbd282df7402cb5228baae6300f62dd1868a42aac2f
+  metadata.gz: aa02186be0f73d998dca870f3559df00184697cc3582ef302eb934e4901d1ab1
+  data.tar.gz: 54cf027ca0fcab8ec1c97b35d7ada8ec7137a32a2adeecfa2c0cb9b653b3b0f2
 SHA512:
-  metadata.gz: 7f4074992a9e19be5425fbf4dcc169ea31f57754261da482b33b5dde2ee14f210f0b4ca3419676268d3fccc9d0aa978e8fe06806d2b3754da6d13116e7332b1b
-  data.tar.gz: 2b6e8718338825d52344fbda655c4fee4d3593dc46a65879a7b178a111923aeefb78c96f201de2a9b8d982a50f0fe4e83453902312aa7d8150d968d0b0bea6ee
+  metadata.gz: 251471002a6b16dd050364344f6a63e99b0e3b266a0dcc65cb4f83382b5ebfece49cc75ea65229282ef61a7206fb1ac4d7aba441b86b19704c6d9d00398ce3f2
+  data.tar.gz: 5fde838af0af79d78ccc4fecd730bc64cc60e6afaf68691f5e046d18ada04df54a922b61d98b22632a9395e21caa4c0cbbafceea70c729da839f9b732beb54c9

data/isodoc.gemspec

@@ -44,6 +44,7 @@ Gem::Specification.new do |spec|
   spec.add_dependency "thread_safe"
   spec.add_dependency "twitter_cldr", ">= 6.6.0"
   spec.add_dependency "uuidtools"
+  spec.add_dependency "lutaml-model", "~> 0.6.0"
 
   spec.add_development_dependency "bigdecimal"
   spec.add_development_dependency "debug"
@@ -53,18 +54,7 @@ Gem::Specification.new do |spec|
   spec.add_development_dependency "rake", "~> 13.0"
   spec.add_development_dependency "rspec", "~> 3.6"
   spec.add_development_dependency "rubocop", "~> 1"
-spec.add_development_dependency "rubocop-performance"
-=begin
-  if RUBY_VERSION.start_with?("3.1")
-    spec.add_development_dependency "sassc-embedded", "1.54.0"
-    spec.add_development_dependency "sass-embedded", "1.54.2"
-  else
-    spec.add_development_dependency "sassc-embedded", "~> 1"
-    spec.add_development_dependency "sass-embedded", "~> 1"
-  end
-  #spec.add_development_dependency "sassc-embedded", "~> 1"
-=end
-  #spec.add_development_dependency "sassc"
+  spec.add_development_dependency "rubocop-performance"
   spec.add_development_dependency "sassc-embedded", "~> 1"
   spec.add_development_dependency "simplecov", "~> 0.15"
   spec.add_development_dependency "timecop", "~> 0.9"

data/lib/isodoc/common.rb

@@ -2,6 +2,7 @@ require_relative "function/blocks"
 require_relative "function/cleanup"
 require_relative "function/form"
 require_relative "function/inline"
+require_relative "function/footnotes"
 require_relative "function/lists"
 require_relative "function/references"
 require_relative "function/section"
@@ -18,6 +19,7 @@ module IsoDoc
     include Function::Cleanup
     include Function::Form
     include Function::Inline
+    include Function::Footnotes
     include Function::Lists
     include Function::References
     include Function::Section

data/lib/isodoc/function/blocks.rb

@@ -192,6 +192,18 @@ module IsoDoc
       end
 
       def columnbreak_parse(node, out); end
+
+      # we dont't want figure dl/dd/fmt-fn-body to be processed
+      # as actual footnotes
+      def fmt_fn_body_parse(node, out)
+        node.ancestors("table, figure").empty? and
+          node.at(ns(".//fmt-fn-label"))&.remove
+        aside = node.parent.name == "fmt-footnote-container"
+        tag = aside ? "aside" : "div"
+        out.send tag, id: "fn:#{node['reference']}", class: "footnote" do |div|
+          node.children.each { |n| parse(n, div) }
+        end
+      end
     end
   end
 end

data/lib/isodoc/function/cleanup.rb

@@ -16,7 +16,7 @@ module IsoDoc
       def cleanup(docxml)
         @i18n ||= i18n_init(@lang, @script, @locale)
         comment_cleanup(docxml)
-        footnote_cleanup(docxml)
+        #footnote_cleanup(docxml)
         inline_header_cleanup(docxml)
         figure_cleanup(docxml)
         table_cleanup(docxml)
@@ -58,7 +58,7 @@ module IsoDoc
         docxml
       end
 
-      # todo PRESENTATION XML
+      # KILL
       def footnote_cleanup(docxml)
         docxml.xpath('//a[@class = "FootnoteRef"]/sup')
           .each_with_index do |x, i|
@@ -68,7 +68,7 @@ module IsoDoc
       end
 
       def merge_fnref_into_fn_text(elem)
-        fn = elem.at('.//span[@class="TableFootnoteRef"]/..')
+        fn = elem.at('.//span[@class="TableFootnoteRef"]/..') or return
         n = fn.next_element
         n&.children&.first&.add_previous_sibling(fn.remove)
       end
@@ -80,7 +80,7 @@ module IsoDoc
             merge_fnref_into_fn_text(a)
             a.name = "div"
             a["class"] = "TableFootnote"
-            t << a.remove
+            t << a.remove # this is redundant
           end
         end
         table_footnote_cleanup_propagate(docxml)
@@ -142,14 +142,6 @@ module IsoDoc
       end
 
       def symbols_cleanup(docxml); end
-
-      def table_footnote_reference_format(link)
-        link
-      end
-
-      def footnote_reference_format(link)
-        link
-      end
     end
   end
 end

data/lib/isodoc/function/footnotes.rb

@@ -0,0 +1,57 @@
+module IsoDoc
+  module Function
+    module Footnotes
+      def make_table_footnote_link(out, fnid, node)
+        attrs = { href: "##{fnid}", class: "TableFootnoteRef" }
+        sup = node.at(ns("./sup")) and sup.replace(sup.children)
+        out.a **attrs do |a|
+          children_parse(node, a)
+        end
+      end
+
+      def get_table_ancestor_id(node)
+        table = node.ancestors("table")
+        table.empty? and table = node.ancestors("figure")
+        table.empty? and return [nil, UUIDTools::UUID.random_create.to_s]
+        [table.last, table.last["id"]]
+      end
+
+      # @seen_footnote:
+      # do not output footnote text if we have already seen it for this table
+
+      def table_footnote_parse(node, out)
+        fn = node["reference"] || UUIDTools::UUID.random_create.to_s
+        table, tid = get_table_ancestor_id(node)
+        make_table_footnote_link(out, tid + fn, node.at(ns("./fmt-fn-label")))
+        # do not output footnote text if we have already seen it for this table
+        @seen_footnote.include?(tid + fn) and return
+        update_table_fn_body_ref(node, table, tid + fn)
+        @seen_footnote << (tid + fn)
+      end
+
+      def update_table_fn_body_ref(fnote, table, reference)
+        fnbody = table.at(ns(".//fmt-fn-body[@id = '#{fnote['target']}']")) or return
+        fnbody["reference"] = reference
+        sup = fnbody.at(ns(".//fmt-fn-label/sup")) and sup.replace(sup.children)
+        fnbody.xpath(ns(".//fmt-fn-label")).each do |s|
+          s["class"] = "TableFootnoteRef"
+          s.name = "span"
+          d = s.at(ns("./span[@class = 'fmt-caption-delim']")) and
+            s.next = d
+        end
+      end
+
+      def footnote_parse(node, out)
+        return table_footnote_parse(node, out) if (@in_table || @in_figure) &&
+          !node.ancestors.map(&:name).include?("fmt-name")
+
+        fn = node["reference"] || UUIDTools::UUID.random_create.to_s
+        attrs = { class: "FootnoteRef", href: "#fn:#{fn}" }
+        f = node.at(ns("./fmt-fn-label"))
+        out.a **attrs do |a|
+          children_parse(f, a)
+        end
+      end
+    end
+  end
+end

data/lib/isodoc/function/inline.rb

@@ -15,8 +15,6 @@ module IsoDoc
         end
       end
 
-      def location_parse(node, out); end
-
       # Presentation XML classes which we need not pass on to HTML or DOC
       SPAN_UNWRAP_CLASSES =
         %w[fmt-caption-label fmt-label-delim fmt-caption-delim fmt-autonum-delim
@@ -189,8 +187,6 @@ module IsoDoc
         end
       end
 
-      def author_parse(node, out); end
-
       def semx_parse(node, out)
         children_parse(node, out)
       end
@@ -199,8 +195,9 @@ module IsoDoc
         node.children.each { |n| parse(n, out) }
       end
 
+      def location_parse(node, out); end
+      def author_parse(node, out); end
       def xref_label_parse(node, out); end
-
       def name_parse(node, out); end
       def semx_definition_parse(node, out); end
       def semx_xref_parse(node, out); end
@@ -209,7 +206,6 @@ module IsoDoc
       def semx_origin_parse(node, out); end
       def date_parse(node, out); end
       def semx_stem_parse(node, out); end
-
       def floating_title_parse(node, out); end
       def identifier_parse(node, out); end
       def concept_parse(node, out); end
@@ -233,6 +229,14 @@ module IsoDoc
       def fmt_date_parse(node, out)
         children_parse(node, out)
       end
+
+      def fmt_fn_label_parse(node, out)
+        children_parse(node, out)
+      end
+
+      def fmt_footnote_container_parse(node, out)
+        children_parse(node, out)
+      end
     end
   end
 end

data/lib/isodoc/function/section.rb

@@ -219,6 +219,13 @@ module IsoDoc
           node.children.each { |n| parse(n, div) }
         end
       end
+
+      def footnotes(docxml, div)
+        docxml.xpath(ns("/*/fmt-footnote-container"))
+          .each do |fn|
+            fn.children.each { |n| parse(n, div) }
+          end
+      end
     end
   end
 end

data/lib/isodoc/function/setup.rb

@@ -0,0 +1,64 @@
+require "fileutils"
+require "pathname"
+
+module IsoDoc
+  module Function
+    module ToWordHtml
+      def note?
+        @note
+      end
+
+      def init_file(filename, debug)
+        filepath = Pathname.new(filename)
+        filename = filepath.sub_ext("").sub(/\.presentation$/, "").to_s
+        dir = init_dir(filename, debug)
+        @filename = filename
+        @localdir = @baseassetpath || filepath.parent.to_s
+        @localdir += "/"
+        @sourcedir = @localdir
+        @sourcefilename and
+          @sourcedir = "#{Pathname.new(@sourcefilename).parent}/"
+        [filename, dir]
+      end
+
+      def init_dir(filename, debug)
+        dir = "#{filename}#{@tmpfilesdir_suffix}"
+        unless debug
+          FileUtils.mkdir_p(dir)
+          FileUtils.chmod 0o777, dir
+          FileUtils.rm_rf "#{dir}/*"
+        end
+        dir
+      end
+
+      # tmp image dir is same directory as @filename
+      def tmpimagedir
+        @filename + @tmpimagedir_suffix
+      end
+
+      def rel_tmpimagedir
+        Pathname.new(@filename).basename.to_s + @tmpimagedir_suffix
+      end
+
+      def info(isoxml, out)
+        @meta.code_css isoxml, out
+        @meta.title isoxml, out
+        @meta.subtitle isoxml, out
+        @meta.docstatus isoxml, out
+        @meta.docid isoxml, out
+        @meta.otherid isoxml, out
+        @meta.docnumeric isoxml, out
+        @meta.doctype isoxml, out
+        @meta.author isoxml, out
+        @meta.bibdate isoxml, out
+        @meta.relations isoxml, out
+        @meta.version isoxml, out
+        @meta.url isoxml, out
+        @meta.keywords isoxml, out
+        @meta.note isoxml, out
+        @meta.presentation isoxml, out
+        @meta.get
+      end
+    end
+  end
+end

data/lib/isodoc/function/table.rb

@@ -86,6 +86,8 @@ module IsoDoc
         (dl = node.at(ns("./dl"))) && parse(dl, out)
         node.xpath(ns("./source")).each { |n| parse(n, out) }
         node.xpath(ns("./note")).each { |n| parse(n, out) }
+        node.xpath(ns("./fmt-footnote-container/fmt-fn-body"))
+          .each { |n| parse(n, out) }
       end
 
       def table_parse_core(node, out)

data/lib/isodoc/function/to_word_html.rb

@@ -1,45 +1,10 @@
 require "fileutils"
 require "pathname"
+require_relative "setup"
 
 module IsoDoc
   module Function
     module ToWordHtml
-      def note?
-        @note
-      end
-
-      def init_file(filename, debug)
-        filepath = Pathname.new(filename)
-        filename = filepath.sub_ext("").sub(/\.presentation$/, "").to_s
-        dir = init_dir(filename, debug)
-        @filename = filename
-        @localdir = @baseassetpath || filepath.parent.to_s
-        @localdir += "/"
-        @sourcedir = @localdir
-        @sourcefilename and
-          @sourcedir = "#{Pathname.new(@sourcefilename).parent}/"
-        [filename, dir]
-      end
-
-      def init_dir(filename, debug)
-        dir = "#{filename}#{@tmpfilesdir_suffix}"
-        unless debug
-          FileUtils.mkdir_p(dir)
-          FileUtils.chmod 0o777, dir
-          FileUtils.rm_rf "#{dir}/*"
-        end
-        dir
-      end
-
-      # tmp image dir is same directory as @filename
-      def tmpimagedir
-        @filename + @tmpimagedir_suffix
-      end
-
-      def rel_tmpimagedir
-        Pathname.new(@filename).basename.to_s + @tmpimagedir_suffix
-      end
-
       # isodoc.css overrides any CSS injected by Html2Doc, which
       # is inserted before this CSS.
       def define_head(head, _filename, _dir)
@@ -88,7 +53,7 @@ module IsoDoc
         body.div class: "main-section" do |div3|
           boilerplate docxml, div3
           content(div3, docxml, ns(self.class::TOP_ELEMENTS))
-          footnotes div3
+          footnotes docxml, div3
           comments div3
         end
       end
@@ -129,26 +94,6 @@ module IsoDoc
         end
       end
 
-      def info(isoxml, out)
-        @meta.code_css isoxml, out
-        @meta.title isoxml, out
-        @meta.subtitle isoxml, out
-        @meta.docstatus isoxml, out
-        @meta.docid isoxml, out
-        @meta.otherid isoxml, out
-        @meta.docnumeric isoxml, out
-        @meta.doctype isoxml, out
-        @meta.author isoxml, out
-        @meta.bibdate isoxml, out
-        @meta.relations isoxml, out
-        @meta.version isoxml, out
-        @meta.url isoxml, out
-        @meta.keywords isoxml, out
-        @meta.note isoxml, out
-        @meta.presentation isoxml, out
-        @meta.get
-      end
-
       def cross_align(isoxml, out)
         isoxml.xpath(ns("//cross-align")).each do |c|
           parse(c, out)
@@ -286,6 +231,9 @@ module IsoDoc
           when "amend" then amend_parse(node, out)
           when "date" then date_parse(node, out)
           when "fmt-date" then fmt_date_parse(node, out)
+          when "fmt-fn-body" then fmt_fn_body_parse(node, out)
+          when "fmt-fn-label" then fmt_fn_label_parse(node, out)
+          when "fmt-footnote-container" then fmt_footnote_container_parse(node, out)
           else error_parse(node, out)
           end
         end

data/lib/isodoc/function/utils.rb

@@ -107,8 +107,7 @@ module IsoDoc
       end
 
       def sentence_join(array)
-        return "" if array.nil? || array.empty?
-
+        array.nil? || array.empty? and return ""
         if array.length == 1 then array[0]
         else
           @i18n.l10n("#{array[0..-2].join(', ')} " \
@@ -242,11 +241,9 @@ module IsoDoc
       def external_path(path)
         win = !!((RUBY_PLATFORM =~ /(win|w)(32|64)$/) ||
                  (RUBY_PLATFORM =~ /mswin|mingw/))
-        if win
-          path.tr!(%{/}, "\\")
-          path[/\s/] ? "\"#{path}\"" : path
-        else path
-        end
+        win or return path
+        path.tr!(%{/}, "\\")
+        path[/\s/] ? "\"#{path}\"" : path
       end
     end
   end

data/lib/isodoc/headlesshtml_convert.rb

@@ -1,12 +1,10 @@
 require_relative "html_function/comments"
-require_relative "html_function/footnotes"
 require_relative "html_function/html"
 require "fileutils"
 
 module IsoDoc
   class HeadlessHtmlConvert < ::IsoDoc::Convert
     include HtmlFunction::Comments
-    include HtmlFunction::Footnotes
     include HtmlFunction::Html
 
     def tmpimagedir_suffix

data/lib/isodoc/html_convert.rb

@@ -1,5 +1,4 @@
 require_relative "html_function/comments"
-require_relative "html_function/footnotes"
 require_relative "html_function/html"
 require_relative "html_function/postprocess"
 require_relative "html_function/form"
@@ -8,7 +7,6 @@ module IsoDoc
   class HtmlConvert < ::IsoDoc::Convert
 
     include HtmlFunction::Comments
-    include HtmlFunction::Footnotes
     include HtmlFunction::Form
     include HtmlFunction::Html
 

data/lib/isodoc/html_function/postprocess.rb

@@ -26,7 +26,8 @@ module IsoDoc
 
       def html_cleanup(html)
         html = term_header(html_footnote_filter(html_preface(htmlstyle(html))))
-        html = footnote_format(footnote_backlinks(html))
+        #html = footnote_format(footnote_backlinks(html))
+        html = (footnote_backlinks(html))
         html = mathml(html_list_clean(remove_placeholder_paras(html)))
         html_toc(heading_anchors(sourcecode_cleanup(html)))
       end

data/lib/isodoc/html_function/postprocess_footnotes.rb

@@ -49,7 +49,8 @@ module IsoDoc
         docxml
       end
 
-      def footnote_format(docxml)
+      # KILL
+      def footnote_formatx(docxml)
         docxml.xpath("//a[@class = 'FootnoteRef']/sup").each do |x|
           footnote_reference_format(x)
         end

data/lib/isodoc/pdf_convert.rb

@@ -1,11 +1,9 @@
 require_relative "html_function/comments"
-require_relative "html_function/footnotes"
 require_relative "html_function/html"
 
 module IsoDoc
   class PdfConvert < ::IsoDoc::Convert
     include HtmlFunction::Comments
-    include HtmlFunction::Footnotes
     include HtmlFunction::Html
 
     def initialize(options)

data/lib/isodoc/presentation_function/block.rb

@@ -8,7 +8,8 @@ module IsoDoc
     def lower2cap(text)
       text.nil? and return text
       x = Nokogiri::XML("<a>#{text}</a>")
-      firsttext = x.at(".//text()[string-length(normalize-space(.))>0]") or return text
+      firsttext = x.at(".//text()[string-length(normalize-space(.))>0]") or
+        return text
       /^[[:upper:]][[:upper:]]/.match?(firsttext.text) and return text
       firsttext.replace(firsttext.text.capitalize)
       to_xml(x.root.children)
@@ -80,7 +81,6 @@ module IsoDoc
     end
 
     def admonition_numbered1(elem)
-      # elem["unnumbered"] && !elem.at(ns("./name")) and return
       label = admonition_label(elem, @xrefs.anchor(elem["id"], :label, false))
       prefix_name(elem, { caption: block_delim }, label, "name")
     end
@@ -88,8 +88,6 @@ module IsoDoc
     def admonition_label(elem, num)
       lbl = if elem["type"] == "box" then @i18n.box
             else @i18n.admonition[elem["type"]]&.upcase end
-      #lbl &&= "<span class='fmt-element-name'>#{lbl}</span>"
-      #num and lbl = l10n("#{lbl} #{autonum(elem['id'], num)}")
       labelled_autonum(lbl, elem["id"], num)
     end
 
@@ -107,8 +105,6 @@ module IsoDoc
       labelled_ancestor(elem) and return
       elem["unnumbered"] && !elem.at(ns("./name")) and return
       n = @xrefs.anchor(elem["id"], :label, false)
-      #lbl = "<span class='fmt-element-name'>#{lower2cap @i18n.table}</span> "\
-        #"#{autonum(elem['id'], n)}"
       lbl = labelled_autonum(lower2cap(@i18n.table), elem["id"], n)
       prefix_name(elem, { caption: table_delim }, l10n(lbl), "name")
     end
@@ -128,15 +124,6 @@ module IsoDoc
       end
     end
 
-    def table_fn(elem)
-      (elem.xpath(ns(".//fn")) - elem.xpath(ns("./name//fn")))
-        .each_with_index do |f, i|
-          table_fn1(elem, f, i)
-        end
-    end
-
-    def table_fn1(table, fnote, idx); end
-
     def amend(docxml)
       docxml.xpath(ns("//amend")).each { |f| amend1(f) }
     end
@@ -233,19 +220,28 @@ module IsoDoc
       author = elem.at(ns("./author"))
       source = elem.at(ns("./source"))
       author.nil? && source.nil? and return
+      p = quote_attribution(author, source, elem)
+      elem << "<attribution><p>#{l10n p}</p></attribution>"
+    end
+
+    # e["deleteme"]: duplicate of source, will be duplicated in fmt-eref,
+    # need to delete after
+    def quote_attribution(author, source, elem)
       p = "&#x2014; "
       p += to_xml(semx_fmt_dup(author)) if author
       p += ", " if author && source
-      if source
-        s = semx_fmt_dup(source)
-e = Nokogiri::XML::Node.new("eref", elem.document)
-e << s.children
-s << e
-source.attributes.each_key { |k| e[k] = source[k] }
-e["deleteme"] = "true" # duplicate of source, will be duplicated in fmt-eref, need to delete after
-        p += to_xml(s)
-      end
-      elem << "<attribution><p>#{l10n p}</p></attribution>"
+      source or return p
+      p + to_xml(quote_source(source, elem))
+    end
+
+    def quote_source(source, elem)
+      s = semx_fmt_dup(source)
+      e = Nokogiri::XML::Node.new("eref", elem.document)
+      e << s.children
+      s << e
+      source.attributes.each_key { |k| e[k] = source[k] }
+      e["deleteme"] = "true"
+      s
     end
   end
 end

data/lib/isodoc/presentation_function/concepts.rb

@@ -20,7 +20,6 @@ module IsoDoc
     end
 
     def concept_render(node, defaults)
-      #require "debug"; binding.b
       opts, render, ref, ret = concept_render_init(node, defaults)
       ret&.at(ns("./refterm"))&.remove
       ref && opts[:ref] != "false" and render&.next = " "
@@ -31,7 +30,8 @@ module IsoDoc
     end
 
     def concept_dup(node, ret)
-      node.xpath(".//xmlns:semx[xmlns:fmt-xref | xmlns:fmt-eref | xmlns:fmt-origin | xmlns:fmt-link]").each(&:remove)
+      node.xpath(".//xmlns:semx[xmlns:fmt-xref | xmlns:fmt-eref | " \
+        "xmlns:fmt-origin | xmlns:fmt-link]").each(&:remove)
       ret.xpath(ns(".//xref | .//eref | .//origin | .//link")).each(&:remove)
       ret.xpath(ns(".//semx")).each do |s|
         s.children.empty? and s.remove
@@ -66,10 +66,11 @@ module IsoDoc
       (opts[:linkmention] == "true" && !renderterm.nil? && !ref.nil?) or return
       ref2 = ref.clone
       r2 = renderterm.clone
-      #renderterm.replace(ref2).children = r2
       ref2.children = r2
       if ref.parent.name == "semx"
-        renderterm.replace("<semx element='#{ref.parent['element']}' source='#{ref.parent['source']}'>#{to_xml(ref2)}</semx>")
+        renderterm.replace(<<~SEMX)
+          <semx element='#{ref.parent['element']}' source='#{ref.parent['source']}'>#{to_xml(ref2)}</semx>
+        SEMX
       else
         renderterm.replace(ref2)
       end
@@ -103,7 +104,7 @@ module IsoDoc
       p, ref, orig = related1_prep(node)
       label = @i18n.relatedterms[orig["type"]].upcase
       if p && ref
-        node.children =(l10n("<p><strong>#{label}:</strong> " \
+        node.children = (l10n("<p><strong>#{label}:</strong> " \
                           "<em>#{to_xml(p)}</em> (#{Common::to_xml(ref)})</p>"))
       else
         node.children = (l10n("<p><strong>#{label}:</strong> " \
@@ -198,8 +199,6 @@ module IsoDoc
     def designation_annotate(desgn, name, orig)
       designation_boldface(desgn)
       designation_field(desgn, name, orig)
-      #g = desgn.at(ns("./expression/grammar")) and
-        #name << ", #{designation_grammar(g).join(', ')}"
       designation_grammar(desgn, name)
       designation_localization(desgn, name, orig)
       designation_pronunciation(desgn, name)
@@ -213,7 +212,7 @@ module IsoDoc
       name.children = "<strong>#{name.children}</strong>"
     end
 
-    def designation_field(desgn, name, orig)
+    def designation_field(_desgn, name, orig)
       f = orig.xpath(ns("./field-of-application | ./usage-info"))
         &.map { |u| to_xml(semx_fmt_dup(u)) }&.join(", ")
       f&.empty? and return nil