isodoc 1.6.3 → 1.6.7.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b9d5c91b697c02cdac2a8221f2995d29f812804b48039967bd94c8aef7ce10c0
-  data.tar.gz: cf66a1f8d4eb41297360341e0da8d16a9ddf0af2679f072d758a259186a291bd
+  metadata.gz: 17e14b978b65a7667074ab1dff17bd5dc76a58e6baf7ace593407dc921ddf558
+  data.tar.gz: 3622549bc44570ec6e8de62fb55a62af937e32306ccc444bfaf0a1063caae511
 SHA512:
-  metadata.gz: 4cfa5637d6c22e73dd2a6c5bc32841183691a5db4121102787fad3caddd61815541e7128b624f03d20bf00d1e94fed81e43aaa19d1b84260e7bde5cfc69d924e
-  data.tar.gz: 7015f57396172858edb4b8499e6c4706894ef540f6b96c95d5c9bf445580003a8e68d4930204e5731ec57dafa460c2cfbcf4da5572e82713070fd214bef5974e
+  metadata.gz: 2cffca2a878c9bc3e05d26b85edb21be9ff27a75b8fd6e9ab7b31cdb861915b41baeb9e4b4027fb5f8f5e751879143dd55ea7cc35d46d6feb507d2a83632ff2b
+  data.tar.gz: 5146c2a8457a5f41232b6176acb3bf5fd77bfb75941b9030f6c015463cfe7646e6a3df153423229a3c9a17e0aa6c02529062bffdd1e98ad116fb91f1d61f6306

data/.github/workflows/rake.yml

@@ -16,19 +16,9 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        ruby: [ '2.7', '2.6', '2.5', '2.4' ]
+        ruby: [ '3.0', '2.7', '2.6', '2.5' ]
         os: [ ubuntu-latest, windows-latest, macos-latest ]
         experimental: [ false ]
-        include:
-          - ruby: '3.0'
-            os: 'ubuntu-latest'
-            experimental: true
-          - ruby: '3.0'
-            os: 'windows-latest'
-            experimental: true
-          - ruby: '3.0'
-            os: 'macos-latest'
-            experimental: true
     steps:
       - uses: actions/checkout@master
 
@@ -49,5 +39,5 @@ jobs:
         with:
           token: ${{ secrets.METANORMA_CI_PAT_TOKEN || secrets.GITHUB_TOKEN }}
           repository: ${{ github.repository }}
-          event-type: notify
+          event-type: tests-passed
           client-payload: '{"ref": "${{ github.ref }}", "sha": "${{ github.sha }}"}'

data/.hound.yml

@@ -1,3 +1,5 @@
+# Auto-generated by Cimas: Do not edit it manually!
+# See https://github.com/metanorma/cimas
 ruby:
-  Enabled: true
+  enabled: true
   config_file: .rubocop.yml

data/.rubocop.yml

@@ -1,12 +1,10 @@
-# This project follows the Ribose OSS style guide.
-# https://github.com/riboseinc/oss-guides
-# All project-specific additions and overrides should be specified in this file.
+# Auto-generated by Cimas: Do not edit it manually!
+# See https://github.com/metanorma/cimas
 inherit_from:
   - https://raw.githubusercontent.com/riboseinc/oss-guides/master/ci/rubocop.yml
 
 # local repo-specific modifications
+# ...
 
 AllCops:
-  DisplayCopNames: false
-  StyleGuideCopsOnly: false
-  TargetRubyVersion: 2.4
+  TargetRubyVersion: 2.5

data/isodoc.gemspec

@@ -26,14 +26,14 @@ Gem::Specification.new do |spec|
   spec.require_paths = ["lib"]
   spec.files         = `git ls-files`.split("\n")
   spec.test_files    = `git ls-files -- {spec}/*`.split("\n")
-  spec.required_ruby_version = Gem::Requirement.new(">= 2.4.0")
+  spec.required_ruby_version = Gem::Requirement.new(">= 2.5.0")
 
   spec.add_dependency "asciimath"
   spec.add_dependency "html2doc", "~> 1.1.1"
   spec.add_dependency "htmlentities", "~> 4.3.4"
   spec.add_dependency "liquid", "~> 4"
   spec.add_dependency "metanorma", ">= 1.2.0"
-  spec.add_dependency "nokogiri", "~> 1.10.4"
+  spec.add_dependency "nokogiri", "~> 1.11.0"
   spec.add_dependency "relaton-cli"
   spec.add_dependency "roman-numerals"
   spec.add_dependency "thread_safe"
