hydra-mods 0.0.1

data/app/models/mods_asset.rb

@@ -0,0 +1,24 @@
+# An EXAMPLE   Basic Model for Assets that conform to Hydra commonMetadata cModel and have basic MODS metadata (currently "Article" is the MODS exemplar)
+class ModsAsset < ActiveFedora::Base
+  
+  # declares a rightsMetadata datastream with type Hydra::Datastream::RightsMetadata
+  #  basically, it is another expression of
+  #  has_metadata :name => "rightsMetadata", :type => Hydra::Datastream::RightsMetadata
+  include Hydra::ModelMixins::CommonMetadata
+
+  ## Convenience methods for manipulating the rights metadata datastream
+  include Hydra::ModelMixins::RightsMetadata
+  
+  # declares a descMetadata datastream with type Hydra::Datastream::ModsArticle
+  #  basically, it is another expression of
+  #  has_metadata :name => "descMetadata", :type => Hydra::Datastream::ModsArticle
+  include Hydra::ModelMixins::ModsObject
+  
+  # adds helpful methods for basic hydra objects
+  include Hydra::ModelMethods
+
+  # adds file_objects methods
+  include ActiveFedora::FileManagement
+
+  
+end

data/hydra-mods.gemspec

@@ -0,0 +1,20 @@
+$:.push File.expand_path("../lib", __FILE__)
+# require "hydra-head/version"
+
+Gem::Specification.new do |s|
+  s.name = 'hydra-mods'
+  s.version = '0.0.1'
+  s.platform = Gem::Platform::RUBY
+  s.authors     = ["Matt Zumwalt, Bess Sadler, Julie Meloni, Naomi Dushay, Jessie Keck, John Scofield, Justin Coyne & many more.  See https://github.com/projecthydra/hydra-head/contributors"]
+  s.email       = ["hydra-tech@googlegroups.com"]
+  s.homepage = 'http://projecthydra.org'
+  s.summary = "Hydra MODS components split out from hydra-head"
+  s.description = "Hydra MODS components split out from hydra-head"
+
+  s.add_dependency "hydra-head"
+
+  s.files = `git ls-files`.split("\n")
+  s.test_files = `git ls-files -- {test,spec,features}/*`.split("\n")
+  s.test_files = `git ls-files -- bin/*`.split("\n").map{ |f| File.basename(f) }
+  s.require_paths = ["lib", "app/models"]
+end

data/lib/hydra-mods.rb

@@ -0,0 +1,9 @@
+require 'hydra/mods'
+
+module HydraMods
+  require 'rails'
+
+  class Engine < Rails::Engine
+  
+  end 
+end

data/lib/hydra/common_mods_index_methods.rb

@@ -0,0 +1,46 @@
+# Provides some helper methods for indexing compound or non-standard facets
+#
+# == Methods
+# 
+# extract_person_full_names
+#   This method returns a Hash of person_full_name_facet values which combine Lastname, Firstname
+# extract_person_organizations
+#   This method returns a Hash of person_full_name_facet values which extract the persons affiliation and puts it in an mods_organization_facet
+# will be removed in release 5.x
+module Hydra::CommonModsIndexMethods
+
+  def self.included(base)
+    ActiveSupport::Deprecation.warn("Hydra::CommonModsIndexMethods has been deprecated. Use Hydra::Datastream::CommonModsIndexMethods instead")
+  end
+
+  # Extracts the first and last names of persons and creates Solr::Field objects with for person_full_name_facet
+  #
+  # == Returns:
+  # An array of Solr::Field objects
+  #
+  def extract_person_full_names
+    names = {}
+    self.find_by_terms(:person).each do |person|
+      name_parts = person.children.inject({}) do |hash,child|
+        hash[child.get_attribute(:type)] = child.text if ["family","given"].include? child.get_attribute(:type)
+        hash
+      end
+      ::Solrizer::Extractor.insert_solr_field_value(names,  "person_full_name_facet", [name_parts["family"], name_parts["given"]].join(", ") ) if name_parts.keys.sort == ["family","given"]
+      names
+    end
+    return names
+  end
+
+  # Extracts the affiliations of persons and creates Solr::Field objects for them
+  #
+  # == Returns:
+  # An array of Solr::Field objects
+  #
+  def extract_person_organizations
+    orgs = {}
+    self.find_by_terms(:person,:affiliation).each do |org| 
+      ::Solrizer::Extractor.insert_solr_field_value(orgs, "mods_organization_facet", org.text) 
+    end
+    return orgs
+  end
+end

