sufia-models 5.0.0 → 6.0.0.beta1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 01d0bd76b4da4b019f47e04155e07d8ceff430db
-  data.tar.gz: d946f287b93c623fe68eb895aa61af23bdb234db
+  metadata.gz: 4314a9ee6401c790ffce36c91f621c9c3fa9ec4a
+  data.tar.gz: d8cb9e5007c3d8130ceaf6e59a279f88a03ec622
 SHA512:
-  metadata.gz: fabe5b9b8676c6bc149fe07365a5d467af40a227160b94a58d89e274fea06ad1c477dc708f49f7e3b081af3b48a8b88b7d9683f83120b9cacc88ce4d0671ec33
-  data.tar.gz: 0c4ea2d162c9a9f26682931a5f3463ae26dca92ac5940987285a3e359a77aa1a760ffd8e62ba503f90f09fa8d21c418e7f9c5fe13f27853a9aab578223241dfa
+  metadata.gz: b1e0ab2d2b46c88005855ae5d609a9d1f4a474a4c5cfa912ca0f59d29f72f486fbc6d062f04c0949c1287d36a28d17e100747d31fab6eb6b0111be93c2d43a63
+  data.tar.gz: b174426b321d1921c85264409f27c7f6a5b984a690061a5e81e32755bc309b2f0c5e37abbd4348fe1d8eed293def5fca38e7482eb0aee7207f61cef022be842e

data/app/actors/sufia/generic_file/actor.rb

@@ -20,7 +20,7 @@ module Sufia::GenericFile
       generic_file.creator = [user.name]
 
       if batch_id
-        generic_file.batch_id = Sufia::Noid.namespaceize(batch_id)
+        generic_file.batch_id = batch_id
       else
         ActiveFedora::Base.logger.warn "unable to find batch to attach to"
       end
@@ -28,8 +28,7 @@ module Sufia::GenericFile
     end
 
     def create_content(file, file_name, dsid)
-      fname = generic_file.label.blank? ? file_name.truncate(255) : generic_file.label
-      generic_file.add_file(file, dsid, fname)
+      generic_file.add_file(file, dsid, file_name.truncate(255))
       save_characterize_and_record_committer do
         if Sufia.config.respond_to?(:after_create_content)
           Sufia.config.after_create_content.call(generic_file, user)
@@ -37,9 +36,9 @@ module Sufia::GenericFile
       end
     end
 
-    def revert_content(revision_id, datastream_id)
-      revision = generic_file.content.get_version(revision_id)
-      generic_file.add_file(revision.content, datastream_id, revision.label)
+    def revert_content(revision_id)
+      generic_file.content.restore_version(revision_id)
+      generic_file.content.create_version
       save_characterize_and_record_committer do
         if Sufia.config.respond_to?(:after_revert_content)
           Sufia.config.after_revert_content.call(generic_file, user, revision_id)
@@ -69,11 +68,10 @@ module Sufia::GenericFile
     end
 
     def destroy
-      pid = generic_file.pid  #Work around for https://github.com/projecthydra/active_fedora/issues/422
       generic_file.destroy
-      FeaturedWork.where(generic_file_id: pid).destroy_all
+      FeaturedWork.where(generic_file_id: generic_file.id).destroy_all
       if Sufia.config.respond_to?(:after_destroy)
-        Sufia.config.after_destroy.call(pid, user)
+        Sufia.config.after_destroy.call(generic_file.id, user)
       end
     end
 
@@ -103,7 +101,7 @@ module Sufia::GenericFile
     end
 
     def push_characterize_job
-      Sufia.queue.push(CharacterizeJob.new(@generic_file.pid))
+      Sufia.queue.push(CharacterizeJob.new(@generic_file.id))
     end
 
     class << self

data/app/jobs/active_fedora_pid_based_job.rb

@@ -10,10 +10,9 @@ class ActiveFedoraPidBasedJob
   end
 
   def object
