enju_biblio 0.3.11 → 0.3.14

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 7ddb2060da09b931f698c95fe7183cea10953bb55d708d79cf118311af367c2b
-  data.tar.gz: de17c215f560054db264c931878c2a1ff2fd3ed0811911fb2feab4384afd03a0
+  metadata.gz: '088d27a989a71a9301fd1946a90d8c790720b2ecffa02bc2eef9d3a9c05a31bb'
+  data.tar.gz: b946f1cd533e99b47cb6a799a3d0f167b597d6291ee864372354037dfc3ad050
 SHA512:
-  metadata.gz: c577988123593c8b17f9cbe904492a4f148fb51d98f8294e863e639375303c9901a4fe8a67e3c471d1e406ab98399d86d6bd1b4bd6723ed547e2118f43c165a2
-  data.tar.gz: 6e65d6634be373354842dee7cf6fb8c6dd2e3b9683185b8ffbc5246e45061fe20e7f4c6566d24118a823598c439939ec7036bda2c9fc48e6eb82da6f3e187498
+  metadata.gz: 5815c72d545147d3048239c9c52749a1239ff73e06aa2fbc8e37014d8272b8d87b9dcd7c260b321512fd7bbc47d1383548abddff5bbfb5431c7f78aa7335cad4
+  data.tar.gz: b4da9e3a16f2a498cd8b70478422fb37dd511a69cb3bb94252d53c3c7df9e2876006c45a843915d594424350eac2f0f41ae567107d04aa60f812f59656d658d2

data/app/controllers/agent_import_results_controller.rb

@@ -15,7 +15,7 @@ class AgentImportResultsController < ApplicationController
     respond_to do |format|
       format.html # index.html.erb
       format.json { render json: @agent_import_results }
-      format.txt
+      format.text
     end
   end
 

data/app/controllers/agents_controller.rb

@@ -227,10 +227,4 @@ class AgentsController < ApplicationController
     @languages = Language.all
     @agent_type = AgentType.where(name: 'person').first
   end
-
-  def filtered_params
-    params.permit([:view, :format, :page, :language, :sort_by, :per_page])
-  end
-
-  helper_method :filtered_params
 end

data/app/controllers/items_controller.rb

@@ -18,7 +18,7 @@ class ItemsController < ApplicationController
     @count = {}
     if user_signed_in?
       if current_user.has_role?('Librarian')
-        if params[:format] == 'txt'
+        if params[:format] == 'text'
           per_page = 65534
         elsif params[:mode] == 'barcode'
           per_page = 40
@@ -51,6 +51,17 @@ class ItemsController < ApplicationController
 
     unless @inventory_file
       search = Sunspot.new_search(Item)
+      selected_attributes = [
+        :id, :item_identifier, :call_number, :manifestation_id, :acquired_at,
+        :binding_item_identifier, :binding_call_number, :binded_at,
+        :include_supplements, :url, :note,
+        :circulation_status_id, :shelf_id,
+        :created_at, :updated_at
+      ]
+      selected_attributes += [
+        :memo, :required_role_id, :budget_type_id, :bookstore_id, :price
+      ] if current_user.try(:has_role?, 'Librarian')
+      search.data_accessor_for(Item).select = selected_attributes
       set_role_query(current_user, search)
 
       @query = query.dup
@@ -122,8 +133,8 @@ class ItemsController < ApplicationController
 
     respond_to do |format|
       format.html # index.html.erb
-      format.json { render json: @items }
-      format.txt  { render layout: false }
+      format.json
+      format.text
       format.atom
     end
   end
@@ -137,7 +148,7 @@ class ItemsController < ApplicationController
 
     respond_to do |format|
       format.html # show.html.erb
-      format.json { render json: @item }
+      format.json
     end
   end
 
@@ -301,10 +312,4 @@ class ItemsController < ApplicationController
       end
     end
   end
-
-  def filtered_params
-    params.permit([:view, :format, :page, :library, :carrier_type, :reservable, :pub_date_from, :pub_date_until, :language, :sort_by, :per_page])
-  end
-
-  helper_method :filtered_params
 end