data/lib/hydra/model_mixins/mods_object.rb

@@ -0,0 +1,19 @@
+# Include this into models to specify that the descMetadata datastream contains basic MODS metadata
+# 
+# Explicitly declares:
+#   descMetadata datastream using Hydra::Datastream::ModsArticle Terminology
+#
+# will move to lib/hydra/model/mods_object_behavior in release 5.x
+module Hydra::ModelMixins
+  module ModsObject
+  
+    def self.included(klazz)
+      # Uses the Hydra MODS Basic profile for tracking descriptive metadata
+      klazz.has_metadata :name => "descMetadata", :type => Hydra::Datastream::ModsArticle
+    
+      # Ensure that objects assert the modsObject cModel
+      # klazz.relationships << :has_model => "info:fedora/hydra-cModel:modsObject"
+    end
+  
+  end
+end

data/lib/hydra/mods.rb

@@ -0,0 +1,14 @@
+module Hydra
+
+  # DEPRECATED STUFF
+  #module ModelMixins
+  #  autoload :CommonModsIndexMethods, 'hydra/model_mixins/mods_object'
+  #end
+  require 'hydra/model_mixins/mods_object'
+  require 'uva/mods_index_methods'
+
+  autoload :ModsArticle, 'hydra/mods_article'
+  autoload :ModsDataset, 'hydra/mods_article'
+  autoload :ModsGenericContent, 'hydra/mods_article'
+  autoload :ModsImage, 'hydra/mods_article'
+end

data/lib/hydra/mods_article.rb