-    @object ||= ActiveFedora::Base.find(pid, cast: true).tap do |f|
-      f.reload_on_save = true
-    end
+    @object ||= ActiveFedora::Base.find(pid)
   end
+
   alias_method :generic_file, :object
   alias_method :generic_file_id, :pid
 

data/app/jobs/audit_job.rb

@@ -6,44 +6,58 @@ class AuditJob < ActiveFedoraPidBasedJob
   PASS = 'Passing Audit Run'
   FAIL = 'Failing Audit Run'
 
-  attr_accessor :pid, :datastream_id, :version_id
+  attr_accessor :uri, :pid, :path
 
-  def initialize(pid, datastream_id, version_id)
-    super(pid)
-    self.datastream_id = datastream_id
-    self.version_id = version_id
+  # URI of the resource to audit.
+  # This URI could include the actual resource (e.g. content) and the version to audit:
+  #     http://localhost:8983/fedora/rest/test/a/b/c/abcxyz/content/fcr:versions/version1
+  # but it could also just be:
+  #     http://localhost:8983/fedora/rest/test/a/b/c/abcxyz/content
+  def initialize(id, path, uri)
+    super(uri)
+    self.pid = id
+    self.path = path
+    self.uri = uri
   end
 
   def run
-    if generic_file
-      datastream = generic_file.datastreams[datastream_id]
-      if datastream
-        version =  datastream.versions.select { |v| v.versionID == version_id}.first
-        log = run_audit(version)
-
-        # look up the user for sending the message to
-        login = generic_file.depositor
-        if login
-          user = User.find_by_user_key(login)
-          ActiveFedora::Base.logger.warn "User '#{login}' not found" unless user
-          job_user = User.audituser()
-          # send the user a message about the failing audit
-          unless (log.pass == 1)
-            message = "The audit run at #{log.created_at} for #{log.pid}:#{log.dsid}:#{log.version} was #{log.pass == 1 ? 'passing' : 'failing'}."
-            subject = (log.pass == 1 ? PASS : FAIL)
-            job_user.send_message(user, message, subject)
-          end 
-        end
+    fixity_ok = false
+    log = run_audit
+    fixity_ok = (log.pass == 1)
+    unless fixity_ok
+      # send the user a message about the failing audit
+      login = generic_file.depositor
+      user = User.find_by_user_key(login)
+      logger.warn "User '#{login}' not found" unless user
+      job_user = User.audituser()
+      file_title = generic_file.title.first
+      message = "The audit run at #{log.created_at} for #{file_title} (#{uri}) failed."
+      subject = FAIL
+      job_user.send_message(user, message, subject)
+    end
+    fixity_ok
+  end
+
+  protected
+
+    def run_audit
+      begin
+        fixity_ok = ActiveFedora::FixityService.new(uri).check
+      rescue Ldp::NotFound
+        error_msg = "resource not found"
+      end
+
+      if fixity_ok
+        passing = 1
+        ChecksumAuditLog.prune_history(pid, path)
       else
-        ActiveFedora::Base.logger.warn "No datastream for audit!!!!! pid: #{pid} dsid: #{datastream_id}"
+        logger.warn "***AUDIT*** Audit failed for #{uri} #{error_msg}"
+        passing = 0
       end
-    else
-      ActiveFedora::Base.logger.warn "No generic file for data stream audit!!!!! pid: #{pid} dsid: #{datastream_id}"
+      ChecksumAuditLog.create!(pass: passing, pid: pid, version: uri, dsid: path)
     end
-  end
 
-  private
-  def run_audit(version)
-    object.class.run_audit(version)
-  end
+    def logger
+      ActiveFedora::Base.logger
+    end
 end

data/app/jobs/batch_update_job.rb

@@ -10,7 +10,7 @@ class BatchUpdateJob
 
   def initialize(login, params)
     self.login = login
-    self.title = params[:title]
+    self.title = params[:title] || {}
     self.file_attributes = params[:generic_file]
     self.visibility = params[:visibility]
     self.batch_id = params[:id]