data/app/controllers/manifestations_controller.rb

@@ -29,29 +29,18 @@ class ManifestationsController < ApplicationController
       sort, @count = {}, {}
       query = ""
 
-      if params[:format] == 'txt'
+      if request.format.text?
         per_page = 65534
       end
 
-      if params[:format] == 'sru'
-        if params[:operation] == 'searchRetrieve'
-          sru = Sru.new(params)
-          query = sru.cql.to_sunspot
-          sort = sru.sort_by
-        else
-          render template: 'manifestations/explain', layout: false
-          return
-        end
+      if params[:api] == 'openurl'
+        openurl = Openurl.new(params)
+        @manifestations = openurl.search
+        query = openurl.query_text
+        sort = set_search_result_order(params[:sort_by], params[:order])
       else
-        if params[:api] == 'openurl'
-          openurl = Openurl.new(params)
-          @manifestations = openurl.search
-          query = openurl.query_text
-          sort = set_search_result_order(params[:sort_by], params[:order])
-        else
-          query = make_query(params[:query], params)
-          sort = set_search_result_order(params[:sort_by], params[:order])
-        end
+        query = make_query(params[:query], params)
+        sort = set_search_result_order(params[:sort_by], params[:order])
       end
 
       # 絞り込みを行わない状態のクエリ
@@ -191,37 +180,37 @@ class ManifestationsController < ApplicationController
       else
         per_page = Manifestation.default_per_page
       end
-      if params[:format] == 'sru'
-        #search.query.start_record(params[:startRecord] || 1, params[:maximumRecords] || 200)
+
+      pub_dates = parse_pub_date(params)
+      pub_date_range = {}
+
+      if pub_dates[:from] == '*'
+        pub_date_range[:from] = 0
       else
-        pub_dates = parse_pub_date(params)
-        pub_date_range = {}
-        if pub_dates[:from] == '*'
-          pub_date_range[:from] = 0
-        else
-          pub_date_range[:from] = Time.zone.parse(pub_dates[:from]).year
-        end
-        if pub_dates[:until] == '*'
-          pub_date_range[:until] = 10000
-        else
-          pub_date_range[:until] = Time.zone.parse(pub_dates[:until]).year
-        end
-        if params[:pub_year_range_interval]
-          pub_year_range_interval = params[:pub_year_range_interval].to_i
-        else
-          pub_year_range_interval = @library_group.pub_year_facet_range_interval || 10
-        end
+        pub_date_range[:from] = Time.zone.parse(pub_dates[:from]).year
+      end
+      if pub_dates[:until] == '*'
+        pub_date_range[:until] = 10000
+      else
+        pub_date_range[:until] = Time.zone.parse(pub_dates[:until]).year
+      end
 
-        search.build do
-          facet :reservable if defined?(EnjuCirculation)
-          facet :carrier_type
-          facet :library
-          facet :language
-          facet :pub_year, range: pub_date_range[:from]..pub_date_range[:until], range_interval: pub_year_range_interval
-          facet :subject_ids if defined?(EnjuSubject)
-          paginate page: page.to_i, per_page: per_page
-        end
+      if params[:pub_year_range_interval]
+        pub_year_range_interval = params[:pub_year_range_interval].to_i
+      else
+        pub_year_range_interval = @library_group.pub_year_facet_range_interval || 10
+      end
+
+      search.build do
+        facet :reservable if defined?(EnjuCirculation)
+        facet :carrier_type
+        facet :library
+        facet :language
+        facet :pub_year, range: pub_date_range[:from]..pub_date_range[:until], range_interval: pub_year_range_interval
+        facet :subject_ids if defined?(EnjuSubject)
+        paginate page: page.to_i, per_page: per_page
       end
+
       search_result = search.execute
       if @count[:query_result] > @max_number_of_results
         max_count = @max_number_of_results
@@ -261,9 +250,8 @@ class ManifestationsController < ApplicationController
       format.html
       format.html.phone
       format.xml  { render xml: @manifestations }
