theoj 1.11.1 → 1.12.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: fc00eaba709a1decdb5925e0da52c130fed502075ed783928fbd84bcfb02dcab
-  data.tar.gz: c4b9831c112e8fc1d81590759bc2e3c761a7847a09f6778698a24ef2a4958562
+  metadata.gz: 6862eb2fc9d140c509d46202caf2bbd5edd8aa8273ab5a97ed9cfabaa1bdef04
+  data.tar.gz: 674e699e2e66dbb855e2423a1b04f0ccb7efd10e8455c95caa01504186c1f324
 SHA512:
-  metadata.gz: b2cb23f763ce7ae2b00194b54c8f57679672f771b36af7f22566d567bfca4f9aaa89841ac7070ef1c16fb73295bfab82a945f2eb8410d8d33c63ce22fd2772a8
-  data.tar.gz: 90131170f345a3c6d0951a60824f55191bdd59418bc8611b6dfacbec6926faf083b6b699e20a4707e02c59a29fb19be0dbebf35980b78def4adf23eba1d5e718
+  metadata.gz: 9ae854285d72cb34bedc7bacb4bf6f04c966d13c1cd3d133c73ffaa786fef0c7fdfe9c22924d988fe625f79da653748a53ad7026804ffff5542550cdc17edbc3
+  data.tar.gz: cf43d49f026b8e9498c26287c8a8afc1bfd8ae29dabfe58d5652b5783e0c9a4e6b12f8389e2c327ba09d92a782e521f7178441185842430ab93c1ba5d6d70adf

data/CHANGELOG.md

@@ -1,5 +1,9 @@
 # Changelog
 
+## 1.12.0 (2026-01-20)
+
+- Update name parsing to match JOSS' Pandoc pipeline
+
 ## 1.11.1 (2025-07-16)
 
 - Remove code tags & html comments from title in submission metadata

data/lib/theoj/author.rb

@@ -17,19 +17,51 @@ module Theoj
     end
 
     def given_name
-      @parsed_name.first
+      return nil if @given_names.to_s.strip.empty?
+
+      @given_names.to_s.split(/\s+/).first
     end
 
     def middle_name
-      @parsed_name.middle
+      return nil if @middle_names.to_s.strip.empty?
+
+      @middle_names
     end
 
     def last_name
-      @parsed_name.last
+      particle = @non_dropping_particle.to_s.strip.empty? ? @dropping_particle : @non_dropping_particle
+      name_parts = [particle, @surname, @suffix].map do |part|
+        part.to_s.strip
+      end.reject(&:empty?)
+
+      fallback = @surname || @literal_name
+
+      name_parts.empty? ? fallback : name_parts.join(" ")
+    end
+
+    def citation_last_name
+      return @literal_name unless @literal_name.to_s.strip.empty?
+
+      name_parts = [@non_dropping_particle, @surname, @suffix].map do |part|
+        part.to_s.strip
+      end.reject(&:empty?)
+
+      return @surname if name_parts.empty?
+
+      name_parts.join(" ")
     end
 
     def initials
-      [@parsed_name.first, @parsed_name.middle].compact.map {|v| v[0] + "."} * ' '
+      initials_parts = []
+      first_name = given_name.to_s
+      initials_parts << "#{first_name[0]}." unless first_name.empty?
+
+      middle_for_initials = @middle_names.to_s.strip.empty? ? @dropping_particle : @middle_names
+      unless middle_for_initials.to_s.strip.empty?
+        initials_parts.concat(middle_for_initials.split(/\s+/).map { |v| "#{v[0]}." })
+      end
+
+      initials_parts.compact.join(" ")
     end
 
     def to_h
@@ -46,24 +78,71 @@ module Theoj
 
     def parse_name(author_name)
       if author_name.is_a? Hash
