enju_biblio 0.3.10 → 0.4.0.beta.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 3d531d406b5fda2fe6871aacfff5e40ee50b4b9e705011b48194043636435d4f
-  data.tar.gz: 254254c68a6ad93a6e05ca9db4bcc38dcb5abd6ba100cdb26dd9f1d6cc8a3ee1
+  metadata.gz: 25f3dd27c99737a4cfbba700c0963f4352b7e237bf30bdea0bf9c446208906be
+  data.tar.gz: 544ee9521de65fb2cc918332e4c15ca88a89b346a021f7ca9a0bdabdb66841d5
 SHA512:
-  metadata.gz: ab53272b3a84a1970be279892113c5296ca7eb3ee5fa8058d807b716737597dabe1d4a087e1abfed6763853920a922096ba0d932de6e2645bb3dfb96040e3f50
-  data.tar.gz: 9967b069e6d50dd77c0ed2f1d42c06d574d1138164e569043fb00861bd4b6246105288f052fb0351993a60531e87f9c79266def019ac45e749f548caf1162e02
+  metadata.gz: 139f55a88e3958bfe75f687fac126219a964d15485bde0e66e264f54244cd550c4b44d284130582026090267de93693fbdcbaf2cf117166e34c0b2c62ccddc54
+  data.tar.gz: 895a8713dea2a69800fd2aa0edf8842ef20f9c9ebae86c92aff7d6c014fb5f5f90f1e689471f13918ed671f304f09fa933027bf7752bcac765aed7c3b209f28c

data/README.md

@@ -1,7 +1,6 @@
 # EnjuBiblio