@@ -25,7 +25,9 @@ class BatchUpdateJob
     batch.generic_files.each do |gf|
       update_file(gf, user)
     end
-    batch.update_attributes({status:["Complete"]})
+
+    batch.update(status: ["Complete"])
+
     if denied.empty?
       send_user_success_message(user, batch) unless saved.empty?
     else
@@ -35,12 +37,12 @@ class BatchUpdateJob
 
   def update_file(gf, user)
     unless user.can? :edit, gf
-      ActiveFedora::Base.logger.error "User #{user.user_key} DENIED access to #{gf.pid}!"
+      ActiveFedora::Base.logger.error "User #{user.user_key} DENIED access to #{gf.id}!"
       denied << gf
       return
     end
-    gf.title = title[gf.pid] if title[gf.pid] rescue gf.label
-    gf.attributes=file_attributes
+    gf.title = title[gf.id] if title[gf.id]
+    gf.attributes = file_attributes
     gf.visibility= visibility
 
     save_tries = 0
@@ -48,13 +50,13 @@ class BatchUpdateJob
       gf.save!
     rescue RSolr::Error::Http => error
       save_tries += 1
-      ActiveFedora::Base.logger.warn "BatchUpdateJob caught RSOLR error on #{gf.pid}: #{error.inspect}"
+      ActiveFedora::Base.logger.warn "BatchUpdateJob caught RSOLR error on #{gf.id}: #{error.inspect}"
       # fail for good if the tries is greater than 3
       raise error if save_tries >=3
       sleep 0.01
       retry
     end #
-    Sufia.queue.push(ContentUpdateEventJob.new(gf.pid, login))
+    Sufia.queue.push(ContentUpdateEventJob.new(gf.id, login))
     saved << gf
   end
 
@@ -67,5 +69,4 @@ class BatchUpdateJob
     message = denied.count > 1 ? multiple_failure(batch.noid, denied) : single_failure(batch.noid, denied.first)
     User.batchuser.send_message(user, message, failure_subject, sanitize_text = false)
   end
-  
 end

data/app/jobs/import_url_job.rb

@@ -11,12 +11,12 @@ class ImportUrlJob < ActiveFedoraPidBasedJob
   def run
     user = User.find_by_user_key(generic_file.depositor)
 
-    Tempfile.open(self.pid) do |f|
+    Tempfile.open(pid.gsub('/', '_')) do |f|
       path = copy_remote_file(generic_file.import_url, f)
       # attach downloaded file to generic file stubbed out
       if Sufia::GenericFile::Actor.new(generic_file, user).create_content(f, path, 'content')
         # add message to user for downloaded file
-        message = "The file (#{generic_file.content.label}) was successfully imported."
+        message = "The file (#{generic_file.label}) was successfully imported."
         job_user.send_message(user, message, 'File Import')
       else
         job_user.send_message(user, generic_file.errors.full_messages.join(', '), 'File Import Error')

data/app/models/batch.rb

@@ -3,24 +3,23 @@ class Batch < ActiveFedora::Base
   include Sufia::ModelMethods
   include Sufia::Noid
 
-  has_metadata name: "descMetadata", type: BatchRdfDatastream
+  has_many :generic_files, predicate: ActiveFedora::RDF::Fcrepo::RelsExt.isPartOf
 
-  belongs_to :user, property: "creator"
-  has_many :generic_files, property: :is_part_of
+  property :creator, predicate: ::RDF::DC.creator
+  property :title, predicate: ::RDF::DC.title
+  property :status, predicate: ::RDF::DC.type
 
-  has_attributes :title, :creator, :part, :status, datastream: :descMetadata, multiple: true
-
-  def self.find_or_create(pid)
+  def self.find_or_create(id)
     begin
-      Batch.find(pid)
+      Batch.find(id)
     rescue ActiveFedora::ObjectNotFoundError
-      Batch.create({pid: pid})
+      Batch.create(id: id)
     end
   end
 