-      format.sru  { render layout: false }
       format.rss  { render layout: false }
-      format.txt  { render layout: false }
+      format.text  { render layout: false }
       format.rdf  { render layout: false }
       format.atom
       format.mods
@@ -333,8 +321,8 @@ class ManifestationsController < ApplicationController
       format.rdf
       format.mods
       format.json
-      format.txt
-      format.js
+      format.text
+      format.js { render template: 'manifestations/show.js' }
       format.download {
         if @manifestation.attachment.path
           if ENV['ENJU_STORAGE'] == 's3'
@@ -394,11 +382,8 @@ class ManifestationsController < ApplicationController
   # POST /manifestations
   # POST /manifestations.json
   def create
-    creators_params = manifestation_params[:creators_attributes]
-    @manifestation = Manifestation.new(manifestation_params.delete_if{|k, v|
-      k == 'creators_attributes'
-    })
-    parent = Manifestation.where(id: @manifestation.parent_id).first
+    @manifestation = Manifestation.new(manifestation_params)
+    parent = Manifestation.find_by(id: @manifestation.parent_id)
     unless @manifestation.original_title?
       @manifestation.original_title = @manifestation.attachment_file_name
     end
@@ -406,14 +391,16 @@ class ManifestationsController < ApplicationController
     respond_to do |format|
       if @manifestation.save
         Manifestation.transaction do
-          @manifestation.creators = Agent.new_agents(creators_params)
-          parent.derived_manifestations << @manifestation if parent
-        end
-        if parent
-          parent.index
-          @manifestation.index
+          set_creators
+
+          if parent
+            parent.derived_manifestations << @manifestation
+            parent.index
+            @manifestation.index
+          end
+
+          Sunspot.commit
         end
-        Sunspot.commit
 
         format.html { redirect_to @manifestation, notice: t('controller.successfully_created', model: t('activerecord.models.manifestation')) }
         format.json { render json: @manifestation, status: :created, location: @manifestation }
@@ -428,17 +415,11 @@ class ManifestationsController < ApplicationController
   # PUT /manifestations/1
   # PUT /manifestations/1.json
   def update
-    creators_params = manifestation_params[:creators_attributes]
-    @manifestation.assign_attributes(manifestation_params.delete_if{|k, v|
-      k == 'creators_attributes'
-    })
+    @manifestation.assign_attributes(manifestation_params)
 
     respond_to do |format|
-      if @manifestation.valid?
-        Manifestation.transaction do
-          @manifestation.creators = Agent.new_agents(creators_params)
-          @manifestation.save!
-        end
+      if @manifestation.save
+        set_creators
         
         format.html { redirect_to @manifestation, notice: t('controller.successfully_updated', model: t('activerecord.models.manifestation')) }
         format.json { head :no_content }
@@ -856,9 +837,20 @@ class ManifestationsController < ApplicationController
     query
   end
 
-  def filtered_params
-    params.permit([:view, :format, :library, :carrier_type, :reservable, :pub_date_from, :pub_date_until, :language, :sort_by, :per_page, :query])
-  end
+  def set_creators
+    creators_params = manifestation_params[:creators_attributes]
+    contributors_params = manifestation_params[:contributors_attributes]
+    publishers_params = manifestation_params[:publishers_attributes]
+
+    Manifestation.transaction do
+      @manifestation.creates.destroy_all
+      @manifestation.realizes.destroy_all
+      @manifestation.produces.destroy_all
+      @manifestation.reload
 
-  helper_method :filtered_params
+      @manifestation.creators = Agent.new_agents(creators_params)
+      @manifestation.contributors = Agent.new_agents(contributors_params)
+      @manifestation.publishers = Agent.new_agents(publishers_params)
+    end
+  end
 end

data/app/controllers/resource_import_results_controller.rb

@@ -7,7 +7,7 @@ class ResourceImportResultsController < ApplicationController
   def index
     @resource_import_file = ResourceImportFile.find_by(id: params[:resource_import_file_id])
     if @resource_import_file
-      if params[:format].to_s.downcase == 'txt'
+      if params[:format].to_s.downcase == 'text'
         @resource_import_results = @resource_import_file.resource_import_results
       else
         @resource_import_results = @resource_import_file.resource_import_results.page(params[:page])
@@ -19,7 +19,7 @@ class ResourceImportResultsController < ApplicationController
     respond_to do |format|
       format.html # index.html.erb
       format.json { render json: @resource_import_results }
-      format.txt
+      format.text
     end
   end
 

data/app/controllers/series_statements_controller.rb

@@ -133,10 +133,4 @@ class SeriesStatementsController < ApplicationController
       :series_master
     )
   end