@@ -50,5 +50,6 @@ Gem::Specification.new do |spec|
   spec.add_development_dependency "rubocop", "~> 1.5.2"
   spec.add_development_dependency "sassc", "~> 2.4.0"
   spec.add_development_dependency "simplecov", "~> 0.15"
+  spec.add_development_dependency "metanorma-iso"
   spec.add_development_dependency "timecop", "~> 0.9"
 end

data/lib/isodoc-yaml/i18n-en.yaml

@@ -14,6 +14,7 @@ termsdefsymbols: Terms, definitions and symbols
 termsdefabbrev: Terms, definitions and abbreviated terms
 normref: Normative references
 bibliography: Bibliography
+preface: Preface
 clause: Clause
 annex: Annex
 appendix: Appendix

data/lib/isodoc-yaml/i18n-fr.yaml

@@ -14,6 +14,7 @@ termsdefsymbols: Terms, définitions et symboles
 termsdefabbrev: Terms, définitions et termes abrégés
 normref: Références normatives
 bibliography: Bibliographie
+preface: Préface
 clause: Article
 annex: Annexe
 appendix: Appendice

data/lib/isodoc-yaml/i18n-zh-Hans.yaml

@@ -14,6 +14,7 @@ termsdefsymbols: 术语、定义、符号和代号
 termsdefabbrev: 术语、定义、符号和缩略语
 normref: 规范性引用文件
 bibliography: 参考文献
+preface: 序言
 clause: 条
 annex: 附件
 appendix: 附录

data/lib/isodoc.rb

@@ -8,7 +8,6 @@ require "mime/types"
 require "image_size"
 require "set"
 require "pp"
-require "uuidtools"
 require "html2doc"
 require "liquid"
 require "htmlentities"
@@ -26,5 +25,4 @@ require "isodoc/xref"
 require "isodoc/i18n"
 
 module IsoDoc
-
 end

data/lib/isodoc/convert.rb

@@ -32,10 +32,13 @@ module IsoDoc
     # monospace: font to use for monospace text
     # suppressheadingnumbers: suppress heading numbers for clauses
     # scripts: Scripts file for HTML
+    # scripts_override: Override scripts file for HTML
     # scripts_pdf: Scripts file for PDF (not used in XSLT PDF)
     # datauriimage: Encode images in HTML output as data URIs
     # break_up_urls_in_tables: whether to insert spaces in URLs in tables
     #   every 40-odd chars
+    # sectionsplit: split up HTML output on sections
+    # bare: do not insert any prefatory material (coverpage, boilerplate)
     def initialize(options)
       @libdir ||= File.dirname(__FILE__) # rubocop:disable Lint/DisjunctiveAssignmentInConstructor
       options.merge!(default_fonts(options)) do |_, old, new|
@@ -64,12 +67,15 @@ module IsoDoc
       @scripts = options[:scripts] ||
         File.join(File.dirname(__FILE__), "base_style", "scripts.html")
       @scripts_pdf = options[:scripts_pdf]
+      @scripts_override = options[:scripts_override]
       @i18nyaml = options[:i18nyaml]
       @ulstyle = options[:ulstyle]
       @olstyle = options[:olstyle]
       @datauriimage = options[:datauriimage]
       @suppressheadingnumbers = options[:suppressheadingnumbers]
       @break_up_urls_in_tables = options[:break_up_urls_in_tables] == "true"
+      @sectionsplit = options[:sectionsplit] == "true"
+      @bare = options[:bare]
       @termdomain = ""
       @termexample = false
       @note = false
@@ -150,7 +156,7 @@ module IsoDoc
       [docxml, filename, dir]
     end
 
