enju_biblio 0.1.0.pre42 → 0.1.0.pre43

data/app/models/{patron_import_file.rb → agent_import_file.rb}

@@ -1,27 +1,27 @@
-class PatronImportFile < ActiveRecord::Base
-  attr_accessible :patron_import, :edit_mode
+class AgentImportFile < ActiveRecord::Base
+  attr_accessible :agent_import, :edit_mode
   include ImportFile
-  default_scope :order => 'patron_import_files.id DESC'
+  default_scope :order => 'agent_import_files.id DESC'
   scope :not_imported, where(:state => 'pending')
   scope :stucked, where('created_at < ? AND state = ?', 1.hour.ago, 'pending')
 
   if Setting.uploaded_file.storage == :s3
-    has_attached_file :patron_import, :storage => :s3, :s3_credentials => "#{Rails.root.to_s}/config/s3.yml",
+    has_attached_file :agent_import, :storage => :s3, :s3_credentials => "#{Rails.root.to_s}/config/s3.yml",
       :s3_permissions => :private
   else
-    has_attached_file :patron_import,
+    has_attached_file :agent_import,
       :path => ":rails_root/private/system/:class/:attachment/:id_partition/:style/:filename"
   end
-  validates_attachment_content_type :patron_import, :content_type => [
+  validates_attachment_content_type :agent_import, :content_type => [
     'text/csv',
     'text/plain',
     'text/tab-separated-values',
     'application/octet-stream',
     'application/vnd.ms-excel'
   ]
-  validates_attachment_presence :patron_import
+  validates_attachment_presence :agent_import
   belongs_to :user, :validate => true
-  has_many :patron_import_results
+  has_many :agent_import_results
 
   state_machine :initial => :pending do
     event :sm_start do
@@ -36,12 +36,12 @@ class PatronImportFile < ActiveRecord::Base
       transition :started => :failed
     end
 
-    before_transition any => :started do |patron_import_file|
-      patron_import_file.executed_at = Time.zone.now
+    before_transition any => :started do |agent_import_file|
+      agent_import_file.executed_at = Time.zone.now
     end
 
-    before_transition any => :completed do |patron_import_file|
-      patron_import_file.error_message = nil
+    before_transition any => :completed do |agent_import_file|
+      agent_import_file.error_message = nil
     end
   end
 
@@ -61,7 +61,7 @@ class PatronImportFile < ActiveRecord::Base
 
   def import
     self.reload
-    num = {:patron_imported => 0, :user_imported => 0, :failed => 0}
+    num = {:agent_imported => 0, :user_imported => 0, :failed => 0}
     row_num = 2
     rows = open_import_file
     field = rows.first
@@ -72,14 +72,14 @@ class PatronImportFile < ActiveRecord::Base
 
     rows.each do |row|
       next if row['dummy'].to_s.strip.present?
-      import_result = PatronImportResult.create!(:patron_import_file_id => self.id, :body => row.fields.join("\t"))
+      import_result = AgentImportResult.create!(:agent_import_file_id => self.id, :body => row.fields.join("\t"))
 
-      patron = Patron.new
-      patron = set_patron_value(patron, row)
+      agent = Agent.new
+      agent = set_agent_value(agent, row)
 
-      if patron.save!
-        import_result.patron = patron
-        num[:patron_imported] += 1
+      if agent.save!
+        import_result.agent = agent
+        num[:agent_imported] += 1
         if row_num % 50 == 0
           Sunspot.commit
           GC.start
@@ -88,7 +88,7 @@ class PatronImportFile < ActiveRecord::Base
 
       #unless row['username'].to_s.strip.blank?
       #  user = User.new
-      #  user.patron = patron
+      #  user.agent = agent
       #  set_user_value(user, row)
       #  if user.password.blank?
       #    user.set_auto_generated_password
@@ -113,11 +113,11 @@ class PatronImportFile < ActiveRecord::Base
   end
 
   def self.import
-    PatronImportFile.not_imported.each do |file|
+    AgentImportFile.not_imported.each do |file|
       file.import_start
     end
   rescue
-    Rails.logger.info "#{Time.zone.now} importing patrons failed!"
+    Rails.logger.info "#{Time.zone.now} importing agents failed!"
   end
 
   def modify
@@ -128,25 +128,25 @@ class PatronImportFile < ActiveRecord::Base
     rows.each do |row|
       next if row['dummy'].to_s.strip.present?
       #user = User.where(:user_number => row['user_number'].to_s.strip).first
-      #if user.try(:patron)
-      #  set_patron_value(user.patron, row)
-      #  user.patron.save!
+      #if user.try(:agent)
+      #  set_agent_value(user.agent, row)
+      #  user.agent.save!
       #  set_user_value(user, row)
       #  user.save!
       #end
-      patron = Patron.where(:id => row['id']).first
-      if patron
-        patron.full_name = row['full_name'] if row['full_name'].to_s.strip.present?
-        patron.full_name_transcription = row['full_name_transcription'] if row['full_name_transcription'].to_s.strip.present?
-        patron.first_name = row['first_name'] if row['first_name'].to_s.strip.present?
-        patron.first_name_transcription = row['first_name_transcription'] if row['first_name_transcription'].to_s.strip.present?
-        patron.middle_name = row['middle_name'] if row['middle_name'].to_s.strip.present?
-        patron.middle_name_transcription = row['middle_name_transcription'] if row['middle_name_transcription'].to_s.strip.present?
-        patron.last_name = row['last_name'] if row['last_name'].to_s.strip.present?
-        patron.last_name_transcription = row['last_name_transcription'] if row['last_name_transcription'].to_s.strip.present?
-        patron.address_1 = row['address_1'] if row['address_1'].to_s.strip.present?
-        patron.address_2 = row['address_2'] if row['address_2'].to_s.strip.present?
-        patron.save!
+      agent = Agent.where(:id => row['id']).first
+      if agent
+        agent.full_name = row['full_name'] if row['full_name'].to_s.strip.present?
+        agent.full_name_transcription = row['full_name_transcription'] if row['full_name_transcription'].to_s.strip.present?
+        agent.first_name = row['first_name'] if row['first_name'].to_s.strip.present?
+        agent.first_name_transcription = row['first_name_transcription'] if row['first_name_transcription'].to_s.strip.present?
+        agent.middle_name = row['middle_name'] if row['middle_name'].to_s.strip.present?
+        agent.middle_name_transcription = row['middle_name_transcription'] if row['middle_name_transcription'].to_s.strip.present?
+        agent.last_name = row['last_name'] if row['last_name'].to_s.strip.present?
+        agent.last_name_transcription = row['last_name_transcription'] if row['last_name_transcription'].to_s.strip.present?
+        agent.address_1 = row['address_1'] if row['address_1'].to_s.strip.present?
+        agent.address_2 = row['address_2'] if row['address_2'].to_s.strip.present?
+        agent.save!
       end
       row_num += 1
     end
@@ -164,9 +164,9 @@ class PatronImportFile < ActiveRecord::Base
 
     rows.each do |row|
       next if row['dummy'].to_s.strip.present?
-      patron = Patron.where(:id => row['id'].to_s.strip).first
-      if patron
-        patron.destroy
+      agent = Agent.where(:id => row['id'].to_s.strip).first
+      if agent
+        agent.destroy
       end
       row_num += 1
     end
@@ -179,11 +179,11 @@ class PatronImportFile < ActiveRecord::Base
 
   private
   def open_import_file
-    tempfile = Tempfile.new('patron_import_file')
+    tempfile = Tempfile.new('agent_import_file')
     if Setting.uploaded_file.storage == :s3
-      uploaded_file_path = patron_import.expiring_url(10)
+      uploaded_file_path = agent_import.expiring_url(10)
     else
-      uploaded_file_path = patron_import.path
+      uploaded_file_path = agent_import.path
     end
     open(uploaded_file_path){|f|
       f.each{|line|
@@ -204,48 +204,48 @@ class PatronImportFile < ActiveRecord::Base
     file = CSV.open(tempfile, :col_sep => "\t")
     header = file.first
     rows = CSV.open(tempfile, :headers => header, :col_sep => "\t")
-    PatronImportResult.create!(:patron_import_file_id => self.id, :body => header.join("\t"))
+    AgentImportResult.create!(:agent_import_file_id => self.id, :body => header.join("\t"))
     tempfile.close(true)
     file.close
     rows
   end
 
-  def set_patron_value(patron, row)
-    patron.first_name = row['first_name'] if row['first_name']
-    patron.middle_name = row['middle_name'] if row['middle_name']
-    patron.last_name = row['last_name'] if row['last_name']
-    patron.first_name_transcription = row['first_name_transcription'] if row['first_name_transcription']
-    patron.middle_name_transcription = row['middle_name_transcription'] if row['middle_name_transcription']
-    patron.last_name_transcription = row['last_name_transcription'] if row['last_name_transcription']
+  def set_agent_value(agent, row)
+    agent.first_name = row['first_name'] if row['first_name']
+    agent.middle_name = row['middle_name'] if row['middle_name']
+    agent.last_name = row['last_name'] if row['last_name']
+    agent.first_name_transcription = row['first_name_transcription'] if row['first_name_transcription']
+    agent.middle_name_transcription = row['middle_name_transcription'] if row['middle_name_transcription']
+    agent.last_name_transcription = row['last_name_transcription'] if row['last_name_transcription']
 
-    patron.full_name = row['full_name'] if row['full_name']
-    patron.full_name_transcription = row['full_name_transcription'] if row['full_name_transcription']
+    agent.full_name = row['full_name'] if row['full_name']
+    agent.full_name_transcription = row['full_name_transcription'] if row['full_name_transcription']
 
-    patron.address_1 = row['address_1'] if row['address_1']
-    patron.address_2 = row['address_2'] if row['address_2']
-    patron.zip_code_1 = row['zip_code_1'] if row['zip_code_1']
-    patron.zip_code_2 = row['zip_code_2'] if row['zip_code_2']
-    patron.telephone_number_1 = row['telephone_number_1'] if row['telephone_number_1']
-    patron.telephone_number_2 = row['telephone_number_2'] if row['telephone_number_2']
-    patron.fax_number_1 = row['fax_number_1'] if row['fax_number_1']
-    patron.fax_number_2 = row['fax_number_2'] if row['fax_number_2']
-    patron.note = row['note'] if row['note']
-    patron.birth_date = row['birth_date'] if row['birth_date']
-    patron.death_date = row['death_date'] if row['death_date']
+    agent.address_1 = row['address_1'] if row['address_1']
+    agent.address_2 = row['address_2'] if row['address_2']
+    agent.zip_code_1 = row['zip_code_1'] if row['zip_code_1']
+    agent.zip_code_2 = row['zip_code_2'] if row['zip_code_2']
+    agent.telephone_number_1 = row['telephone_number_1'] if row['telephone_number_1']
+    agent.telephone_number_2 = row['telephone_number_2'] if row['telephone_number_2']
+    agent.fax_number_1 = row['fax_number_1'] if row['fax_number_1']
+    agent.fax_number_2 = row['fax_number_2'] if row['fax_number_2']
+    agent.note = row['note'] if row['note']
+    agent.birth_date = row['birth_date'] if row['birth_date']
+    agent.death_date = row['death_date'] if row['death_date']
 
     #if row['username'].to_s.strip.blank?
-      patron.email = row['email'].to_s.strip
-      patron.required_role = Role.where(:name => row['required_role_name'].to_s.strip.camelize).first || Role.find('Guest')
+      agent.email = row['email'].to_s.strip
+      agent.required_role = Role.where(:name => row['required_role_name'].to_s.strip.camelize).first || Role.find('Guest')
     #else
-    #  patron.required_role = Role.where(:name => row['required_role_name'].to_s.strip.camelize).first || Role.find('Librarian')
+    #  agent.required_role = Role.where(:name => row['required_role_name'].to_s.strip.camelize).first || Role.find('Librarian')
     #end
     language = Language.where(:name => row['language'].to_s.strip.camelize).first
     language = Language.where(:iso_639_2 => row['language'].to_s.strip.downcase).first unless language
     language = Language.where(:iso_639_1 => row['language'].to_s.strip.downcase).first unless language
-    patron.language = language if language
+    agent.language = language if language
     country = Country.where(:name => row['country'].to_s.strip).first
-    patron.country = country if country
-    patron
+    agent.country = country if country
+    agent
   end
 
   #def set_user_value(user, row)
@@ -277,7 +277,7 @@ end
 
 # == Schema Information
 #
-# Table name: patron_import_files
+# Table name: agent_import_files
 #
 #  id                         :integer          not null, primary key
 #  parent_id                  :integer
@@ -287,13 +287,13 @@ end
 #  note                       :text
 #  executed_at                :datetime
 #  state                      :string(255)
-#  patron_import_file_name    :string(255)
-#  patron_import_content_type :string(255)
-#  patron_import_file_size    :integer
-#  patron_import_updated_at   :datetime
+#  agent_import_file_name    :string(255)
+#  agent_import_content_type :string(255)
+#  agent_import_file_size    :integer
+#  agent_import_updated_at   :datetime
 #  created_at                 :datetime         not null
 #  updated_at                 :datetime         not null
-#  patron_import_fingerprint  :string(255)
+#  agent_import_fingerprint  :string(255)
 #  error_message              :text
 #  edit_mode                  :string(255)
 #

data/app/models/agent_import_result.rb

@@ -0,0 +1,26 @@
+class AgentImportResult < ActiveRecord::Base
+  attr_accessible :agent_import_file_id, :agent_id, :user_id, :body
+  default_scope :order => 'agent_import_results.id'
+  scope :file_id, proc{|file_id| where(:agent_import_file_id => file_id)}
+  scope :failed, where(:agent_id => nil)
+
+  belongs_to :agent_import_file
+  belongs_to :agent
+  belongs_to :user
+
+  validates_presence_of :agent_import_file_id
+end
+
+# == Schema Information
+#
+# Table name: agent_import_results
+#
+#  id                    :integer          not null, primary key
+#  agent_import_file_id :integer
+#  agent_id             :integer
+#  user_id               :integer
+#  body                  :text
+#  created_at            :datetime         not null
+#  updated_at            :datetime         not null
+#
+

data/app/models/{patron_relationship.rb → agent_relationship.rb}

@@ -1,8 +1,8 @@
-class PatronRelationship < ActiveRecord::Base
-  attr_accessible :parent_id, :child_id, :patron_relationship_type_id
-  belongs_to :parent, :foreign_key => 'parent_id', :class_name => 'Patron'
-  belongs_to :child, :foreign_key => 'child_id', :class_name => 'Patron'
-  belongs_to :patron_relationship_type
+class AgentRelationship < ActiveRecord::Base
+  attr_accessible :parent_id, :child_id, :agent_relationship_type_id
+  belongs_to :parent, :foreign_key => 'parent_id', :class_name => 'Agent'
+  belongs_to :child, :foreign_key => 'child_id', :class_name => 'Agent'
+  belongs_to :agent_relationship_type
   validate :check_parent
   validates_presence_of :parent_id, :child_id
   acts_as_list :scope => :parent_id
@@ -14,12 +14,12 @@ end
 
 # == Schema Information
 #
-# Table name: patron_relationships
+# Table name: agent_relationships
 #
 #  id                          :integer          not null, primary key
 #  parent_id                   :integer
 #  child_id                    :integer
-#  patron_relationship_type_id :integer
+#  agent_relationship_type_id :integer
 #  created_at                  :datetime         not null
 #  updated_at                  :datetime         not null
 #  position                    :integer

data/app/models/{patron_relationship_type.rb → agent_relationship_type.rb}

@@ -1,13 +1,13 @@
-class PatronRelationshipType < ActiveRecord::Base
+class AgentRelationshipType < ActiveRecord::Base
   attr_accessible :name, :display_name, :note
   include MasterModel
-  default_scope :order => 'patron_relationship_types.position'
-  has_many :patron_relationships
+  default_scope :order => 'agent_relationship_types.position'
+  has_many :agent_relationships
 end
 
 # == Schema Information
 #
-# Table name: patron_relationship_types
+# Table name: agent_relationship_types
 #
 #  id           :integer          not null, primary key
 #  name         :string(255)      not null

data/app/models/{patron_sweeper.rb → agent_sweeper.rb}

@@ -1,6 +1,6 @@
-class PatronSweeper < ActionController::Caching::Sweeper
+class AgentSweeper < ActionController::Caching::Sweeper
   include ExpireEditableFragment
-  observe Patron
+  observe Agent
 
   def after_save(record)
     expire_editable_fragment(record)
@@ -16,11 +16,11 @@ class PatronSweeper < ActionController::Caching::Sweeper
     record.donated_items.each do |item|
       expire_editable_fragment(item)
     end
-    record.original_patrons.each do |patron|
-      expire_editable_fragment(patron)
+    record.original_agents.each do |agent|
+      expire_editable_fragment(agent)
     end
-    record.derived_patrons.each do |patron|
-      expire_editable_fragment(patron)
+    record.derived_agents.each do |agent|
+      expire_editable_fragment(agent)
     end
   end
 

data/app/models/{patron_type.rb → agent_type.rb}

@@ -1,13 +1,13 @@
-class PatronType < ActiveRecord::Base
+class AgentType < ActiveRecord::Base
   attr_accessible :name, :display_name, :note
   include MasterModel
-  default_scope :order => "patron_types.position"
-  has_many :patrons
+  default_scope :order => "agent_types.position"
+  has_many :agents
 end
 
 # == Schema Information
 #
-# Table name: patron_types
+# Table name: agent_types
 #
 #  id           :integer          not null, primary key
 #  name         :string(255)      not null

data/app/models/country.rb

@@ -2,7 +2,7 @@ class Country < ActiveRecord::Base
   attr_accessible :name, :display_name, :alpha_2, :alpha_3, :numeric_3, :note
   include MasterModel
   default_scope :order => "countries.position"
-  has_many :patrons
+  has_many :agents
   #has_many :people
   #has_many :corporate_bodies
   #has_many :families

data/app/models/create.rb

@@ -1,19 +1,19 @@
 class Create < ActiveRecord::Base
-  attr_accessible :patron_id, :work_id, :create_type_id, :position
-  belongs_to :patron
+  attr_accessible :agent_id, :work_id, :create_type_id, :position
+  belongs_to :agent
   belongs_to :work, :class_name => 'Manifestation', :foreign_key => 'work_id'
   belongs_to :create_type
 
-  validates_associated :patron, :work
-  validates_presence_of :patron_id, :work_id
-  validates_uniqueness_of :work_id, :scope => :patron_id
+  validates_associated :agent, :work
+  validates_presence_of :agent_id, :work_id
+  validates_uniqueness_of :work_id, :scope => :agent_id
   after_save :reindex
   after_destroy :reindex
 
   acts_as_list :scope => :work
 
   def reindex
-    patron.try(:index)
+    agent.try(:index)
     work.try(:index)
   end
 end
@@ -23,7 +23,7 @@ end
 # Table name: creates
 #
 #  id             :integer          not null, primary key
-#  patron_id      :integer          not null
+#  agent_id      :integer          not null
 #  work_id        :integer          not null
 #  position       :integer
 #  created_at     :datetime         not null

data/app/models/donate.rb

@@ -1,9 +1,9 @@
 class Donate < ActiveRecord::Base
-  attr_accessible :patron_id, :item_id
-  belongs_to :patron, :validate => true
+  attr_accessible :agent_id, :item_id
+  belongs_to :agent, :validate => true
   belongs_to :item, :validate => true
-  validates_associated :patron, :item
-  validates_presence_of :patron, :item
+  validates_associated :agent, :item
+  validates_presence_of :agent, :item
 end
 
 # == Schema Information
@@ -11,7 +11,7 @@ end
 # Table name: donates
 #
 #  id         :integer          not null, primary key
-#  patron_id  :integer          not null
+#  agent_id  :integer          not null
 #  item_id    :integer          not null
 #  created_at :datetime         not null
 #  updated_at :datetime         not null

data/app/models/enju_biblio/ability.rb

@@ -31,10 +31,10 @@ module EnjuBiblio
           ManifestationRelationship,
           ManifestationRelationshipType,
           Own,
-          Patron,
-          PatronImportFile,
-          PatronRelationship,
-          PatronRelationshipType,
+          Agent,
+          AgentImportFile,
+          AgentRelationship,
+          AgentRelationshipType,
           PictureFile,
           Produce,
           ProduceType,
@@ -52,7 +52,7 @@ module EnjuBiblio
           Language,
           License,
           MediumOfPerformance,
-          PatronType,
+          AgentType,
           RequestStatusType,
           RequestType
         ] if LibraryGroup.site_config.network_access_allowed?(ip_address)
@@ -67,8 +67,8 @@ module EnjuBiblio
           Language,
           License,
           MediumOfPerformance,
-          PatronImportResult,
-          PatronType,
+          AgentImportResult,
+          AgentType,
           RequestStatusType,
           RequestType,
           ResourceImportResult
@@ -80,11 +80,11 @@ module EnjuBiblio
         can [:destroy, :delete], Manifestation do |manifestation|
           manifestation.items.empty? and !manifestation.series_master?
         end
-        can [:index, :create], Patron
-        can :show, Patron do |patron|
+        can [:index, :create], Agent
+        can :show, Agent do |patron|
           patron.required_role_id <= 3
         end
-        can [:update, :destroy, :delete], Patron do |patron|
+        can [:update, :destroy, :delete], Agent do |patron|
           !patron.user.try(:has_role?, 'Librarian') and patron.required_role_id <= 3
         end
         can :manage, [
@@ -95,8 +95,8 @@ module EnjuBiblio
           ImportRequest,
           ManifestationRelationship,
           Own,
-          PatronImportFile,
-          PatronRelationship,
+          AgentImportFile,
+          AgentRelationship,
           PictureFile,
           Produce,
           Realize,
@@ -114,9 +114,9 @@ module EnjuBiblio
           Language,
           License,
           ManifestationRelationshipType,
-          PatronImportResult,
-          PatronRelationshipType,
-          PatronType,
+          AgentImportResult,
+          AgentRelationshipType,
+          AgentType,
           RequestStatusType,
           RequestType,
           ResourceImportResult,
@@ -131,11 +131,11 @@ module EnjuBiblio
         can [:show, :edit], Manifestation do |manifestation|
           manifestation.required_role_id <= 2
         end
-        can :index, Patron
-        can :update, Patron do |patron|
+        can :index, Agent
+        can :update, Agent do |patron|
           patron.user == user
         end
-        can :show, Patron do |patron|
+        can :show, Agent do |patron|
           #if patron.user == user
           #  true
           #elsif patron.user != user
@@ -167,8 +167,8 @@ module EnjuBiblio
           ManifestationRelationshipType,
           MediumOfPerformance,
           Own,
-          PatronRelationship,
-          PatronRelationshipType,
+          AgentRelationship,
+          AgentRelationshipType,
           Produce,
           Realize,
           SeriesStatement
@@ -178,8 +178,8 @@ module EnjuBiblio
         can :show, Manifestation do |manifestation|
           manifestation.required_role_id == 1
         end
-        can :index, Patron
-        can :show, Patron do |patron|
+        can :index, Agent
+        can :show, Agent do |patron|
           patron.required_role_id == 1 #name == 'Guest'
         end
         can :read, [
@@ -200,8 +200,8 @@ module EnjuBiblio
           ManifestationRelationshipType,
           MediumOfPerformance,
           Own,
-          PatronRelationship,
-          PatronRelationshipType,
+          AgentRelationship,
+          AgentRelationshipType,
           PictureFile,
           Produce,
           Realize,

data/app/models/item.rb

@@ -15,11 +15,11 @@ class Item < ActiveRecord::Base
   has_one :exemplify, :dependent => :destroy
   has_one :manifestation, :through => :exemplify
   has_many :owns
-  has_many :patrons, :through => :owns
+  has_many :agents, :through => :owns
   delegate :display_name, :to => :shelf, :prefix => true
   belongs_to :bookstore, :validate => true
   has_many :donates
-  has_many :donors, :through => :donates, :source => :patron
+  has_many :donors, :through => :donates, :source => :agent
   belongs_to :required_role, :class_name => 'Role', :foreign_key => 'required_role_id', :validate => true
   has_one :resource_import_result
   belongs_to :budget_type
@@ -43,7 +43,7 @@ class Item < ActiveRecord::Base
       manifestation.id if manifestation
     end
     integer :shelf_id
-    integer :patron_ids, :multiple => true
+    integer :agent_ids, :multiple => true
     time :created_at
     time :updated_at
     time :acquired_at
@@ -69,8 +69,8 @@ class Item < ActiveRecord::Base
     manifestation.try(:publisher)
   end
 
-  def owned(patron)
-    owns.where(:patron_id => patron.id).first
+  def owned(agent)
+    owns.where(:agent_id => agent.id).first
   end
 
   def manifestation_url

data/app/models/item_sweeper.rb

@@ -5,8 +5,8 @@ class ItemSweeper < ActionController::Caching::Sweeper
   def after_save(record)
     expire_editable_fragment(record)
     expire_editable_fragment(record.manifestation)
-    record.patrons.each do |patron|
-      expire_editable_fragment(patron)
+    record.agents.each do |agent|
+      expire_editable_fragment(agent)
     end
     record.donors.each do |donor|
       expire_editable_fragment(donor)