@@ -0,0 +1,535 @@
+# @deprecated -- this will be removed in 5.x release
+module Hydra
+  
+  class ModsArticle < ActiveFedora::NokogiriDatastream       
+    include Hydra::Datastream::CommonModsIndexMethods
+
+    def initialize(digital_object, dsid, options={})
+      ActiveSupport::Deprecation.warn("Hydra::ModsArticle has been deprecated. Use Hydra::Datastream::ModsArticle instead")
+      super
+
+    end
+
+    set_terminology do |t|
+      t.root(:path=>"mods", :xmlns=>"http://www.loc.gov/mods/v3", :schema=>"http://www.loc.gov/standards/mods/v3/mods-3-2.xsd")
+
+
+      t.title_info(:path=>"titleInfo") {
+        t.main_title(:index_as=>[:facetable],:path=>"title", :label=>"title")
+        t.language(:index_as=>[:facetable],:path=>{:attribute=>"lang"})
+      } 
+      t.language{
+        t.lang_code(:index_as=>[:facetable], :path=>"languageTerm", :attributes=>{:type=>"code"})
+      }
+      t.abstract   
+      t.subject {
+        t.topic(:index_as=>[:facetable])
+      }      
+      t.topic_tag(:proxy=>[:subject, :topic])    
+      # t.topic_tag(:index_as=>[:facetable],:path=>"subject", :default_content_path=>"topic")
+      # This is a mods:name.  The underscore is purely to avoid namespace conflicts.
+      t.name_ {
+        # this is a namepart
+        t.namePart(:type=>:string, :label=>"generic name")
+        # affiliations are great
+        t.affiliation
+        t.institution(:path=>"affiliation", :index_as=>[:facetable], :label=>"organization")
+        t.displayForm
+        t.role(:ref=>[:role])
+        t.description(:index_as=>[:facetable])
+        t.date(:path=>"namePart", :attributes=>{:type=>"date"})
+        t.last_name(:path=>"namePart", :attributes=>{:type=>"family"})
+        t.first_name(:path=>"namePart", :attributes=>{:type=>"given"}, :label=>"first name")
+        t.terms_of_address(:path=>"namePart", :attributes=>{:type=>"termsOfAddress"})
+        t.computing_id
+      }
+      # lookup :person, :first_name        
+      t.person(:ref=>:name, :attributes=>{:type=>"personal"}, :index_as=>[:facetable])
+      t.department(:proxy=>[:person,:description],:index_as=>[:facetable])
+      t.organization(:ref=>:name, :attributes=>{:type=>"corporate"}, :index_as=>[:facetable])
+      t.conference(:ref=>:name, :attributes=>{:type=>"conference"}, :index_as=>[:facetable])
+      t.role {
+        t.text(:path=>"roleTerm",:attributes=>{:type=>"text"})
+        t.code(:path=>"roleTerm",:attributes=>{:type=>"code"})
+      }
+      t.journal(:path=>'relatedItem', :attributes=>{:type=>"host"}) {
+        t.title_info(:index_as=>[:facetable],:ref=>[:title_info])
+        t.origin_info(:path=>"originInfo") {
+          t.publisher
+          t.date_issued(:path=>"dateIssued")
+          t.issuance(:index_as=>[:facetable])
+        }
+        t.issn(:path=>"identifier", :attributes=>{:type=>"issn"})
+        t.issue(:path=>"part") {
+          t.volume(:path=>"detail", :attributes=>{:type=>"volume"}, :default_content_path=>"number")
+          t.level(:path=>"detail", :attributes=>{:type=>"number"}, :default_content_path=>"number")
+          t.extent
+          t.pages(:path=>"extent", :attributes=>{:unit=>"pages"}) {
+            t.start
+            t.end
+          }
+          t.start_page(:proxy=>[:pages, :start])
+          t.end_page(:proxy=>[:pages, :end])
+          t.publication_date(:path=>"date")
+        }
+      }
+      t.note
+      t.location(:path=>"location") {
+        t.url(:path=>"url")
+      }
+      t.publication_url(:proxy=>[:location,:url])
+      t.peer_reviewed(:proxy=>[:journal,:origin_info,:issuance], :index_as=>[:facetable])
+      t.title(:proxy=>[:mods,:title_info, :main_title])
+    end
+    
+    # Generates an empty Mods Article (used when you call ModsArticle.new without passing in existing xml)
+    def self.xml_template
+      builder = Nokogiri::XML::Builder.new do |xml|
+        xml.mods(:version=>"3.3", "xmlns:xlink"=>"http://www.w3.org/1999/xlink",
+           "xmlns:xsi"=>"http://www.w3.org/2001/XMLSchema-instance",
+           "xmlns"=>"http://www.loc.gov/mods/v3",
+           "xsi:schemaLocation"=>"http://www.loc.gov/mods/v3 http://www.loc.gov/standards/mods/v3/mods-3-3.xsd") {
+             xml.titleInfo(:lang=>"") {
+               xml.title
+             }
+             xml.name(:type=>"personal") {
+               xml.namePart(:type=>"given")
+               xml.namePart(:type=>"family")
+               xml.affiliation
+               xml.computing_id
+               xml.description
+               xml.role {
+                 xml.roleTerm("Author", :authority=>"marcrelator", :type=>"text")
+               }
+             }
+             xml.typeOfResource
+             xml.genre(:authority=>"marcgt")
+             xml.language {
+               xml.languageTerm(:authority=>"iso639-2b", :type=>"code")
+             }
+             xml.abstract
+             xml.subject {
+               xml.topic
+             }
+             xml.relatedItem(:type=>"host") {
+               xml.titleInfo {
+                 xml.title
+               }
+               xml.identifier(:type=>"issn")
+               xml.originInfo {
+                 xml.publisher
+                 xml.dateIssued
+                 xml.issuance
+               }
+               xml.part {
+                 xml.detail(:type=>"volume") {
+                   xml.number
+                 }
+                 xml.detail(:type=>"number") {
+                   xml.number
+                 }
+                 xml.extent(:unit=>"pages") {
+                   xml.start
+                   xml.end
+                 }
+                 xml.date
+               }
+             }
+             xml.location {
+               xml.url
+             }
+        }
+      end
+      return builder.doc
+    end
+    
+    # Generates a new Person node
+    def self.person_template
+      builder = Nokogiri::XML::Builder.new do |xml|
+        xml.name(:type=>"personal") {
+          xml.namePart(:type=>"family")
+          xml.namePart(:type=>"given")
+          xml.affiliation
+          xml.computing_id
+          xml.description
+          xml.role {
+            xml.roleTerm("Author", :type=>"text")
+          }
+        }
+      end
+      return builder.doc.root
+    end
+
+    def self.full_name_template
+      builder = Nokogiri::XML::Builder.new do |xml|
+        xml.full_name(:type => "personal")
+      end
+      return builder.doc.root
+    end
+
+    # Generates a new Organization node
+    # Uses mods:name[@type="corporate"]
+    def self.organization_template
+      builder = Nokogiri::XML::Builder.new do |xml|
+        xml.name(:type=>"corporate") {
+          xml.namePart
+          xml.role {
+            xml.roleTerm(:authority=>"marcrelator", :type=>"text")
+          }                          
+        }
+      end
+      return builder.doc.root
+    end
+    
+    # Generates a new Conference node
+    def self.conference_template
+      builder = Nokogiri::XML::Builder.new do |xml|
+        xml.name(:type=>"conference") {
+          xml.namePart
+          xml.role {
+            xml.roleTerm(:authority=>"marcrelator", :type=>"text")
+          }                          
+        }
+      end
+      return builder.doc.root
+    end
+    
+    # Inserts a new contributor (mods:name) into the mods document
+    # creates contributors of type :person, :organization, or :conference
+    def insert_contributor(type, opts={})
+      case type.to_sym 
+      when :person
+        node = Hydra::ModsArticle.person_template
+        nodeset = self.find_by_terms(:person)
+      when :organization
+        node = Hydra::ModsArticle.organization_template
+        nodeset = self.find_by_terms(:organization)
+      when :conference
+        node = Hydra::ModsArticle.conference_template
+        nodeset = self.find_by_terms(:conference)
+      else
+        ActiveFedora.logger.warn("#{type} is not a valid argument for Hydra::ModsArticle.insert_contributor")
+        node = nil
+        index = nil
+      end
+      
+      unless nodeset.nil?
+        if nodeset.empty?
+          self.ng_xml.root.add_child(node)
+          index = 0
+        else
+          nodeset.after(node)
+          index = nodeset.length
+        end
+        self.dirty = true
+      end
+      
+      return node, index
+    end
+    
+    # Remove the contributor entry identified by @contributor_type and @index
+    def remove_contributor(contributor_type, index)
+      contributor = self.find_by_terms( {contributor_type.to_sym => index.to_i} ).first
+      unless contributor.nil?
+        contributor.remove
+        self.dirty = true
+      end
+    end
+    
+    def self.common_relator_terms
+       {"aut" => "Author",
+        "clb" => "Collaborator",
+        "com" => "Compiler",
+        "ctb" => "Contributor",
+        "cre" => "Creator",
+        "edt" => "Editor",
+        "ill" => "Illustrator",
+        "oth" => "Other",
+        "trl" => "Translator",
+        }
+    end
+    
+    def self.person_relator_terms
+      {"aut" => "Author",
+       "clb" => "Collaborator",
+       "com" => "Compiler",
+       "cre" => "Creator",
+       "ctb" => "Contributor",
+       "edt" => "Editor",
+       "ill" => "Illustrator",
+       "res" => "Researcher",
+       "rth" => "Research team head",
+       "rtm" => "Research team member",
+       "trl" => "Translator"
+       }
+    end
+    
+    def self.conference_relator_terms
+      {
+        "hst" => "Host"
+      }
+    end
+    
+    def self.organization_relator_terms
+      {
+        "fnd" => "Funder",
+        "hst" => "Host"
+      }
+    end
+    
+    def self.dc_relator_terms
+       {"acp" => "Art copyist",
+        "act" => "Actor",
+        "adp" => "Adapter",
+        "aft" => "Author of afterword, colophon, etc.",
+        "anl" => "Analyst",
+        "anm" => "Animator",
+        "ann" => "Annotator",
+        "ant" => "Bibliographic antecedent",
+        "app" => "Applicant",
+        "aqt" => "Author in quotations or text abstracts",
+        "arc" => "Architect",
+        "ard" => "Artistic director ",
+        "arr" => "Arranger",
+        "art" => "Artist",
+        "asg" => "Assignee",
+        "asn" => "Associated name",
+        "att" => "Attributed name",
+        "auc" => "Auctioneer",
+        "aud" => "Author of dialog",
+        "aui" => "Author of introduction",
+        "aus" => "Author of screenplay",
+        "aut" => "Author",
+        "bdd" => "Binding designer",
+        "bjd" => "Bookjacket designer",
+        "bkd" => "Book designer",
+        "bkp" => "Book producer",
+        "bnd" => "Binder",
+        "bpd" => "Bookplate designer",
+        "bsl" => "Bookseller",
+        "ccp" => "Conceptor",
+        "chr" => "Choreographer",
+        "clb" => "Collaborator",
+        "cli" => "Client",
+        "cll" => "Calligrapher",
+        "clt" => "Collotyper",
+        "cmm" => "Commentator",
+        "cmp" => "Composer",
+        "cmt" => "Compositor",
+        "cng" => "Cinematographer",
+        "cnd" => "Conductor",
+        "cns" => "Censor",
+        "coe" => "Contestant -appellee",
+        "col" => "Collector",
+        "com" => "Compiler",
+        "cos" => "Contestant",
+        "cot" => "Contestant -appellant",
+        "cov" => "Cover designer",
+        "cpc" => "Copyright claimant",
+        "cpe" => "Complainant-appellee",
+        "cph" => "Copyright holder",
+        "cpl" => "Complainant",
+        "cpt" => "Complainant-appellant",
+        "cre" => "Creator",
+        "crp" => "Correspondent",
+        "crr" => "Corrector",
+        "csl" => "Consultant",
+        "csp" => "Consultant to a project",
+        "cst" => "Costume designer",
+        "ctb" => "Contributor",
+        "cte" => "Contestee-appellee",
+        "ctg" => "Cartographer",
+        "ctr" => "Contractor",
+        "cts" => "Contestee",
+        "ctt" => "Contestee-appellant",
+        "cur" => "Curator",
+        "cwt" => "Commentator for written text",
+        "dfd" => "Defendant",
+        "dfe" => "Defendant-appellee",
+        "dft" => "Defendant-appellant",
+        "dgg" => "Degree grantor",
+        "dis" => "Dissertant",
+        "dln" => "Delineator",
+        "dnc" => "Dancer",
+        "dnr" => "Donor",
+        "dpc" => "Depicted",
+        "dpt" => "Depositor",
+        "drm" => "Draftsman",
+        "drt" => "Director",
+        "dsr" => "Designer",
+        "dst" => "Distributor",
+        "dtc" => "Data contributor ",
+        "dte" => "Dedicatee",
+        "dtm" => "Data manager ",
+        "dto" => "Dedicator",
+        "dub" => "Dubious author",
+        "edt" => "Editor",
+        "egr" => "Engraver",
+        "elg" => "Electrician ",
+        "elt" => "Electrotyper",
+        "eng" => "Engineer",
+        "etr" => "Etcher",
+        "exp" => "Expert",
+        "fac" => "Facsimilist",
+        "fld" => "Field director ",
+        "flm" => "Film editor",
+        "fmo" => "Former owner",
+        "fpy" => "First party",
+        "fnd" => "Funder",
+        "frg" => "Forger",
+        "gis" => "Geographic information specialist ",
+        "grt" => "Graphic technician",
+        "hnr" => "Honoree",
+        "hst" => "Host",
+        "ill" => "Illustrator",
+        "ilu" => "Illuminator",
+        "ins" => "Inscriber",
+        "inv" => "Inventor",
+        "itr" => "Instrumentalist",
+        "ive" => "Interviewee",
+        "ivr" => "Interviewer",
+        "lbr" => "Laboratory ",
+        "lbt" => "Librettist",
+        "ldr" => "Laboratory director ",
+        "led" => "Lead",
+        "lee" => "Libelee-appellee",
+        "lel" => "Libelee",
+        "len" => "Lender",
+        "let" => "Libelee-appellant",
+        "lgd" => "Lighting designer",
+        "lie" => "Libelant-appellee",
+        "lil" => "Libelant",
+        "lit" => "Libelant-appellant",
+        "lsa" => "Landscape architect",
+        "lse" => "Licensee",
+        "lso" => "Licensor",
+        "ltg" => "Lithographer",
+        "lyr" => "Lyricist",
+        "mcp" => "Music copyist",
+        "mfr" => "Manufacturer",
+        "mdc" => "Metadata contact",
+        "mod" => "Moderator",
+        "mon" => "Monitor",
+        "mrk" => "Markup editor",
+        "msd" => "Musical director",
+        "mte" => "Metal-engraver",
+        "mus" => "Musician",
+        "nrt" => "Narrator",
+        "opn" => "Opponent",
+        "org" => "Originator",
+        "orm" => "Organizer of meeting",
+        "oth" => "Other",
+        "own" => "Owner",
+        "pat" => "Patron",
+        "pbd" => "Publishing director",
+        "pbl" => "Publisher",
+        "pdr" => "Project director",
+        "pfr" => "Proofreader",
+        "pht" => "Photographer",
+        "plt" => "Platemaker",
+        "pma" => "Permitting agency",
+        "pmn" => "Production manager",
+        "pop" => "Printer of plates",
+        "ppm" => "Papermaker",
+        "ppt" => "Puppeteer",
+        "prc" => "Process contact",
+        "prd" => "Production personnel",
+        "prf" => "Performer",
+        "prg" => "Programmer",
+        "prm" => "Printmaker",
+        "pro" => "Producer",
+        "prt" => "Printer",
+        "pta" => "Patent applicant",
+        "pte" => "Plaintiff -appellee",
+        "ptf" => "Plaintiff",
+        "pth" => "Patent holder",
+        "ptt" => "Plaintiff-appellant",
+        "rbr" => "Rubricator",
+        "rce" => "Recording engineer",
+        "rcp" => "Recipient",
+        "red" => "Redactor",
+        "ren" => "Renderer",
+        "res" => "Researcher",
+        "rev" => "Reviewer",
+        "rps" => "Repository",
+        "rpt" => "Reporter",
+        "rpy" => "Responsible party",
+        "rse" => "Respondent-appellee",
+        "rsg" => "Restager",
+        "rsp" => "Respondent",
+        "rst" => "Respondent-appellant",
+        "rth" => "Research team head",
+        "rtm" => "Research team member",
+        "sad" => "Scientific advisor",
+        "sce" => "Scenarist",
+        "scl" => "Sculptor",
+        "scr" => "Scribe",
+        "sds" => "Sound designer",
+        "sec" => "Secretary",
+        "sgn" => "Signer",
+        "sht" => "Supporting host",
+        "sng" => "Singer",
+        "spk" => "Speaker",
+        "spn" => "Sponsor",
+        "spy" => "Second party",
+        "srv" => "Surveyor",
+        "std" => "Set designer",
+        "stl" => "Storyteller",
+        "stm" => "Stage manager",
+        "stn" => "Standards body",
+        "str" => "Stereotyper",
+        "tcd" => "Technical director",
+        "tch" => "Teacher",
+        "ths" => "Thesis advisor",
+        "trc" => "Transcriber",
+        "trl" => "Translator",
+        "tyd" => "Type designer",
+        "tyg" => "Typographer",
+        "vdg" => "Videographer",
+        "voc" => "Vocalist",
+        "wam" => "Writer of accompanying material",
+        "wdc" => "Woodcutter",
+        "wde" => "Wood -engraver",
+        "wit" => "Witness"}
+      end
+    
+      def self.valid_child_types
+        ["data", "supporting file", "profile", "lorem ipsum", "dolor"]
+      end
+
+    def to_solr(solr_doc=Hash.new)
+      super(solr_doc)
+      
+      extract_person_full_names.each_pair {|n,v| ::Solrizer::Extractor.insert_solr_field_value(solr_doc, n, v) }
+      extract_person_organizations.each_pair {|n,v| ::Solrizer::Extractor.insert_solr_field_value(solr_doc, n, v) }
+      extract_person_full_names_and_computing_ids.each_pair {|n,v| ::Solrizer::Extractor.insert_solr_field_value(solr_doc, n, v) }
+      
+      ::Solrizer::Extractor.insert_solr_field_value(solr_doc, "object_type_facet", "Article")
+      ::Solrizer::Extractor.insert_solr_field_value(solr_doc, "mods_journal_title_info_facet", "Unknown") if solr_doc["mods_journal_title_info_facet"].nil? || solr_doc["mods_journal_title_info_facet"].blank?
+
+      solr_doc
+    end
+
+    # extracts the last_name##full_name##computing_id to be used by home view
+    def extract_person_full_names_and_computing_ids
+      names = {}
+      self.find_by_terms(:person).each do |person|
+        name_parts = person.children.inject({}) do |hash,child|
+          hash[child.get_attribute(:type)] = child.text if ["family","given"].include? child.get_attribute(:type)
+          hash["computing_id"] = child.text if child.name == 'computing_id'
+          hash
+        end
+        if name_parts.length == 3 and person.search(:roleTerm).children.text.include?("Author")
+          if name_parts["family"].blank? && name_parts["given"].blank? && name_parts["computing_id"].blank?
+            value = "Unknown Author"
+          else
+            value = "#{name_parts["family"]}, #{name_parts["given"]} (#{name_parts["computing_id"]})"
+          end
+          ::Solrizer::Extractor.insert_solr_field_value(names, "person_full_name_cid_facet", value) if name_parts.length == 3        
+        end      
+      end
+      names
+    end
+
+  end
+end