-  def to_solr(solr_doc={}, opts={})
-    solr_doc = super(solr_doc, opts)
-    solr_doc[Solrizer.solr_name('noid', Sufia::GenericFile.noid_indexer)] = noid
-    return solr_doc
+  def to_solr(solr_doc={})
+    super.tap do |solr_doc|
+      solr_doc[Solrizer.solr_name('noid', Sufia::GenericFile::Indexing.noid_indexer)] = noid
+    end
   end
 end

data/app/models/checksum_audit_log.rb

@@ -1,19 +1,21 @@
 class ChecksumAuditLog < ActiveRecord::Base
 
-  def ChecksumAuditLog.get_audit_log(version)
-    ChecksumAuditLog.find_or_create_by_pid_and_dsid_and_version(pid: version.pid,
-                                                                dsid: version.dsid,
-                                                                version: version.versionID)
+  def self.get_audit_log(id, path, version_uri)
+    ChecksumAuditLog.find_or_create_by(pid: id, dsid: path, version: version_uri)
   end
 
-  def ChecksumAuditLog.prune_history(version)
-    ## Check to see if there are previous passing logs that we can delete
-    # we want to keep the first passing event after a failure, the most current passing event, and all failures so that this table doesn't grow too large
-    # Simple way (a little naieve): if the last 2 were passing, delete the first one
-    logs = GenericFile.load_instance_from_solr(version.pid).logs(version.dsid)
-    list = logs.limit(2)
+  # Check to see if there are previous passing logs that we can delete
+  # we want to keep the first passing event after a failure, the most current passing event,
+  # and all failures so that this table doesn't grow too large
+  # Simple way (a little naieve): if the last 2 were passing, delete the first one
+  def self.prune_history(id, path)
+    list = logs_for(id, path).limit(2)
     if list.size > 1 && (list[0].pass == 1) && (list[1].pass == 1)
       list[0].destroy
     end
   end
+
+  def self.logs_for(id, path)
+    ChecksumAuditLog.where(pid: id, dsid: path).order('created_at desc, id desc')
+  end
 end

data/app/models/concerns/sufia/ability.rb

@@ -15,8 +15,8 @@ module Sufia
     end
 
     def proxy_deposit_abilities
-      can :transfer, String do |pid|
-        get_depositor_from_pid(pid) == current_user.user_key
+      can :transfer, String do |id|
+        depositor_for_document(id) == current_user.user_key
       end
       can :create, ProxyDepositRequest if user_groups.include? 'registered'
       can :accept, ProxyDepositRequest, receiving_user_id: current_user.id, status: 'pending'
@@ -47,10 +47,8 @@ module Sufia
 
     private
 
-    def get_depositor_from_pid(pid)
-      ::GenericFile.load_instance_from_solr(pid).depositor
-    rescue
-      nil
+    def depositor_for_document(document_id)
+      ::GenericFile.load_instance_from_solr(document_id).depositor
     end
   end
 end

data/app/models/concerns/sufia/collection.rb

@@ -11,7 +11,6 @@ module Sufia
       before_save :update_permissions
       validates :title, presence: true
 
-      has_metadata "properties", type: PropertiesDatastream
     end
 
     def terms_for_display
@@ -35,10 +34,10 @@ module Sufia
       noid
     end
 
-    def to_solr(solr_doc={}, opts={})
-      super(solr_doc, opts)
-      solr_doc[Solrizer.solr_name("noid", Sufia::GenericFile.noid_indexer)] = noid
-      return solr_doc
+    def to_solr(solr_doc={})
+      super.tap do |solr_doc|
+        solr_doc[Solrizer.solr_name("noid", Sufia::GenericFile::Indexing.noid_indexer)] = noid
+      end
     end
 
     def update_permissions

data/app/models/concerns/sufia/generic_file/batches.rb