-    def convert(input_filename, file = nil, debug = false, 
+    def convert(input_filename, file = nil, debug = false,
                 output_filename = nil)
       file = File.read(input_filename, encoding: "utf-8") if file.nil?
       @openmathdelim, @closemathdelim = extract_delims(file)

data/lib/isodoc/function/blocks.rb

@@ -14,10 +14,11 @@ module IsoDoc::Function
       end
     end
 
-    def figure_name_parse(node, div, name)
-      return if name.nil? 
+    def figure_name_parse(_node, div, name)
+      return if name.nil?
+
       div.p **{ class: "FigureTitle", style: "text-align:center;" } do |p|
-        name.children.each { |n| parse(n, div) }
+        name.children.each { |n| parse(n, p) }
       end
     end
 
@@ -61,7 +62,7 @@ module IsoDoc::Function
     end
 
     def sourcecode_name_parse(node, div, name)
-      return if name.nil? 
+      return if name.nil?
 
       div.p **{ class: "SourceTitle", style: "text-align:center;" } do |p|
         name.children.each { |n| parse(n, p) }

data/lib/isodoc/function/cleanup.rb

@@ -1,16 +1,17 @@
 module IsoDoc::Function
   module Cleanup
     def textcleanup(docxml)
-      docxml = termref_cleanup(passthrough_cleanup(docxml))
+      termref_cleanup(passthrough_cleanup(docxml))
     end
 
     def termref_cleanup(docxml)
-      docxml.
-        gsub(/\s*\[MODIFICATION\]\s*\[\/TERMREF\]/, l10n(", #{@i18n.modified} [/TERMREF]")).
-        gsub(%r{\s*\[/TERMREF\]\s*</p>\s*<p>\s*\[TERMREF\]}, "; ").
-        gsub(/\[TERMREF\]\s*/, l10n("[#{@i18n.source}: ")).
-        gsub(%r{\s*\[\/TERMREF\]\s*}, l10n("]")).
-        gsub(/\s*\[MODIFICATION\]/, l10n(", #{@i18n.modified} &mdash; "))
+      docxml
+        .gsub(/\s*\[MODIFICATION\]\s*\[\/TERMREF\]/,
+              l10n(", #{@i18n.modified} [/TERMREF]"))
+        .gsub(%r{\s*\[/TERMREF\]\s*</p>\s*<p>\s*\[TERMREF\]}, "; ")
+        .gsub(/\[TERMREF\]\s*/, l10n("[#{@i18n.source}: "))
+        .gsub(%r{\s*\[/TERMREF\]\s*}, l10n("]"))
+        .gsub(/\s*\[MODIFICATION\]/, l10n(", #{@i18n.modified} &mdash; "))
     end
 
     def passthrough_cleanup(docxml)
@@ -21,7 +22,7 @@ module IsoDoc::Function
     end
 
     def cleanup(docxml)
-      @i18n ||= i18n_init(@lang, @script) 
+      @i18n ||= i18n_init(@lang, @script)
       comment_cleanup(docxml)
       footnote_cleanup(docxml)
       inline_header_cleanup(docxml)
@@ -34,30 +35,36 @@ module IsoDoc::Function
 
     def table_long_strings_cleanup(docxml)
       return unless @break_up_urls_in_tables == true
+
       docxml.xpath("//td | //th").each do |d|
         d.traverse do |n|
           next unless n.text?
+
           n.replace(HTMLEntities.new.encode(
-            break_up_long_strings(n.text)))
+                      break_up_long_strings(n.text),
+                    ))
         end
       end
     end
 
-    def break_up_long_strings(t)
-      return t if t.match(/^\s*$/)
-      t.split(/(?=\s)/).map do |w|
-        (/^\s*$/.match(t) or w.size < 30) ? w :
+    def break_up_long_strings(text)
+      return text if /^\s*$/.match?(text)
+
+      text.split(/(?=\s)/).map do |w|
+        if /^\s*$/.match(text) || (w.size < 30) then w
+        else
           w.scan(/.{,30}/).map do |w1|
-          (w1.size < 30) ? w1 : break_up_long_strings1(w1)
-        end.join
+            w1.size < 30 ? w1 : break_up_long_strings1(w1)
+          end.join
+        end
       end.join
     end
 
-    def break_up_long_strings1(w1)
-      s = w1.split(%r{(?<=[,.?+;/=])})
-      if s.size == 1 then w1 + " "
+    def break_up_long_strings1(text)
+      s = text.split(%r{(?<=[,.?+;/=])})
+      if s.size == 1 then "#{text} "
       else
-        s[-1] = " " + s[-1]
+        s[-1] = " #{s[-1]}"
         s.join
       end
     end
@@ -78,11 +85,11 @@ module IsoDoc::Function
       docxml
     end
 
-    def figure_get_or_make_dl(t)
-      dl = t.at(".//dl")
+    def figure_get_or_make_dl(elem)
+      dl = elem.at(".//dl")
       if dl.nil?
