enju_biblio 0.3.6 → 0.3.7

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 1c5d6d8f73b0961bc27b8d475802d2d22372f06e77fc380daba1a2c57f16b1f5
-  data.tar.gz: 19afaf59c4800afacbe0c7457414527f0dd46e1c1966212845739e168552cf5d
+  metadata.gz: bb38665a417d9e1788c1d171ac936357450704fd4d8ab7e87bbc02b1fe964aa1
+  data.tar.gz: 318c358c22dd3df104b6e2b4353e6b7d4758994f6fde55401e8fdac97345750a
 SHA512:
-  metadata.gz: 8821f0a2291b6f03aaaa528985fa7bd9240a0f7cf50fa97d8689e1edf287766aeff6b9e61dec69797a23e367e91c989bf96a30910dfd6470b2e0c99edf51df40
-  data.tar.gz: 59247ed9b84ec043a99f8b6d7b68bc79b8454dc0fe19c38611b8081b8d873cc885bda488ddeb7bfd9fa7c71b1f3d8bba37ecbc28e0f7c84d77336ea64e7a2b8a
+  metadata.gz: e90ea6fdc7120d2ce1456862cd5e55baafa9403f85af50119ad2b6dea81dd251f924025ef70d27a1c9060f7364ffb391eae7a22b0f5e08ea36201191b42687e5
+  data.tar.gz: 2c15b0e9b79fda1c4e7164a49a9048ab8d024e5748ec72ca295ab68f3da67d2cac07053de6848577fcdd98507abd0f4d095a832d6676dedca6411c3a9e1f7c9d

data/app/controllers/items_controller.rb

@@ -273,7 +273,11 @@ class ItemsController < ApplicationController
       :lock_version, :manifestation_id, :library_id, :required_role_id,
       :binding_item_identifier, :binding_call_number, :binded_at,
       :use_restriction_id, :memo,
-      item_has_use_restriction_attributes: :use_restriction_id # EnjuCirculation
+      {item_has_use_restriction_attributes: :use_restriction_id}, # EnjuCirculation
+      {custom_properties_attributes: [
+        :id, :label, :value,
+        :_destroy
+      ]}
     )
   end
 

data/app/controllers/manifestations_controller.rb

@@ -429,15 +429,17 @@ class ManifestationsController < ApplicationController
   # PUT /manifestations/1.json
   def update
     creators_params = manifestation_params[:creators_attributes]
-    Manifestation.transaction do
-      @manifestation.assign_attributes(manifestation_params.delete_if{|k, v|
-        k == 'creators_attributes'
-      })
-      @manifestation.creators = Agent.new_agents(creators_params)
-    end
+    @manifestation.assign_attributes(manifestation_params.delete_if{|k, v|
+      k == 'creators_attributes'
+    })
 
     respond_to do |format|
-      if @manifestation.save
+      if @manifestation.valid?
+        Manifestation.transaction do
+          @manifestation.creators = Agent.new_agents(creators_params)
+          @manifestation.save!
+        end
+        
         format.html { redirect_to @manifestation, notice: t('controller.successfully_updated', model: t('activerecord.models.manifestation')) }
         format.json { head :no_content }
       else
@@ -547,6 +549,10 @@ class ManifestationsController < ApplicationController
       {identifiers_attributes: [
         :id, :body, :identifier_type_id,
         :_destroy
+      ]},
+      {custom_properties_attributes: [
+        :id, :label, :value,
+        :_destroy
       ]}
     )
   end

data/app/models/custom_property.rb

@@ -0,0 +1,17 @@
+class CustomProperty < ApplicationRecord
+  belongs_to :resource, polymorphic: true
+  validates :label, presence: true
+end
+
+# == Schema Information
+#
+# Table name: custom_properties
+#
+#  id            :integer          not null, primary key
+#  resource_id   :integer          not null
+#  resource_type :string           not null
+#  label         :text             not null
+#  value         :text
+#  created_at    :datetime         not null
+#  updated_at    :datetime         not null
+#

data/app/models/item.rb