@@ -0,0 +1,29 @@
+module Sufia
+  module GenericFile
+    module Batches
+      extend ActiveSupport::Concern
+      included do
+        belongs_to :batch, predicate: ActiveFedora::RDF::Fcrepo::RelsExt.isPartOf
+      end
+
+      # Get the files with a sibling relationship (belongs_to :batch)
+      # The batch id is minted when visiting the upload screen and attached
+      # to each file when it is done uploading.  The Batch object is not created
+      # until all objects are done uploading and the user is redirected to
+      # BatchController#edit.  Therefore, we must handle the case where
+      # self.batch_id is set but self.batch returns nil.
+      def related_files
+        return [] if batch.nil?
+        batch.generic_files.reject { |sibling| sibling.id == id }
+      end
+
+      # Is this file in the middle of being processed by a batch?
+      def processing?
+         return false if self.batch.blank? || !self.batch.methods.include?(:status)
+         (!self.batch.status.empty?) && (self.batch.status.count == 1) && (self.batch.status[0] == "processing")
+      end
+
+    end
+  end
+end
+

data/app/models/concerns/sufia/generic_file/characterization.rb

@@ -3,7 +3,7 @@ module Sufia
     module Characterization
       extend ActiveSupport::Concern
       included do
-        has_metadata "characterization", type: FitsDatastream
+        contains "characterization", class_name: 'FitsDatastream'
         has_attributes :mime_type, datastream: :characterization, multiple: false
         has_attributes :format_label, :file_size, :last_modified,
                                         :filename, :original_checksum, :rights_basis,
@@ -45,11 +45,11 @@ module Sufia
         metadata = content.extract_metadata
         characterization.ng_xml = metadata if metadata.present?
         append_metadata
-        self.filename = [self.label]
+        self.filename = [content.original_name]
         save
       end
 
-      # Populate descMetadata with fields from FITS (e.g. Author from pdfs)
+      # Populate GenericFile's properties with fields from FITS (e.g. Author from pdfs)
       def append_metadata
         terms = self.characterization_terms
         Sufia.config.fits_to_desc_mapping.each_pair do |k, v|

data/app/models/concerns/sufia/generic_file/content.rb

@@ -0,0 +1,13 @@
+module Sufia
+  module GenericFile
+    module Content
+      extend ActiveSupport::Concern
+
+      included do
+        contains "content", class_name: 'FileContentDatastream'
+        contains "thumbnail"
+      end
+
+    end
+  end
+end

data/app/models/concerns/sufia/generic_file/derivatives.rb

@@ -9,15 +9,15 @@ module Sufia
         makes_derivatives do |obj|
           case obj.mime_type
           when *pdf_mime_types
-            obj.transform_datastream :content, { thumbnail: { format: 'jpg', size: '338x493', datastream: 'thumbnail' } }
+            obj.transform_file :content, { thumbnail: { format: 'jpg', size: '338x493', datastream: 'thumbnail' } }
           when *office_document_mime_types
-            obj.transform_datastream :content, { thumbnail: { format: 'jpg', size: '200x150>', datastream: 'thumbnail' } }, processor: :document
+            obj.transform_file :content, { thumbnail: { format: 'jpg', size: '200x150>', datastream: 'thumbnail' } }, processor: :document
           when *audio_mime_types
-            obj.transform_datastream :content, { mp3: { format: 'mp3', datastream: 'mp3' }, ogg: { format: 'ogg', datastream: 'ogg' } }, processor: :audio
+            obj.transform_file :content, { mp3: { format: 'mp3', datastream: 'mp3' }, ogg: { format: 'ogg', datastream: 'ogg' } }, processor: :audio
           when *video_mime_types
-            obj.transform_datastream :content, { webm: { format: 'webm', datastream: 'webm' }, mp4: { format: 'mp4', datastream: 'mp4' }, thumbnail: { format: 'jpg', datastream: 'thumbnail' } }, processor: :video
+            obj.transform_file :content, { webm: { format: 'webm', datastream: 'webm' }, mp4: { format: 'mp4', datastream: 'mp4' }, thumbnail: { format: 'jpg', datastream: 'thumbnail' } }, processor: :video
           when *image_mime_types