-[![Travis CI](https://travis-ci.com/next-l/enju_biblio.svg?branch=1.3)](https://travis-ci.com/next-l/enju_biblio)
-[![Coverage Status](https://coveralls.io/repos/github/next-l/enju_biblio/badge.svg?branch=1.3)](https://coveralls.io/github/next-l/enju_biblio?branch=1.3)
-[![security](https://hakiri.io/github/next-l/enju_biblio/1.3.svg)](https://hakiri.io/github/next-l/enju_biblio/1.3)
+[![CircleCI](https://circleci.com/gh/next-l/enju_biblio/tree/2.x.svg?style=svg)](https://circleci.com/gh/next-l/enju_biblio/tree/2.x)
+[![Coverage Status](https://coveralls.io/repos/github/next-l/enju_biblio/badge.svg?branch=master)](https://coveralls.io/github/next-l/enju_biblio?branch=master)
 
 This project rocks and uses MIT-LICENSE.
 

data/app/controllers/content_types_controller.rb

@@ -5,7 +5,7 @@ class ContentTypesController < ApplicationController
   # GET /content_types
   # GET /content_types.json
   def index
-    @content_types = ContentType.order(:position)
+    @content_types = ContentType.all
 
     respond_to do |format|
       format.html # index.html.erb

data/app/controllers/items_controller.rb

@@ -272,11 +272,8 @@ class ItemsController < ApplicationController
       :acquired_at, :bookstore_id, :missing_since, :budget_type_id,
       :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_custom_values_attributes: [
-        :id, :item_custom_property_id, :item_id, :value,:_destroy
-      ]}
+      :use_restriction_id,
+      item_has_use_restriction_attributes: :use_restriction_id # EnjuCirculation
     )
   end
 

data/app/controllers/manifestations_controller.rb

@@ -332,7 +332,7 @@ class ManifestationsController < ApplicationController
       }
       format.rdf
       format.mods
-      format.json
+      format.json { render json: @manifestation }
       format.txt
       format.js
       format.download {
@@ -355,8 +355,8 @@ class ManifestationsController < ApplicationController
   # GET /manifestations/new.json
   def new
     @manifestation = Manifestation.new
-    @manifestation.language = Language.find_by(iso_639_1: @locale)
-    @parent = Manifestation.find_by(id: params[:parent_id]) if params[:parent_id].present?
+    @manifestation.language = Language.where(iso_639_1: @locale).first
+    @parent = Manifestation.where(id: params[:parent_id]).first if params[:parent_id].present?
     if @parent
       @manifestation.parent_id = @parent.id
       [ :original_title, :title_transcription,
@@ -429,17 +429,15 @@ class ManifestationsController < ApplicationController
   # 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.transaction do
+      @manifestation.update(manifestation_params.delete_if{|k, v|
+        k == 'creators_attributes'
+      })
+      @manifestation.creators = Agent.new_agents(creators_params)
+    end
 
     respond_to do |format|
       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
@@ -491,7 +489,7 @@ class ManifestationsController < ApplicationController
       :valid_until, :date_submitted, :date_accepted, :date_captured,
       :ndl_bib_id, :pub_date, :edition_string, :volume_number, :issue_number,
       :serial_number, :content_type_id, :attachment, :lock_version,
-      :dimensions, :fulltext_content, :extent, :memo,
+      :dimensions, :fulltext_content, :extent,
       :parent_id,
       :serial, :statement_of_responsibility,
       {creators_attributes: [
@@ -549,9 +547,6 @@ class ManifestationsController < ApplicationController
       {identifiers_attributes: [
         :id, :body, :identifier_type_id,
         :_destroy
-      ]},
-      {manifestation_custom_values_attributes: [
-        :id, :manifestation_custom_property_id, :manifestation_id, :value,:_destroy
       ]}
     )
   end
@@ -737,12 +732,12 @@ class ManifestationsController < ApplicationController
   end
 
   def prepare_options
-    @carrier_types = CarrierType.order(:position).select([:id, :display_name, :position])
-    @content_types = ContentType.order(:position).select([:id, :display_name, :position])
-    @roles = Role.select([:id, :display_name, :position])
-    @languages = Language.order(:position).select([:id, :display_name, :position])
-    @frequencies = Frequency.order(:position).select([:id, :display_name, :position])
-    @identifier_types = IdentifierType.order(:position).select([:id, :display_name, :position])
+    @carrier_types = CarrierType.select([:id, :display_name_translations, :position])
+    @content_types = ContentType.select([:id, :display_name_translations, :position])
+    @roles = Role.select([:id, :display_name_translations, :position])
+    @languages = Language.select([:id, :display_name, :position])
+    @frequencies = Frequency.select([:id, :display_name_translations, :position])
+    @identifier_types = IdentifierType.select([:id, :display_name, :position])
     @nii_types = NiiType.select([:id, :display_name, :position]) if defined?(EnjuNii)
     if defined?(EnjuSubject)
       @subject_types = SubjectType.select([:id, :display_name, :position])
@@ -782,7 +777,7 @@ class ManifestationsController < ApplicationController
     if options[:pub_date_from].blank?
       pub_date[:from] = "*"
     else
-      year = options[:pub_date_from].to_s.gsub(/\D/, '').rjust(4, "0")
+      year = options[:pub_date_from].rjust(4, "0")
       if year.length == 4
         pub_date[:from] = Time.zone.parse(Time.utc(year).to_s).beginning_of_year.utc.iso8601
       else
@@ -796,7 +791,7 @@ class ManifestationsController < ApplicationController
     if options[:pub_date_until].blank?
       pub_date[:until] = "*"
     else
-      year = options[:pub_date_until].to_s.gsub(/\D/, '').rjust(4, "0")
+      year = options[:pub_date_until].rjust(4, "0")
       if year.length == 4
         pub_date[:until] = Time.zone.parse(Time.utc(year).to_s).end_of_year.utc.iso8601
       else
@@ -811,6 +806,8 @@ class ManifestationsController < ApplicationController
 
   def set_pub_date(query, options)
     unless options[:pub_date_from].blank? && options[:pub_date_until].blank?
+      options[:pub_date_from].to_s.gsub!(/\D/, '')
+      options[:pub_date_until].to_s.gsub!(/\D/, '')
       pub_date = parse_pub_date(options)
       query = "#{query} date_of_publication_d:[#{pub_date[:from]} TO #{pub_date[:until]}]"
     end

data/app/controllers/resource_import_results_controller.rb

@@ -5,13 +5,9 @@ class ResourceImportResultsController < ApplicationController
   # GET /resource_import_results
   # GET /resource_import_results.json
   def index
-    @resource_import_file = ResourceImportFile.find_by(id: params[:resource_import_file_id])
+    @resource_import_file = ResourceImportFile.where(id: params[:resource_import_file_id]).first
     if @resource_import_file
-      if params[:format].to_s.downcase == 'txt'
-        @resource_import_results = @resource_import_file.resource_import_results
-      else
-        @resource_import_results = @resource_import_file.resource_import_results.page(params[:page])
-      end
+      @resource_import_results = @resource_import_file.resource_import_results.page(params[:page])
     else
       @resource_import_results = ResourceImportResult.page(params[:page])
     end

data/app/helpers/enju_biblio/application_helper.rb

@@ -1,7 +1,7 @@
 module EnjuBiblio
   module ApplicationHelper
     def form_icon(carrier_type)
-      image_tag(carrier_type_path(carrier_type, format: :download), size: '16x16', class: 'enju_icon', alt: carrier_type.display_name.localize)
+      image_tag(carrier_type_path(carrier_type, format: :download), size: '16x16', class: 'enju_icon', alt: carrier_type.display_name)
     rescue NoMethodError
       image_tag('icons/help.png', size: '16x16', class: 'enju_icon', alt: t('page.unknown'))
     end
@@ -9,13 +9,13 @@ module EnjuBiblio
     def content_type_icon(content_type)
       case content_type.name
       when 'text'
-        image_tag('icons/page_white_text.png', size: '16x16', class: 'enju_icon', alt: content_type.display_name.localize)
+        image_tag('icons/page_white_text.png', size: '16x16', class: 'enju_icon', alt: content_type.display_name)
       when 'still_image'
-        image_tag('icons/picture.png', size: '16x16', class: 'enju_icon', alt: content_type.display_name.localize)
+        image_tag('icons/picture.png', size: '16x16', class: 'enju_icon', alt: content_type.display_name)
       when 'sounds'
-        image_tag('icons/sound.png', size: '16x16', class: 'enju_icon', alt: content_type.display_name.localize)
+        image_tag('icons/sound.png', size: '16x16', class: 'enju_icon', alt: content_type.display_name)
       when 'two_dimensional_moving_image'
-        image_tag('icons/film.png', size: '16x16', class: 'enju_icon', alt: content_type.display_name.localize)
+        image_tag('icons/film.png', size: '16x16', class: 'enju_icon', alt: content_type.display_name)
       else
         image_tag('icons/help.png', size: '16x16', class: 'enju_icon', alt: t('page.unknown'))
       end

data/app/helpers/manifestations_helper.rb

@@ -2,7 +2,7 @@ module ManifestationsHelper
   include EnjuCirculation::ManifestationsHelper if defined?(EnjuCirculation)
 
   def resource_title(manifestation, action)
-    string = LibraryGroup.site_config.display_name.localize.dup
+    string = LibraryGroup.site_config.display_name.dup
     unless action == ('index' or 'new')
       if manifestation.try(:original_title)
         string << ' - ' + manifestation.original_title.to_s
@@ -40,7 +40,7 @@ module ManifestationsHelper
   def language_list(languages)
     list = []
     languages.each do |language|
-      list << language.display_name.localize if language.name != 'unknown'
+      list << language.display_name if language.name != 'unknown'
     end
     list.join("; ")
   end
@@ -71,41 +71,41 @@ 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} (" + facet.count.to_s + ")", url_for(filtered_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} (" + facet.count.to_s + ")", url_for(filtered_params.merge(page: nil, language: language.name, view: nil, only_path: true)))
     end
   end
 
   def library_facet(current_libraries, facet)
-    library = Library.where(name: facet.value).select([:name, :display_name]).first
+    library = Library.where(name: facet.value).select([:name, :display_name_translations]).first
     return nil unless library
     string = ''
     current = true if current_libraries.include?(library.name)
     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} (" + facet.count.to_s + ")", url_for(filtered_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} (" + facet.count.to_s + ")", url_for(filtered_params.merge(page: nil, library: (current_libraries << library.name).uniq.join(' '), view: nil, only_path: true)))
       end
     end
   end
 
   def carrier_type_facet(facet)
     string = ''
-    carrier_type = CarrierType.where(name: facet.value).select([:id, :name, :display_name]).first
+    carrier_type = CarrierType.where(name: facet.value).select([:id, :name, :display_name_translations]).first
     if carrier_type
       string << form_icon(carrier_type)
       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} (" + facet.count.to_s + ")", url_for(filtered_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} (" + facet.count.to_s + ")", url_for(filtered_params.merge(carrier_type: carrier_type.name, page: nil, view: nil, only_path: true)))
       end
     end
   end

data/app/models/agent.rb

@@ -17,7 +17,7 @@ class Agent < ApplicationRecord
   has_many :derived_agents, through: :children, source: :child
   has_many :original_agents, through: :parents, source: :parent
   has_many :picture_files, as: :picture_attachable, dependent: :destroy
-  has_many :donates, dependent: :destroy
+  has_many :donates
   has_many :donated_items, through: :donates, source: :item
   has_many :owns, dependent: :destroy
   has_many :items, through: :owns
@@ -97,7 +97,7 @@ class Agent < ApplicationRecord
   def set_date_of_birth
     return if birth_date.blank?
     begin
-      date = Time.zone.parse(birth_date.to_s)
+      date = Time.zone.parse("#{birth_date}")
     rescue ArgumentError
       begin
         date = Time.zone.parse("#{birth_date}-01")
@@ -115,7 +115,7 @@ class Agent < ApplicationRecord
   def set_date_of_death
     return if death_date.blank?
     begin
-      date = Time.zone.parse(death_date.to_s)
+      date = Time.zone.parse("#{death_date}")
     rescue ArgumentError
       begin
         date = Time.zone.parse("#{death_date}-01")

data/app/models/agent_import_file.rb

@@ -151,7 +151,7 @@ class AgentImportFile < ApplicationRecord
     rows.each do |row|
       row_num += 1
       next if row['dummy'].to_s.strip.present?
-      agent = Agent.find_by(id: row['id'].to_s.strip)
+      agent = Agent.where(id: row['id'].to_s.strip).first
       if agent
         agent.picture_files.destroy_all
         agent.reload

data/app/models/agent_relationship_type.rb

@@ -8,11 +8,12 @@ end
 #
 # Table name: agent_relationship_types
 #
-#  id           :integer          not null, primary key
-#  name         :string           not null
-#  display_name :text
-#  note         :text
-#  position     :integer
-#  created_at   :datetime
-#  updated_at   :datetime
+#  id                        :integer          not null, primary key
+#  name                      :string           not null
+#  display_name              :text
+#  note                      :text
+#  position                  :integer
+#  created_at                :datetime
+#  updated_at                :datetime
+#  display_name_translations :jsonb            not null
 #

data/app/models/agent_type.rb

@@ -1,5 +1,7 @@
 class AgentType < ApplicationRecord
   include MasterModel
+  default_scope { order('agent_types.position') }
+  translates :display_name
   has_many :agents
 end
 
@@ -7,11 +9,12 @@ end
 #
 # Table name: agent_types
 #
-#  id           :integer          not null, primary key
-#  name         :string           not null
-#  display_name :text
-#  note         :text
-#  position     :integer
-#  created_at   :datetime
-#  updated_at   :datetime
+#  id                        :integer          not null, primary key
+#  name                      :string           not null
+#  display_name              :text
+#  note                      :text
+#  position                  :integer
+#  created_at                :datetime
+#  updated_at                :datetime
+#  display_name_translations :jsonb            not null
 #

data/app/models/carrier_type.rb

@@ -1,5 +1,7 @@
 class CarrierType < ApplicationRecord
   include MasterModel
+  default_scope { order("carrier_types.position") }
+  translates :display_name
   has_many :manifestations
   if ENV['ENJU_STORAGE'] == 's3'
     has_attached_file :attachment, storage: :s3,
@@ -34,15 +36,16 @@ end
 #
 # Table name: carrier_types
 #
-#  id                      :integer          not null, primary key
-#  name                    :string           not null
-#  display_name            :text
-#  note                    :text
-#  position                :integer
-#  created_at              :datetime
-#  updated_at              :datetime
-#  attachment_file_name    :string
-#  attachment_content_type :string
-#  attachment_file_size    :integer
-#  attachment_updated_at   :datetime
+#  id                        :integer          not null, primary key
+#  name                      :string           not null
+#  display_name              :text
+#  note                      :text
+#  position                  :integer
+#  created_at                :datetime
+#  updated_at                :datetime
+#  attachment_file_name      :string
+#  attachment_content_type   :string
+#  attachment_file_size      :bigint
+#  attachment_updated_at     :datetime
+#  display_name_translations :jsonb            not null
 #

data/app/models/content_type.rb

@@ -1,5 +1,7 @@
 class ContentType < ApplicationRecord
   include MasterModel
+  default_scope { order('content_types.position') }
+  translates :display_name
   has_many :manifestations
 end
 
@@ -7,11 +9,12 @@ end
 #
 # Table name: content_types
 #
-#  id           :integer          not null, primary key
-#  name         :string           not null
-#  display_name :text
-#  note         :text
-#  position     :integer
-#  created_at   :datetime
-#  updated_at   :datetime
+#  id                        :integer          not null, primary key
+#  name                      :string           not null
+#  display_name              :text
+#  note                      :text
+#  position                  :integer
+#  created_at                :datetime
+#  updated_at                :datetime
+#  display_name_translations :jsonb            not null
 #

data/app/models/create_type.rb

@@ -7,11 +7,12 @@ end
 #
 # Table name: create_types
 #
-#  id           :integer          not null, primary key
-#  name         :string
-#  display_name :text
-#  note         :text
-#  position     :integer
-#  created_at   :datetime
-#  updated_at   :datetime
+#  id                        :integer          not null, primary key
+#  name                      :string
+#  display_name              :text
+#  note                      :text
+#  position                  :integer
+#  created_at                :datetime
+#  updated_at                :datetime
+#  display_name_translations :jsonb            not null
 #

data/app/models/doi_record.rb

@@ -0,0 +1,36 @@
+class DoiRecord < ApplicationRecord
+  belongs_to :manifestation
+  validates :body, presence: true, uniqueness: true, format: {with: /\A[0-9]{2}\.[0-9]{2,}\/.+\Z/}
+  before_save :normalize
+  before_save :set_display_body
+
+  strip_attributes
+
+  def normalize
+    url = URI.parse(body)
+    if url.host =~ /doi\.org\Z/
+      self.body = url.path.gsub(/\A\//, '').downcase
+    else
+      self.body = body.downcase
+    end
+  rescue URI::InvalidURIError
+  end
+
+  def set_display_body
+    self.display_body = body unless display_body
+  end
+end
+
+# == Schema Information
+#
+# Table name: doi_records
+#
+#  id               :bigint           not null, primary key
+#  body             :string           not null
+#  display_body     :string           not null
+#  source           :string
+#  response         :jsonb            not null
+#  manifestation_id :bigint           not null
+#  created_at       :datetime         not null
+#  updated_at       :datetime         not null
+#