@@ -17,8 +17,10 @@ class Item < ApplicationRecord
   belongs_to :bookstore, optional: true
   belongs_to :required_role, class_name: 'Role', foreign_key: 'required_role_id'
   belongs_to :budget_type, optional: true
-  has_one :accept
-  has_one :withdraw
+  has_one :accept, dependent: :destroy
+  has_one :withdraw, dependent: :destroy
+  has_many :custom_properties, as: :resource, dependent: :destroy
+  accepts_nested_attributes_for :custom_properties, allow_destroy: true, reject_if: :all_blank
   scope :accepted_between, lambda{|from, to| includes(:accept).where('items.created_at BETWEEN ? AND ?', Time.zone.parse(from).beginning_of_day, Time.zone.parse(to).end_of_day)}
 
   belongs_to :shelf, counter_cache: true
@@ -96,6 +98,52 @@ class Item < ApplicationRecord
       true
     end
   end
+
+  def self.csv_header(role: 'Guest')
+    Item.new.to_hash(role: role).keys
+  end
+
+  def to_hash(role: 'Guest')
+    record = {
+      item_id: id,
+      item_identifier: item_identifier,
+      call_number: call_number,
+      shelf: shelf.name,
+      item_note: note,
+      accepted_at: accept.try(:created_at),
+      acquired_at: acquired_at,
+      item_created_at: created_at,
+      item_updated_at: updated_at
+    }
+
+    if ['Administrator', 'Librarian'].include?(role)
+      record.merge!({
+        bookstore: bookstore.try(:name),
+        budget_type: budget_type.try(:name),
+        item_price: price,
+        memo: memo
+      })
+
+      # 最もカスタム項目の多い資料について、カスタム項目の個数を取得する
+      ActiveRecord::Base.connection.execute('SELECT max(record_count) FROM (SELECT count(*) AS record_count, resource_id, resource_type FROM custom_properties GROUP BY resource_id, resource_type) AS type_count ;').first.values[0].to_i.times do |i|
+        property = custom_properties[i]
+        if property
+          record[:"item_custom_property_#{i + 1}"] = "#{property.label}: #{property.value}"
+        else
+          record[:"item_custom_property_#{i + 1}"] = nil
+        end
+      end
+
+      if defined?(EnjuCirculation)
+        record.merge!({
+          use_restriction: use_restriction.try(:name),
+          total_checkouts: checkouts.count
+        })
+      end
+    end
+
+    record
+  end
 end
 
 # == Schema Information
@@ -125,4 +173,5 @@ end
 #  binding_call_number     :string
 #  binded_at               :datetime
 #  manifestation_id        :integer          not null
+#  memo                    :text
 #

data/app/models/manifestation.rb

@@ -25,11 +25,13 @@ class Manifestation < ApplicationRecord
   belongs_to :required_role, class_name: 'Role', foreign_key: 'required_role_id'
   has_one :resource_import_result
   has_many :identifiers, dependent: :destroy
+  has_many :custom_properties, as: :resource, dependent: :destroy
   accepts_nested_attributes_for :creators, allow_destroy: true, reject_if: :all_blank
   accepts_nested_attributes_for :contributors, allow_destroy: true, reject_if: :all_blank
   accepts_nested_attributes_for :publishers, allow_destroy: true, reject_if: :all_blank
   accepts_nested_attributes_for :series_statements, allow_destroy: true, reject_if: :all_blank
   accepts_nested_attributes_for :identifiers, allow_destroy: true, reject_if: :all_blank
+  accepts_nested_attributes_for :custom_properties, allow_destroy: true, reject_if: :all_blank
 
   searchable do
     text :title, default_boost: 2 do
@@ -528,266 +530,128 @@ class Manifestation < ApplicationRecord
     end
   end
 