-        t.add_child("<p><b>#{@i18n.key}</b></p><dl></dl>")
-        dl = t.at(".//dl")
+        elem.add_child("<p><b>#{@i18n.key}</b></p><dl></dl>")
+        dl = elem.at(".//dl")
       end
       dl
     end
@@ -91,10 +98,10 @@ module IsoDoc::Function
       "//div[@class = 'figure'][descendant::aside]"\
       "[not(descendant::div[@class = 'figure'])]".freeze
 
-    def figure_aside_process(f, aside, key)
+    def figure_aside_process(elem, aside, key)
       # get rid of footnote link, it is in diagram
-      f&.at("./a[@class='TableFootnoteRef']")&.remove
-      fnref = f.at(".//span[@class='TableFootnoteRef']/..")
+      elem&.at("./a[@class='TableFootnoteRef']")&.remove
+      fnref = elem.at(".//span[@class='TableFootnoteRef']/..")
       dt = key.add_child("<dt></dt>").first
       dd = key.add_child("<dd></dd>").first
       fnref.parent = dt
@@ -109,6 +116,7 @@ module IsoDoc::Function
     def figure_cleanup(docxml)
       docxml.xpath(FIGURE_WITH_FOOTNOTES).each do |f|
         next unless f.at(".//aside[not(ancestor::p[@class = 'FigureTitle'])]")
+
         key = figure_get_or_make_dl(f)
         f.xpath(".//aside").each do |aside|
           figure_aside_process(f, aside, key)
@@ -137,12 +145,13 @@ module IsoDoc::Function
       docxml
     end
 
-    def merge_fnref_into_fn_text(a)
-      fn = a.at('.//span[@class="TableFootnoteRef"]/..')
+    def merge_fnref_into_fn_text(elem)
+      fn = elem.at('.//span[@class="TableFootnoteRef"]/..')
       n = fn.next_element
       n&.children&.first&.add_previous_sibling(fn.remove)
     end
 
+    # preempt html2doc putting MsoNormal under TableFootnote class
     def table_footnote_cleanup(docxml)
       docxml.xpath("//table[descendant::aside]").each do |t|
         t.xpath(".//aside").each do |a|
@@ -152,36 +161,36 @@ module IsoDoc::Function
           t << a.remove
         end
       end
-      # preempt html2doc putting MsoNormal there
       docxml.xpath("//p[not(self::*[@class])]"\
                    "[ancestor::*[@class = 'TableFootnote']]").each do |p|
         p["class"] = "TableFootnote"
       end
     end
 
-    def remove_bottom_border(td)
-      td["style"] =
-        td["style"].gsub(/border-bottom:[^;]+;/, "border-bottom:0pt;")
+    def remove_bottom_border(cell)
+      cell["style"] =
+        cell["style"].gsub(/border-bottom:[^;]+;/, "border-bottom:0pt;")
     end
 
-    def table_get_or_make_tfoot(t)
-      tfoot = t.at(".//tfoot")
+    def table_get_or_make_tfoot(table)
+      tfoot = table.at(".//tfoot")
       if tfoot.nil?
-        t.add_child("<tfoot></tfoot>")
-        tfoot = t.at(".//tfoot")
+        table.add_child("<tfoot></tfoot>")
+        tfoot = table.at(".//tfoot")
       else
         tfoot.xpath(".//td | .//th").each { |td| remove_bottom_border(td) }
       end
       tfoot
     end
 
-    def new_fullcolspan_row(t, tfoot)
+    def new_fullcolspan_row(table, tfoot)
       # how many columns in the table?
       cols = 0
-      t.at(".//tr").xpath("./td | ./th").each do |td|
+      table.at(".//tr").xpath("./td | ./th").each do |td|
         cols += (td["colspan"] ? td["colspan"].to_i : 1)
       end