-
-  def filtered_params
-    params.permit([:locale, :format, :page, :per_page])
-  end
-
-  helper_method :filtered_params
 end

data/app/helpers/manifestations_helper.rb

@@ -71,10 +71,10 @@ module ManifestationsHelper
     current = true if languages.include?(language.name)
     if current
       content_tag :strong do
-        link_to("#{language.display_name.localize} (" + facet.count.to_s + ")", url_for(filtered_params.merge(page: nil, language: language.name, view: nil, only_path: true)))
+        link_to("#{language.display_name.localize} (" + facet.count.to_s + ")", url_for(request.params.merge(page: nil, language: language.name, view: nil, only_path: true)))
       end
     else
-      link_to("#{language.display_name.localize} (" + facet.count.to_s + ")", url_for(filtered_params.merge(page: nil, language: language.name, view: nil, only_path: true)))
+      link_to("#{language.display_name.localize} (" + facet.count.to_s + ")", url_for(request.params.merge(page: nil, language: language.name, view: nil, only_path: true)))
     end
   end
 
@@ -86,10 +86,10 @@ module ManifestationsHelper
     content_tag :li do
       if current
         content_tag :strong do
-          link_to("#{library.display_name.localize} (" + facet.count.to_s + ")", url_for(filtered_params.merge(page: nil, library: (current_libraries << library.name).uniq.join(' '), view: nil, only_path: true)))
+          link_to("#{library.display_name.localize} (" + facet.count.to_s + ")", url_for(request.params.merge(page: nil, library: (current_libraries << library.name).uniq.join(' '), view: nil, only_path: true)))
         end
       else
-        link_to("#{library.display_name.localize} (" + facet.count.to_s + ")", url_for(filtered_params.merge(page: nil, library: (current_libraries << library.name).uniq.join(' '), view: nil, only_path: true)))
+        link_to("#{library.display_name.localize} (" + facet.count.to_s + ")", url_for(request.params.merge(page: nil, library: (current_libraries << library.name).uniq.join(' '), view: nil, only_path: true)))
       end
     end
   end
@@ -102,10 +102,10 @@ module ManifestationsHelper
       current = true if params[:carrier_type] == carrier_type.name
       if current
         content_tag :strong do
-          link_to("#{carrier_type.display_name.localize} (" + facet.count.to_s + ")", url_for(filtered_params.merge(carrier_type: carrier_type.name, page: nil, view: nil, only_path: true)))
+          link_to("#{carrier_type.display_name.localize} (" + facet.count.to_s + ")", url_for(request.params.merge(carrier_type: carrier_type.name, page: nil, view: nil, only_path: true)))
         end
       else
-        link_to("#{carrier_type.display_name.localize} (" + facet.count.to_s + ")", url_for(filtered_params.merge(carrier_type: carrier_type.name, page: nil, view: nil, only_path: true)))
+        link_to("#{carrier_type.display_name.localize} (" + facet.count.to_s + ")", url_for(request.params.merge(carrier_type: carrier_type.name, page: nil, view: nil, only_path: true)))
       end
     end
   end
@@ -115,10 +115,10 @@ module ManifestationsHelper
     current = true if facet.value.first.to_i == pub_date_from.to_i and facet.value.last.to_i - 1 == pub_date_until.to_i
     if current
       content_tag :strong do