-            obj.transform_datastream :content, { thumbnail: { format: 'jpg', size: '200x150>', datastream: 'thumbnail' } }
+            obj.transform_file :content, { thumbnail: { format: 'jpg', size: '200x150>', datastream: 'thumbnail' } }
           end
         end
       end

data/app/models/concerns/sufia/generic_file/export.rb

@@ -41,6 +41,10 @@ module Sufia
         return text.join("\n")
       end
 
+      def persistent_url
+        "#{Sufia.config.persistent_hostpath}#{noid}"
+      end
+
       # MIME type: 'application/x-openurl-ctx-kev'
       def export_as_openurl_ctx_kev
         export_text = []

data/app/models/concerns/sufia/generic_file/full_text_indexing.rb

@@ -4,7 +4,7 @@ module Sufia
       extend ActiveSupport::Concern
 
       included do
-        has_file_datastream 'full_text', versionable: false
+        contains 'full_text'
       end
 
       def append_metadata
@@ -27,7 +27,7 @@ module Sufia
           extracted_text = JSON.parse(resp.body)[''].rstrip
           full_text.content = extracted_text if extracted_text.present?
         rescue => e
-          logger.error("Error extracting content from #{self.pid}: #{e.inspect}")
+          logger.error("Error extracting content from #{self.id}: #{e.inspect}")
         end
     end
   end

data/app/models/concerns/sufia/generic_file/indexing.rb

@@ -0,0 +1,23 @@
+module Sufia
+  module GenericFile
+    module Indexing
+      extend ActiveSupport::Concern
+
+      # Unstemmed, searchable, stored
+      def self.noid_indexer
+        @noid_indexer ||= Solrizer::Descriptor.new(:text, :indexed, :stored)
+      end
+
+      def to_solr(solr_doc={})
+        super.tap do |solr_doc|
+          solr_doc[Solrizer.solr_name('label')] = label
+          solr_doc[Solrizer.solr_name('noid', Sufia::GenericFile::Indexing.noid_indexer)] = noid
+          solr_doc[Solrizer.solr_name('file_format')] = file_format
+          solr_doc[Solrizer.solr_name('file_format', :facetable)] = file_format
+          solr_doc['all_text_timv'] = full_text.content
+          solr_doc = index_collection_ids(solr_doc)
+        end
+      end
+    end
+  end
+end

data/app/models/concerns/sufia/generic_file/metadata.rb

@@ -4,17 +4,86 @@ module Sufia
       extend ActiveSupport::Concern
 
       included do