-      style = %{border-top:0pt;border-bottom:#{IsoDoc::Function::Table::SW} 1.5pt;}
+      style =
+        %{border-top:0pt;border-bottom:#{IsoDoc::Function::Table::SW} 1.5pt;}
       tfoot.add_child("<tr><td colspan='#{cols}' style='#{style}'/></tr>")
       tfoot.xpath(".//td").last
     end
@@ -206,12 +215,12 @@ module IsoDoc::Function
 
     def symbols_cleanup(docxml); end
 
-    def table_footnote_reference_format(a)
-      a
+    def table_footnote_reference_format(link)
+      link
     end
 
-    def footnote_reference_format(a)
-      a
+    def footnote_reference_format(link)
+      link
     end
   end
 end

data/lib/isodoc/function/inline.rb

@@ -3,11 +3,12 @@ require_relative "inline_simple"
 module IsoDoc::Function
   module Inline
     def link_parse(node, out)
-      out.a **attr_code(href: node["target"], title: node["alt"]) do |l|
+      url = node["target"]
+      node["updatetype"] == "true" and url = suffix_url(url)
+      out.a **attr_code(href: url, title: node["alt"]) do |l|
         if node.text.empty?
           l << node["target"].sub(/^mailto:/, "")
-        else
-          node.children.each { |n| parse(n, l) }
+        else node.children.each { |n| parse(n, l) }
         end
       end
     end
@@ -79,8 +80,7 @@ module IsoDoc::Function
                "#{@openmathdelim}#{HTMLEntities.new.encode(node.text)}"\
                  "#{@closemathdelim}"
              when "MathML" then node.first_element_child.to_s
-             else
-               HTMLEntities.new.encode(node.text)
+             else HTMLEntities.new.encode(node.text)
              end
       out.span **{ class: "stem" } do |span|
         span.parent.add_child ooml
@@ -123,13 +123,13 @@ module IsoDoc::Function
     end
 
     def add_parse(node, out)
-      out.span **{class: "addition"} do |e|
+      out.span **{ class: "addition" } do |e|
         node.children.each { |n| parse(n, e) }
       end
     end
 
     def del_parse(node, out)
-      out.span **{class: "deletion"} do |e|
+      out.span **{ class: "deletion" } do |e|
         node.children.each { |n| parse(n, e) }
       end
     end

data/lib/isodoc/function/references.rb

@@ -10,9 +10,9 @@ module IsoDoc::Function
     end
 
     # TODO generate formatted ref if not present
-    def nonstd_bibitem(list, b, ordinal, biblio)
-      list.p **attr_code(iso_bibitem_entry_attrs(b, biblio)) do |ref|
-        ids = bibitem_ref_code(b)
+    def nonstd_bibitem(list, bib, ordinal, biblio)
+      list.p **attr_code(iso_bibitem_entry_attrs(bib, biblio)) do |ref|
+        ids = bibitem_ref_code(bib)
         identifiers = render_identifier(ids)
         if biblio then ref_entry_code(ref, ordinal, identifiers, ids)
         else
@@ -20,21 +20,21 @@ module IsoDoc::Function
           ref << ", #{identifiers[1]}" if identifiers[0] && identifiers[1]
         end
         ref << ", " unless biblio && !identifiers[1]
-        reference_format(b, ref)
+        reference_format(bib, ref)
       end
     end
 
-    def std_bibitem_entry(list, b, ordinal, biblio)
-      list.p **attr_code(iso_bibitem_entry_attrs(b, biblio)) do |ref|
-        identifiers = render_identifier(bibitem_ref_code(b))
+    def std_bibitem_entry(list, bib, ordinal, biblio)
+      list.p **attr_code(iso_bibitem_entry_attrs(bib, biblio)) do |ref|
+        identifiers = render_identifier(bibitem_ref_code(bib))
         if biblio then ref_entry_code(ref, ordinal, identifiers, nil)
         else
           ref << (identifiers[0] || identifiers[1]).to_s
           ref << ", #{identifiers[1]}" if identifiers[0] && identifiers[1]
         end
-        date_note_process(b, ref)
+        date_note_process(bib, ref)
         ref << ", " unless biblio && !identifiers[1]
-        reference_format(b, ref)
+        reference_format(bib, ref)
       end
     end
 
@@ -45,20 +45,20 @@ module IsoDoc::Function
       t[1] and r << (t[1]).to_s
     end
 
-    def pref_ref_code(b)
-      b.at(ns("./docidentifier[not(@type = 'DOI' or @type = 'metanorma' "\
+    def pref_ref_code(bib)
+      bib.at(ns("./docidentifier[not(@type = 'DOI' or @type = 'metanorma' "\
               "or @type = 'ISSN' or @type = 'ISBN' or @type = 'rfc-anchor')]"))
     end
 
     # returns [metanorma, non-metanorma, DOI/ISSN/ISBN] identifiers
-    def bibitem_ref_code(b)
-      id = b.at(ns("./docidentifier[@type = 'metanorma']"))
-      id1 = pref_ref_code(b)
-      id2 = b.at(ns("./docidentifier[@type = 'DOI' or @type = 'ISSN' or "\
+    def bibitem_ref_code(bib)
+      id = bib.at(ns("./docidentifier[@type = 'metanorma']"))
+      id1 = pref_ref_code(bib)
+      id2 = bib.at(ns("./docidentifier[@type = 'DOI' or @type = 'ISSN' or "\
                     "@type = 'ISBN']"))
       return [id, id1, id2] if id || id1 || id2
 
-      id = Nokogiri::XML::Node.new("docidentifier", b.document)
+      id = Nokogiri::XML::Node.new("docidentifier", bib.document)
       id << "(NO ID)"
       [nil, id, nil]
     end
@@ -72,20 +72,10 @@ module IsoDoc::Function
       num
     end
 
-    def render_identifier(id)
-      [
-        bracket_if_num(id[0]),
-        if id[1].nil?
-          nil
-        else
-          docid_prefix(id[1]["type"], id[1].text.sub(/^\[/, "").sub(/\]$/, ""))
-        end,
-        if id[2].nil?
-          nil
-        else
-          docid_prefix(id[2]["type"], id[2].text.sub(/^\[/, "").sub(/\]$/, ""))
-        end,
-      ]
+    def render_identifier(ident)
+      [bracket_if_num(ident[0]),
+       ident[1].nil? ? nil : ident[1].text.sub(/^\[/, "").sub(/\]$/, ""),
+       ident[2].nil? ? nil : ident[2].text.sub(/^\[/, "").sub(/\]$/, "")]
     end
 
     def docid_prefix(prefix, docid)
@@ -97,19 +87,20 @@ module IsoDoc::Function
     def omit_docid_prefix(prefix)
       return true if prefix.nil? || prefix.empty?
 
-      %w(ISO IEC IEV ITU W3C csd metanorma rfc-anchor).include? prefix
+      %w(ISO IEC IEV ITU W3C csd metanorma repository rfc-anchor)
+        .include? prefix
     end
 
-    def date_note_process(b, ref)
-      date_note = b.at(ns("./note[@type = 'Unpublished-Status']"))
+    def date_note_process(bib, ref)
+      date_note = bib.at(ns("./note[@type = 'Unpublished-Status']"))
       return if date_note.nil?
 
       date_note.children.first.replace("<p>#{date_note.content}</p>")
       footnote_parse(date_note, ref)
     end
 
-    def iso_bibitem_entry_attrs(b, biblio)
-      { id: b["id"], class: biblio ? "Biblio" : "NormRef" }
+    def iso_bibitem_entry_attrs(bib, biblio)
+      { id: bib["id"], class: biblio ? "Biblio" : "NormRef" }
     end
 
     def iso_title(bib)
@@ -130,22 +121,21 @@ module IsoDoc::Function
       insert_tab(ref, 1)
     end
 
-    def reference_format(bib, r)
+    def reference_format(bib, out)
       if ftitle = bib.at(ns("./formattedref"))
-        ftitle&.children&.each { |n| parse(n, r) }
+        ftitle&.children&.each { |n| parse(n, out) }
       else
-        title = iso_title(bib)
-        r.i do |i|
-          title&.children&.each { |n| parse(n, i) }
+        out.i do |i|
+          iso_title(bib)&.children&.each { |n| parse(n, i) }
         end
       end
     end
 
     def is_standard(bib)
       ret = false
+      drop = %w(metanorma DOI ISSN ISBN)
       bib.xpath(ns("./docidentifier")).each do |id|
-        next if id["type"].nil? ||
-          %w(metanorma DOI ISSN ISBN).include?(id["type"])
+        next if id["type"].nil? || drop.include?(id["type"])
 
         ret = true
       end
@@ -180,8 +170,7 @@ module IsoDoc::Function
         clause_name(num, f.at(ns("./title")), div, nil)
         if f.name == "clause"
           f.elements.each { |e| parse(e, div) unless e.name == "title" }
-        else
-          biblio_list(f, div, false)
+        else biblio_list(f, div, false)
         end
       end
       num
@@ -212,13 +201,5 @@ module IsoDoc::Function
         biblio_list(node, div, true)
       end
     end
-
-    def format_ref(ref, prefix, _isopub, _date, _allparts)
-      ref = docid_prefix(prefix, ref)
-      return "[#{ref}]" if ref && /^\d+$/.match(ref) && !prefix &&
-        !/^\[.*\]$/.match(ref)
-
-      ref
-    end
   end
 end