-        link_to("#{facet.value.first.to_i} - #{facet.value.last.to_i - 1} (" + facet.count.to_s + ")", url_for(filtered_params.merge(pub_date_from: facet.value.first.to_i, pub_date_until: facet.value.last.to_i - 1, page: nil, view: nil, only_path: true)))
+        link_to("#{facet.value.first.to_i} - #{facet.value.last.to_i - 1} (" + facet.count.to_s + ")", url_for(request.params.merge(pub_date_from: facet.value.first.to_i, pub_date_until: facet.value.last.to_i - 1, page: nil, view: nil, only_path: true)))
       end
     else
-      link_to("#{facet.value.first.to_i} - #{facet.value.last.to_i - 1} (" + facet.count.to_s + ")", url_for(filtered_params.merge(pub_date_from: facet.value.first.to_i, pub_date_until: facet.value.last.to_i - 1, page: nil, view: nil, only_path: true)))
+      link_to("#{facet.value.first.to_i} - #{facet.value.last.to_i - 1} (" + facet.count.to_s + ")", url_for(request.params.merge(pub_date_from: facet.value.first.to_i, pub_date_until: facet.value.last.to_i - 1, page: nil, view: nil, only_path: true)))
     end
   end
 

data/app/models/agent.rb

@@ -256,17 +256,26 @@ class Agent < ApplicationRecord
     Agent.transaction do
       agents_params.each do |k, v|
         next if v['_destroy'] == '1'
+
+        agent = nil
+
         if v['agent_id'].present?
           agent = Agent.find(v['agent_id'])
         elsif v['id'].present?
           agent = Agent.find(v['id'])
-        else
+        end
+
+        if !agent or agent.full_name != v['full_name']
+          v.delete('id')
+          v.delete('agent_id')
           v.delete('_destroy')
           agent = Agent.create(v)
         end
+
         agents << agent
       end
     end
+
     agents
   end
 end

data/app/models/form_of_work.rb

@@ -1,6 +1,6 @@
 class FormOfWork < ApplicationRecord
   include MasterModel
-  has_many :works, class_name: 'Manifestation'
+  has_many :works, class_name: 'Manifestation' #, dependent: :restrict_with_exception
 end
 
 # == Schema Information

data/app/models/frequency.rb

@@ -1,6 +1,6 @@
 class Frequency < ApplicationRecord
   include MasterModel
-  has_many :manifestations
+  has_many :manifestations, dependent: :restrict_with_exception
 end
 
 # == Schema Information

data/app/models/identifier.rb

@@ -8,7 +8,7 @@ class Identifier < ApplicationRecord
   before_validation :normalize
   before_save :convert_isbn
   scope :id_type, -> type {
-    where(identifier_type: IdentifierType.where(name: type).first)
+    where(identifier_type: IdentifierType.find_by(name: type))
   }
 
   acts_as_list scope: :manifestation_id

data/app/models/import_request.rb

@@ -27,9 +27,8 @@ class ImportRequest < ApplicationRecord
 
   def check_imported
     if isbn.present?
-      identifier_type = IdentifierType.where(name: 'isbn').first
-      identifier_type = IdentifierType.where(name: 'isbn').create! unless identifier_type
-      if Identifier.where(body: isbn, identifier_type_id: identifier_type.id).first.try(:manifestation)
+      identifier_type = IdentifierType.find_by_or_create!(name: 'isbn')
+      if Identifier.find_by(body: isbn, identifier_type_id: identifier_type.id)&.manifestation
         errors.add(:isbn, I18n.t('import_request.isbn_taken'))
       end
     end

data/app/models/item.rb

@@ -66,23 +66,23 @@ class Item < ApplicationRecord
   paginates_per 10
 
   def title
-    manifestation.try(:original_title)
+    manifestation&.original_title
   end
 
   def creator
-    manifestation.try(:creator)
+    manifestation&.creator
   end
 
   def contributor
-    manifestation.try(:contributor)
+    manifestation&.contributor
   end
 
   def publisher
-    manifestation.try(:publisher)
+    manifestation&.publisher
   end
 
   def owned(agent)
