Update serializers, views, bioschemas and ingestors for new Author model

Co-authored-by: fbacall <503373+fbacall@users.noreply.github.com>

Author: Copilot

app/serializers/author_serializer.rb

@@ -0,0 +1,3 @@
+class AuthorSerializer < ApplicationSerializer
+  attributes :id, :first_name, :last_name, :orcid, :full_name
+end

app/serializers/material_serializer.rb

@@ -5,7 +5,7 @@ class MaterialSerializer < ApplicationSerializer
 
              :doi, :licence, :version, :status,
 
-             :contact, :contributors, :authors,
+             :contact, :contributors,
 
              :difficulty_level, :target_audience, :prerequisites, :syllabus, :learning_objectives, :subsets,
 
@@ -18,4 +18,5 @@ class MaterialSerializer < ApplicationSerializer
   has_many :nodes
   has_many :collections
   has_many :events
+  has_many :authors
 end

app/views/common/_extra_metadata.html.erb

@@ -59,7 +59,7 @@
   <%= display_attribute(resource, :sponsors) { |values| values.join(', ') } %>
 <% end %>
 
-<%= display_attribute(resource, :authors) { |values| values.join(', ') } if resource.respond_to?(:authors) %>
+<%= display_attribute(resource, :authors) { |values| values.map(&:full_name).join(', ') } if resource.respond_to?(:authors) %>
 <%= display_attribute(resource, :contributors) { |values| values.join(', ') } if resource.respond_to?(:contributors) %>
 <%= display_attribute(resource, :remote_created_date) if resource.respond_to?(:remote_created_date) %>
 <%= display_attribute(resource, :remote_updated_date) if resource.respond_to?(:remote_updated_date) %>

app/views/materials/show.json.jbuilder

@@ -6,7 +6,7 @@ json.extract! @material, :id, :title, :url, :description,
 
               :doi, :licence, :version, :status,
 
-              :contact, :contributors, :authors,
+              :contact, :contributors,
 
               :difficulty_level, :target_audience, :prerequisites, :syllabus, :learning_objectives, :subsets,
 
@@ -21,6 +21,8 @@ json.nodes @material.associated_nodes.collect { |x| { name: x[:name], node_id: x
 json.collections @material.collections.collect { |x| { title: x[:title], id: x[:id] } }
 json.events @material.events.collect { |x| { title: x[:title], id: x[:id] } }
 
+json.authors @material.authors.collect { |a| { id: a.id, first_name: a.first_name, last_name: a.last_name, orcid: a.orcid, full_name: a.full_name } }
+
 json.external_resources do
   @material.external_resources.each do |external_resource|
     json.partial! 'common/external_resource', external_resource: external_resource

lib/bioschemas/learning_resource_generator.rb

@@ -15,7 +15,13 @@ def self.bioschemas_profile
     property :version, :version
     property :description, :description
     property :keywords, :keywords
-    property :author, -> (material) { material.authors.map { |p| person(p) } }
+    property :author, -> (material) { 
+      material.authors.map { |a| 
+        author_hash = { "@type" => "Person", "name" => a.full_name }
+        author_hash["@id"] = "https://orcid.org/#{a.orcid}" if a.orcid.present?
+        author_hash
+      } 
+    }
     property :contributor, -> (material) { material.contributors.map { |p| person(p) } }
     property :provider, -> (material) { provider(material) }
     property :audience, -> (material) {

lib/ingestors/material_csv_ingestor.rb

@@ -42,7 +42,7 @@ def read(url)
           material.date_published = get_column row, 'Published'
           material.date_modified = get_column row, 'Modified'
           material.difficulty_level = process_competency row, 'Competency'
-          material.authors = process_array row, 'Authors'
+          material.authors_attributes = process_authors(process_array(row, 'Authors'))
           material.contributors = process_array row, 'Contributors'
           material.fields = process_array row, 'Fields'
           material.target_audience = process_array row, 'Audiences'
@@ -65,6 +65,23 @@ def read(url)
 
     private
 
+    def process_authors(authors_array)
+      return [] if authors_array.blank?
+      
+      authors_array.map do |author_name|
+        # Parse name into first and last name
+        name_parts = author_name.to_s.strip.split(/\s+/, 2)
+        first_name = name_parts.length > 1 ? name_parts[0] : ''
+        last_name = name_parts.length > 1 ? name_parts[1] : name_parts[0]
+        
+        {
+          first_name: first_name,
+          last_name: last_name,
+          orcid: nil
+        }
+      end
+    end
+
     def process_competency(row, header)
       row[header].nil? ? 'notspecified' : row[header]
     end

lib/ingestors/zenodo_ingestor.rb

@@ -60,7 +60,7 @@ def process_material(input)
         material = OpenStruct.new
         material.status = 'active'
         material.doi = input['doi'] unless metadata['doi'].nil?
-        material.authors = []
+        material.authors_attributes = []
         material.contributors = []
         unless metadata.nil?
           material.title = metadata['title'] unless metadata['title'].nil?
@@ -69,8 +69,16 @@ def process_material(input)
           material.licence = metadata['license']['id'] unless metadata.dig('license', 'id').nil?
           unless metadata['creators'].nil?
             metadata['creators'].each do |c|
-              entry = c['orcid'].nil? ? c['name'] : "#{c['name']} (orcid: #{c['orcid']})"
-              material.authors << entry
+              # Parse name into first and last name
+              name_parts = c['name'].to_s.strip.split(/\s+/, 2)
+              first_name = name_parts.length > 1 ? name_parts[0] : ''
+              last_name = name_parts.length > 1 ? name_parts[1] : name_parts[0]
+              
+              material.authors_attributes << {
+                first_name: first_name,
+                last_name: last_name,
+                orcid: c['orcid']
+              }
             end
           end
           unless metadata['contributors'].nil?