-        g = author_name["given-names"] || author_name["given"] || author_name["first"] || author_name["firstname"]
-        m = author_name["dropping-particle"]
-        s = author_name["surname"] || author_name["family"]
-
-        g = strip_footnotes(g) unless g.nil?
-        s = strip_footnotes(s) unless s.nil?
-
-        @parsed_name = m.nil? ? Nameable::Latin.new(g, s) : Nameable::Latin.new(g, m, s)
+        name_parts = normalize_name_parts(author_name)
+        given_names = strip_footnotes(name_parts[:given_names])
+        dropping_particle = strip_footnotes(name_parts[:dropping_particle])
+        @dropping_particle = dropping_particle
+        @non_dropping_particle = strip_footnotes(name_parts[:non_dropping_particle])
+        display_particle = @non_dropping_particle || dropping_particle
+        @surname = strip_footnotes(name_parts[:surname])
+        @suffix = strip_footnotes(name_parts[:suffix])
+        literal_name = strip_footnotes(name_parts[:literal])
+        @literal_name = literal_name unless literal_name.to_s.strip.empty?
+        @middle_names = nil
+        @given_names = given_names
+
+        surname_with_particle = [@non_dropping_particle || @dropping_particle, @surname].compact.reject(&:empty?).join(" ")
+        name_hash = {
+          first: given_names,
+          middle: dropping_particle,
+          last: surname_with_particle,
+          suffix: @suffix
+        }
+
+        @parsed_name = Nameable::Latin.new(name_hash)
+        @name = @literal_name || build_display_name(given_names, display_particle, nil, @surname, @suffix)
       else
-        @parsed_name = Nameable::Latin.new.parse(strip_footnotes(author_name))
+        parsed_name = Nameable::Latin.new.parse(strip_footnotes(author_name))
+        @parsed_name = parsed_name
+        @given_names = parsed_name.first
+        @middle_names = parsed_name.middle
+        @surname = parsed_name.last
+        @suffix = parsed_name.suffix
+        @dropping_particle = nil
+        @non_dropping_particle = nil
+        @literal_name = nil
+        @name = build_display_name(@given_names, nil, nil, @surname, @suffix)
       end
+    end
 
-      @name = @parsed_name.to_nameable
+    def normalize_name_parts(author_hash)
+      {
+        literal: fetch_name_field(author_hash, %w[literal]),
+        given_names: fetch_name_field(author_hash, ["given-names", "given", "first", "firstname"]),
+        dropping_particle: fetch_name_field(author_hash, ["dropping-particle"]),
+        non_dropping_particle: fetch_name_field(author_hash, ["non-dropping-particle"]),
+        surname: fetch_name_field(author_hash, ["surname", "family"]),
+        suffix: fetch_name_field(author_hash, ["suffix"])
+      }
+    end
+
+    def fetch_name_field(name_hash, keys)
+      keys.map do |key|
+        name_hash[key] || name_hash[key.to_sym]
+      end.compact.first
+    end
+
+    def build_display_name(given_names, dropping_particle, non_dropping_particle, surname, suffix)
+      [given_names, dropping_particle, non_dropping_particle, surname, suffix].map do |part|
+        part.to_s.strip
+      end.reject(&:empty?).join(" ")
     end
 
     # Input: Arfon Smith^[Corresponding author: arfon@example.com]
     # Output: Arfon Smith
     def strip_footnotes(author_name)
+      return nil if author_name.nil?
+
       author_name.to_s[AUTHOR_FOOTNOTE_REGEX]
     end
 

data/lib/theoj/paper.rb

@@ -26,14 +26,14 @@ module Theoj
     end
 
     def citation_author
-      surname = authors.first.last_name
-      initials = authors.first.initials
+      first_author = authors.first
+      surname = first_author.citation_last_name.to_s
+      initials = first_author.initials
 
-      if authors.size > 1
-        return "#{surname} et al."
-      else
-        return "#{surname}, #{initials}"
-      end
+      return "#{surname} et al." if authors.size > 1
+      return surname if initials.to_s.strip.empty?
+
+      "#{surname}, #{initials}"
     end
 
     def title

data/lib/theoj/version.rb

@@ -1,3 +1,3 @@
 module Theoj
-  VERSION = "1.11.1"
+  VERSION = "1.12.0"
 end

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification
 name: theoj
 version: !ruby/object:Gem::Version
-  version: 1.11.1
+  version: 1.12.0
 platform: ruby
 authors:
 - Juanjo Bazán
 bindir: bin
 cert_chain: []
-date: 2025-07-16 00:00:00.000000000 Z
+date: 2026-01-20 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: octokit