-    owns.where(agent_id: agent.id).first
+    owns.find_by(agent_id: agent.id)
   end
 
   def manifestation_url
@@ -123,7 +123,7 @@ class Item < ApplicationRecord
         budget_type: budget_type&.name,
         item_required_role: required_role.name,
         item_price: price,
-        memo: memo
+        item_memo: memo
       })
 
       ItemCustomProperty.order(:position).each do |custom_property|

data/app/models/item_custom_value.rb

@@ -1,6 +1,6 @@
 class ItemCustomValue < ApplicationRecord
   belongs_to :item_custom_property
-  belongs_to :item, required: false
+  belongs_to :item, optional: true
   validates :item_custom_property, uniqueness: {scope: :item_id}
 end
 

data/app/models/manifestation.rb

@@ -5,12 +5,12 @@ class Manifestation < ApplicationRecord
   enju_nii_cinii_books if defined?(EnjuNii)
   enju_bookmark_manifestation_model if defined?(EnjuBookmark)
 
-  has_many :creates, dependent: :destroy, foreign_key: 'work_id'
-  has_many :creators, through: :creates, source: :agent #, order: 'creates.position'
-  has_many :realizes, dependent: :destroy, foreign_key: 'expression_id'
-  has_many :contributors, through: :realizes, source: :agent #, order: 'realizes.position'
-  has_many :produces, dependent: :destroy, foreign_key: 'manifestation_id'
-  has_many :publishers, through: :produces, source: :agent #, order: 'produces.position'
+  has_many :creates, -> { order('creates.position') }, dependent: :destroy, foreign_key: 'work_id'
+  has_many :creators, through: :creates, source: :agent
+  has_many :realizes, -> { order('realizes.position') }, dependent: :destroy, foreign_key: 'expression_id'
+  has_many :contributors, through: :realizes, source: :agent
+  has_many :produces, -> { order('produces.position') }, dependent: :destroy, foreign_key: 'manifestation_id'
+  has_many :publishers, through: :produces, source: :agent
   has_many :items, dependent: :destroy
   has_many :children, foreign_key: 'parent_id', class_name: 'ManifestationRelationship', dependent: :destroy
   has_many :parents, foreign_key: 'child_id', class_name: 'ManifestationRelationship', dependent: :destroy
@@ -37,7 +37,7 @@ class Manifestation < ApplicationRecord
     text :title, default_boost: 2 do
       titles
     end
-    [ :fulltext, :note, :creator, :contributor, :publisher, :description, :statement_of_responsibility ].each do |field|
+    [ :fulltext, :note, :creator, :contributor, :publisher, :abstract, :description, :statement_of_responsibility ].each do |field|
       text field do
         if series_master?
           derived_manifestations.map{|c| c.send(field) }.flatten.compact
@@ -518,16 +518,7 @@ class Manifestation < ApplicationRecord
   end
 
   def identifier_contents(name)
-    if Rails::VERSION::MAJOR > 3
-      identifiers.id_type(name).order(:position).pluck(:body)
-    else
-      identifier_type = IdentifierType.find_by(name: name)
-      if identifier_type
-        identifiers.where(identifier_type_id: identifier_type.id).order(:position).pluck(:body)
-      else
-        []
-      end
-    end
+    identifiers.id_type(name).order(:position).pluck(:body)
   end
 
   # CSVのヘッダ
@@ -596,6 +587,7 @@ class Manifestation < ApplicationRecord
       price: price,
       access_address: access_address,
       manifestation_required_role: required_role.name,
+      abstract: abstract,
       description: description,
       note: note
     }
@@ -624,7 +616,7 @@ class Manifestation < ApplicationRecord
 
     if ['Administrator', 'Librarian'].include?(role)
       record.merge!({
-        memo: memo
+        manifestation_memo: memo
       })
       ManifestationCustomProperty.order(:position).each do |custom_property|
         custom_value = manifestation_custom_values.find_by(manifestation_custom_property: custom_property)

data/app/models/manifestation_custom_value.rb

@@ -1,6 +1,6 @@
 class ManifestationCustomValue < ApplicationRecord
   belongs_to :manifestation_custom_property