-        has_metadata "descMetadata", type: GenericFileRdfDatastream
-        has_metadata "properties", type: PropertiesDatastream
-        has_file_datastream "content", type: FileContentDatastream
-        has_file_datastream "thumbnail"
-
-        has_attributes :relative_path, :depositor, :import_url, datastream: :properties, multiple: false
-        has_attributes :date_uploaded, :date_modified, datastream: :descMetadata, multiple: false
-        has_attributes :related_url, :based_near, :part_of, :creator,
-                                    :contributor, :title, :tag, :description, :rights,
-                                    :publisher, :date_created, :subject,
-                                    :resource_type, :identifier, :language, datastream: :descMetadata, multiple: true
+
+        property :label, predicate: ::RDF::DC.title, multiple: false
+
+        property :depositor, predicate: ::RDF::URI.new("http://id.loc.gov/vocabulary/relators/dpt"), multiple: false do |index|
+          index.as :symbol, :stored_searchable
+        end
+
+        property :relative_path, predicate: ::RDF::URI.new('http://scholarsphere.psu.edu/ns#relativePath'), multiple: false
+
+        property :import_url, predicate: ::RDF::URI.new('http://scholarsphere.psu.edu/ns#importUrl'), multiple: false do |index|
+          index.as :symbol
+        end
+
+        property :part_of, predicate: ::RDF::DC.isPartOf
+        property :resource_type, predicate: ::RDF::DC.type do |index|
+          index.as :stored_searchable, :facetable
+        end
+        property :title, predicate: ::RDF::DC.title do |index|
+          index.as :stored_searchable, :facetable
+        end
+        property :creator, predicate: ::RDF::DC.creator do |index|
+          index.as :stored_searchable, :facetable
+        end
+        property :contributor, predicate: ::RDF::DC.contributor do |index|
+          index.as :stored_searchable, :facetable
+        end
+        property :description, predicate: ::RDF::DC.description do |index|
+          index.type :text
+          index.as :stored_searchable
+        end
+        property :tag, predicate: ::RDF::DC.relation do |index|
+          index.as :stored_searchable, :facetable
+        end
+        property :rights, predicate: ::RDF::DC.rights do |index|
+          index.as :stored_searchable
+        end
+        property :publisher, predicate: ::RDF::DC.publisher do |index|
+          index.as :stored_searchable, :facetable
+        end
+        property :date_created, predicate: ::RDF::DC.created do |index|
+          index.as :stored_searchable
+        end
+        property :date_uploaded, predicate: ::RDF::DC.dateSubmitted, multiple: false do |index|
+          index.type :date
+          index.as :stored_sortable
+        end
+        property :date_modified, predicate: ::RDF::DC.modified, multiple: false do |index|
+          index.type :date
+          index.as :stored_sortable
+        end
+        property :subject, predicate: ::RDF::DC.subject do |index|
+          index.as :stored_searchable, :facetable
+        end
+        property :language, predicate: ::RDF::DC.language do |index|
+          index.as :stored_searchable, :facetable
+        end
+        property :identifier, predicate: ::RDF::DC.identifier do |index|
+          index.as :stored_searchable
+        end
+        property :based_near, predicate: ::RDF::FOAF.based_near do |index|
+          index.as :stored_searchable, :facetable
+        end
+        property :related_url, predicate: ::RDF::RDFS.seeAlso do |index|
+          index.as :stored_searchable
+        end
+        property :bibliographic_citation, predicate: ::RDF::DC.bibliographicCitation do |index|
+          index.as :stored_searchable
+        end
+        property :source, predicate: ::RDF::DC.source do |index|
+          index.as :stored_searchable
+        end
+
+        # TODO: Move this somewhere more appropriate
+        begin
+          LocalAuthority.register_vocabulary(self, "subject", "lc_subjects")
+          LocalAuthority.register_vocabulary(self, "language", "lexvo_languages")
+          LocalAuthority.register_vocabulary(self, "tag", "lc_genres")
+        rescue
+          puts "tables for vocabularies missing"
+        end
       end
 
       # Add a schema.org itemtype

data/app/models/concerns/sufia/generic_file/proxy_deposit.rb

@@ -4,18 +4,27 @@ module Sufia
       extend ActiveSupport::Concern
 
       included do
-        has_attributes :proxy_depositor, :on_behalf_of, datastream: :properties, multiple: false
+        property :proxy_depositor, predicate: ::RDF::URI.new('http://scholarsphere.psu.edu/ns#proxyDepositor'), multiple: false do |index|
+          index.as :symbol
+        end
+
+        # This value is set when a user indicates they are depositing this for someone else
+        property :on_behalf_of, predicate: ::RDF::URI.new('http://scholarsphere.psu.edu/ns#onBehalfOf'), multiple: false do |index|
+          index.as :symbol
+        end
+
         after_create :create_transfer_request
       end
 
+
       def create_transfer_request
-        Sufia.queue.push(ContentDepositorChangeEventJob.new(pid, on_behalf_of)) if on_behalf_of.present?
+        Sufia.queue.push(ContentDepositorChangeEventJob.new(id, on_behalf_of)) if on_behalf_of.present?
       end
 
       def request_transfer_to(target)
         raise ArgumentError, "Must provide a target" unless target
         deposit_user = ::User.find_by_user_key(depositor)
-        ProxyDepositRequest.create!(pid: pid, receiving_user: target, sending_user: deposit_user)
+        ProxyDepositRequest.create!(pid: id, receiving_user: target, sending_user: deposit_user)
       end
     end
   end