-  def self.csv_header(role, options = {col_sep: "\t", role: :Guest})
-    header = %w(
-      manifestation_id
-      original_title
-      title_transcription
-      creator
-      contributor
-      publisher
-      pub_date
-      statement_of_responsibility
-      manifestation_price
-      manifestation_created_at
-      manifestation_updated_at
-      manifestation_identifier
-      access_address
-      description
-      note
-      extent
-      dimensions
-      carrier_type
-      edition
-      edition_string
-      volume_number
-      volume_number_string
-      issue_number
-      issue_number_string
-      serial_number
+  # CSVのヘッダ
+  # @param [String] role 権限
+  def self.csv_header(role: 'Guest')
+    Manifestation.new.to_hash(role: role).keys
+  end
+
+  # CSV出力用のハッシュ
+  # @param [String] role 権限
+  def to_hash(role: 'Guest')
+    record = {
+      manifestation_id: id,
+      title: original_title,
+      title_alternative: title_alternative,
+      title_transcription: title_transcription,
+      statement_of_responsibility: statement_of_responsibility,
+      serial: serial,
+      manifestation_identifier: manifestation_identifier,
+      creator: creators.pluck(:full_name).join('//'),
+      contributor: contributors.pluck(:full_name).join('//'),
+      publisher: publishers.pluck(:full_name).join('//'),
+      date_of_publication: date_of_publication,
+      year_of_publication: year_of_publication,
+      publication_place: publication_place,
+      manifestation_created_at: created_at,
+      manifestation_updated_at: updated_at,
+      carrier_type: carrier_type.name,
+      content_type: manifestation_content_type.name,
+      frequency: frequency.name,
+      language: language.name,
+      isbn: identifier_contents(:isbn).join('//'),
+      issn: identifier_contents(:issn).join('//'),
+      volume_number: volume_number,
+      volume_number_string: volume_number_string,
+      edition: edition,
+      edition_string: edition_string,
+      issue_number: issue_number,
+      issue_number_string: issue_number_string,
+      serial_number: serial_number,
+      extent: extent,
+      start_page: start_page,
+      end_page: end_page,
+      dimensions: dimensions,
+      height: height,
+      width: width,
+      depth: depth,
+      price: price,
+      access_address: access_address,
+      required_role: required_role.name,
+      description: description,
+      note: note
+    }
+
+    IdentifierType.find_each do |type|
+      record[type.name.to_sym] = identifiers.where(identifier_type: type).pluck(:body).join('//')
+    end
+
+    series = series_statements.order(:position)
+    record.merge!(
+      series_statement_id: series.pluck(:id).join('//'),
+      series_statement_original_title: series.pluck(:original_title).join('.//'),
+      series_statement_title_subseries: series.pluck(:title_subseries).join('//'),
+      series_statement_title_subseries_transcription: series.pluck(:title_subseries_transcription).join('//'),
+      series_statement_title_transcription: series.pluck(:title_transcription).join('//'),
+      series_statement_creator: series.pluck(:creator_string).join('//'),
+      series_statement_volume_number: series.pluck(:volume_number_string).join('//'),
+      series_statement_series_master: series.pluck(:series_master).join('//'),
+      series_statement_root_manifestation_id: series.pluck(:root_manifestation_id).join('//'),
+      series_statement_manifestation_id: series.pluck(:manifestation_id).join('//'),
+      series_statement_position: series.pluck(:position).join('//'),
+      series_statement_note: series.pluck(:note).join('//'),
+      series_statement_created_at: series.pluck(:created_at).join('//'),
+      series_statement_updated_at: series.pluck(:updated_at).join('//')
     )
 
-    header += IdentifierType.order(:position).pluck(:name)
-    if defined?(EnjuSubject)
-      header += SubjectHeadingType.order(:position).pluck(:name).map{|type| "subject:#{type}"}
-      header += ClassificationType.order(:position).pluck(:name).map{|type| "classification:#{type}"}
-    end
+    if ['Administrator', 'Librarian'].include?(role)
+      record.merge!({
+        memo: memo
+      })
 
-    header += %w(
-      item_id
-      item_identifier
-      call_number
-      item_note
-    )
-    case role.to_sym
-    when :Administrator, :Librarian
-      header << "item_price"
+      # 最もカスタム項目の多い資料について、カスタム項目の個数を取得する
+      ActiveRecord::Base.connection.execute('SELECT max(record_count) FROM (SELECT count(*) AS record_count, resource_id, resource_type FROM custom_properties GROUP BY resource_id, resource_type) AS type_count ;').first.values[0].to_i.times do |i|
+        property = custom_properties[i]
+        if property
+          record[:"manifestation_custom_property_#{i + 1}"] = "#{property.label}: #{property.value}"
+        else
+          record[:"manifestation_custom_property_#{i + 1}"] = nil
+        end
+      end
     end
-    header += %w(
-      acquired_at
-      accepted_at
-    )
-    case role.to_sym
-    when :Administrator, :Librarian
-      header += %w(
-        bookstore
-        budget_type
-        total_checkouts
-      )
-    end
-    header += %w(
-      circulation_status
-      shelf
-      library
-      item_created_at
-      item_updated_at
-    )
-    case role.to_sym
-    when :Administrator, :Librarian
-      header << "use_restriction"
+
+    if defined?(EnjuSubject)
+      SubjectHeadingType.find_each do |type|
+        record[:"subject:#{type.name}"] = subjects.where(subject_heading_type: type).pluck(:term).join('//')
+      end
+      ClassificationType.find_each do |type|
+        record[:"classification:#{type.name}"] = classifications.where(classification_type: type).pluck(:category).join('//')
+      end
     end
 
-    header.to_csv(options)
+    record
   end
 
-  def to_csv(options = {format: :txt, role: :Guest})
-    lines = []
-    if items.exists?
-      items.includes(shelf: :library).each do |i|
-        item_lines = []
-        item_lines << id
-        item_lines << original_title
-        item_lines << title_transcription
-        if creators.exists?
-          item_lines << creators.pluck(:full_name).join("//")
-        else
-          item_lines << nil
-        end
-        if contributors.exists?
-          item_lines << contributors.pluck(:full_name).join("//")
-        else
-          item_lines << nil
-        end
-        if publishers.exists?
-          item_lines << publishers.pluck(:full_name).join("//")
-        else
-          item_lines << nil
-        end
-        item_lines << pub_date
-        item_lines << statement_of_responsibility
-        item_lines << price
-        item_lines << created_at
-        item_lines << updated_at
-        item_lines << manifestation_identifier
-        item_lines << access_address
-        item_lines << description.try(:gsub, /\r?\n/, '\n')
-        item_lines << note.try(:gsub, /\r?\n/, '\n')
-        item_lines << extent
-        item_lines << dimensions
-        item_lines << carrier_type.name
-        item_lines << edition
-        item_lines << edition_string
-        item_lines << volume_number
-        item_lines << volume_number_string
-        item_lines << issue_number
-        item_lines << issue_number_string
-        item_lines << serial_number
-
-        IdentifierType.order(:position).pluck(:name).each do |identifier_type|
-          identifier_list = identifier_contents(identifier_type.to_sym)
-          if identifier_list
-            item_lines << identifier_list.join("//")
-          else
-            item_lines << nil
+  # TSVでのエクスポート
+  # @param [String] role 権限
+  # @param [String] col_sep 区切り文字
+  def self.export(role: 'Guest', col_sep: "\t")
+    file = Tempfile.create('manifestation_export') do |f|
+      f.write (Manifestation.csv_header(role: role) + Item.csv_header(role: role)).to_csv(col_sep: col_sep)
+      Manifestation.find_each do |manifestation|
+        if manifestation.items.exists?
+          manifestation.items.each do |item|
+            f.write (manifestation.to_hash(role: role).values + item.to_hash(role: role).values).to_csv(col_sep: col_sep)
           end
-        end
-        if defined?(EnjuSubject)
-          SubjectHeadingType.order(:position).each do |subject_heading_type|
-            if subjects.exists?
-              item_lines << subjects.where(subject_heading_type: subject_heading_type).pluck(:term).join('//')
-            else
-              item_lines << nil
-            end
-          end
-          ClassificationType.order(:position).each do |classification_type|
-            if classifications.exists?
-              item_lines << classifications.where(classification_type: classification_type).pluck(:category).join('//')
-            else
-              item_lines << nil
-            end
-          end
-        end
-
-        item_lines << i.id
-        item_lines << i.item_identifier
-        item_lines << i.call_number
-        item_lines << i.note.try(:gsub, /\r?\n/, '\n')
-        case options[:role].to_sym
-        when :Administrator, :Librarian
-          item_lines << i.price
-        end
-        item_lines << i.acquired_at
-        item_lines << i.accept.try(:created_at)
-        case options[:role].to_sym
-        when :Administrator, :Librarian
-          item_lines << i.bookstore.try(:name)
-          item_lines << i.budget_type.try(:name)
-          if defined?(EnjuCirculation)
-            item_lines << Checkout.where(item_id: i.id).count
-          else
-            item_lines << ''
-          end
-        end
-        if defined?(EnjuCirculation)
-          item_lines << i.circulation_status.try(:name)
-        else
-          item_lines << ''
-        end
-        item_lines << i.shelf.name
-        item_lines << i.shelf.library.name
-        item_lines << i.created_at
-        item_lines << i.updated_at
-        case options[:role].to_sym
-        when :Administrator, :Librarian
-          if defined?(EnjuCirculation)
-            item_lines << i.use_restriction.try(:name)
-          else
-            item_lines << ''
-          end
-        end
-        lines << item_lines
-      end
-    else
-      line = []
-      line << id
-      line << original_title
-      line << title_transcription
-      if creators.exists?
-        line << creators.pluck(:full_name).join("//")
-      else
-        line << nil
-      end
-      if contributors.exists?
-        line << contributors.pluck(:full_name).join("//")
-      else
-        line << nil
-      end
-      if publishers.exists?
-        line << publishers.pluck(:full_name).join("//")
-      else
-        line << nil
-      end
-      line << pub_date
-      line << statement_of_responsibility
-      line << price
-      line << created_at
-      line << updated_at
-      line << manifestation_identifier
-      line << access_address
-      line << description.try(:gsub, /\r?\n/, '\n')
-      line << note.try(:gsub, /\r?\n/, '\n')
-      line << extent
-      line << dimensions
-      line << carrier_type.name
-      line << edition
-      line << edition_string
-      line << volume_number
-      line << volume_number_string
-      line << issue_number
-      line << issue_number_string
-      line << serial_number
-
-      IdentifierType.order(:position).pluck(:name).each do |identifier_type|
-        identifier_list = identifier_contents(identifier_type.to_sym)
-        if identifier_list
-          line << identifier_list.join("//")
         else
-          line << nil
-        end
-      end
-      if defined?(EnjuSubject)
-        SubjectHeadingType.order(:position).each do |subject_heading_type|
-          if subjects.exists?
-            line << subjects.where(subject_heading_type: subject_heading_type).pluck(:term).join('//')
-          else
-            line << nil
-          end
-        end
-        ClassificationType.order(:position).each do |classification_type|
-          if classifications.exists?
-            line << classifications.where(classification_type: classification_type).pluck(:category).join('//')
-          else
-            line << nil
-          end
+          f.write manifestation.to_hash(role: role).values.to_csv(col_sep: col_sep)
         end
       end
 
-      lines << line
-    end
-
-    if options[:format] == :txt
-      lines.map{|i| i.to_csv(col_sep: "\t")}.join
-    else
-      lines
+      f.rewind
+      f.read
     end
-  end
 
-  def self.export(options = {format: :txt, role: :Guest})
-    file = ''
-    file += Manifestation.csv_header(options[:role], col_sep: "\t") if options[:format].to_sym == :txt
-    Manifestation.find_each do |manifestation|
-      file += manifestation.to_csv(options)
-    end
     file
   end
 
@@ -872,4 +736,5 @@ end
 #  publication_place               :text
 #  extent                          :text
 #  dimensions                      :text
+#  memo                            :text
 #