-  belongs_to :manifestation, required: false
+  belongs_to :manifestation, optional: true
   validates :manifestation_custom_property, uniqueness: {scope: :manifestation_id}
 end
 

data/app/models/resource_import_file.rb

@@ -458,12 +458,12 @@ class ResourceImportFile < ApplicationRecord
       edition edition_string serial_number isbn issn manifestation_price
       width height depth number_of_pages jpno lccn budget_type bookstore
       language fulltext_content required_role doi content_type frequency
-      extent start_page end_page dimensions
+      extent start_page end_page dimensions abstract manifestation_memo
       ncid
       ndl_bib_id
       statement_of_responsibility acquired_at call_number circulation_status
       binding_item_identifier binding_call_number binded_at item_price
-      use_restriction include_supplements item_note item_url
+      use_restriction include_supplements item_note item_url item_memo
       dummy
     )
     header_columns += ManifestationCustomProperty.order(:position).pluck(:name).map{|c| "manifestation:#{c}"}
@@ -541,7 +541,8 @@ class ResourceImportFile < ApplicationRecord
       binding_call_number: row['binding_call_number'],
       binded_at: binded_at,
       url: row['item_url'],
-      note: row['item_note'].try(:gsub, /\\n/, "\n")
+      note: row['item_note'].try(:gsub, /\\n/, "\n"),
+      memo: row['item_memo'].try(:gsub, /\\n/, "\n")
     )
     manifestation.items << item
     if defined?(EnjuCirculation)
@@ -600,6 +601,7 @@ class ResourceImportFile < ApplicationRecord
 
     item.price = row['item_price'] if row['item_price'].present?
     item.note = row['item_note'].try(:gsub, /\\n/, "\n") if row['item_note'].present?
+    item.memo = row['item_memo'].try(:gsub, /\\n/, "\n") if row['item_memo'].present?
     item.url = row['item_url'] if row['item_url'].present?
 
     if row['include_supplements']
@@ -716,9 +718,11 @@ class ResourceImportFile < ApplicationRecord
         depth: row['depth'],
         height: row['height'],
         price: row['manifestation_price'],
+        abstract: row['abstract'].try(:gsub, /\\n/, "\n"),
         description: row['description'].try(:gsub, /\\n/, "\n"),
         #:description_transcription => row['description_transcription'],
         note: row['note'].try(:gsub, /\\n/, "\n"),
+        memo: row['manifestation_memo'].try(:gsub, /\\n/, "\n"),
         statement_of_responsibility: row['statement_of_responsibility'],
         access_address: row['access_address'],
         manifestation_identifier: row['manifestation_identifier'],
@@ -783,7 +787,7 @@ class ResourceImportFile < ApplicationRecord
             unless identifiers.empty?
               identifiers.each do |v|
                 v.manifestation = manifestation
-                v.save!
+                v.save
               end
             end
           else

data/app/views/agent_relationship_types/index.html.erb

@@ -18,7 +18,7 @@
 <%- @agent_relationship_types.each do |agent_relationship_type| -%>
   <tr class="line<%= cycle("0", "1") -%>">
     <td>
-      <%- if policy(agent).update?_relationship_type -%>
+      <%- if policy(agent_relationship_type).update? -%>
         <%= move_position(agent_relationship_type) -%>
       <%- end -%>
     </td>
@@ -29,10 +29,10 @@
       <%= agent_relationship_type.note %>
     </td>
     <td>
-      <%- if policy(agent).update?_relationship_type -%>
+      <%- if policy(agent_relationship_type).update? -%>
         <%= link_to t('page.edit'), edit_agent_relationship_type_path(agent_relationship_type) -%>
       <% end %>
-      <%- if policy(agent).destroy?_relationship_type -%>
+      <%- if policy(agent_relationship_type).destroy? -%>
         <%= link_to t('page.destroy'), agent_relationship_type_path(agent_relationship_type), data: {confirm: t('page.are_you_sure')}, method: :delete -%>
       <%- end -%>
     </td>