enju_manifestation_viewer 0.3.0.beta.1 → 0.3.0.beta.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 71cf47e65b6284135a40ae1576726d518591a285
-  data.tar.gz: ce8719da4c69cd74aa02723519f453e2a86071d3
+  metadata.gz: d5f0bbe14880e5173dbc23fa6c25cd570cbff398
+  data.tar.gz: f070c89e7a503e71a96714656142a61e00be2947
 SHA512:
-  metadata.gz: 75ed2ab370757cafb67b352c4f96d53650ab946c7358c754ca7ec9841b373a2364ca006a6f7af95f73f850a9c1b3879199c34d6c04a9f7376dc6440a400ae213
-  data.tar.gz: fd6cf0b7348ee3090a1d79b064b303d5c7dcc771edf04a663d294b1fcd29c3f0fd37fc7049fe9899c556bdd1bd0376e83ee83e2d3c7d10ba34ed3c4cc58ab9f8
+  metadata.gz: 6ee64bf311eb0b123ed91f4a9af908fa1c5be5ec430e7bb9124e029f96a8a5c3d78a756b48433547122a95b0d015595e8d4f37d33eb3d86f673c6b841e291290
+  data.tar.gz: 0afe868b9701a6bf98b4d171c9f229515d808fe9ac27396d60f21baa267434fc7632df699e192e97636cd18dad598e7b0a36a790047aa186c4f96c33f721c133

data/app/helpers/enju_manifestation_viewer/application_helper.rb

@@ -1,19 +1,18 @@
 module EnjuManifestationViewer
   module ApplicationHelper
     def google_book_search_preview(isbn)
-      render :partial => 'manifestations/google_book_search', :locals => {:isbn => isbn}
+      render partial: 'manifestations/google_book_search', locals: {isbn: isbn}
     end
 
     def embed_content(manifestation)
-      case
-      when manifestation.youtube_id
-        render :partial => 'manifestations/youtube', :locals => {:manifestation => manifestation}
-      when manifestation.nicovideo_id
-        render :partial => 'manifestations/nicovideo', :locals => {:manifestation => manifestation}
-      when manifestation.flickr.present?
-        render :partial => 'manifestations/flickr', :locals => {:manifestation => manifestation}
-      when manifestation.respond_to?(:ipaper_id)
-        render :partial => 'manifestations/scribd', :locals => {:manifestation => manifestation}
+      if manifestation.youtube_id
+        render partial: 'manifestations/youtube', locals: {manifestation: manifestation}
+      elsif manifestation.nicovideo_id
+        render partial: 'manifestations/nicovideo', locals: {manifestation: manifestation}
+      elsif manifestation.flickr.present?
+        render partial: 'manifestations/flickr', locals: {manifestation: manifestation}
+      elsif manifestation.respond_to?(:ipaper_id)
+        render partial: 'manifestations/scribd', locals: {manifestation: manifestation}
       end
     end
   end

data/app/helpers/enju_manifestation_viewer/book_jacket_helper.rb

@@ -4,30 +4,27 @@ module EnjuManifestationViewer
       if manifestation.picture_files.exists?
         link = ''
         manifestation.picture_files.each_with_index do |picture_file, i|
-          if i == 0
-            link += link_to(show_image(picture_file, size: :thumb), picture_file_path(picture_file, format: :download), rel: "manifestation_#{manifestation.id}")
-          else
-            link += content_tag :span, style: "display: none" do
-              link_to(show_image(picture_file, size: :thumb), picture_file_path(picture_file, format: :download), rel: "manifestation_#{manifestation.id}")
-            end
-          end
+          link += if i == 0
+                    link_to(show_image(picture_file, size: :thumb), picture_file_path(picture_file, format: :download), rel: "manifestation_#{manifestation.id}")
+                  else
+                    content_tag :span, style: "display: none" do
+                      link_to(show_image(picture_file, size: :thumb), picture_file_path(picture_file, format: :download), rel: "manifestation_#{manifestation.id}")
+                    end
+                  end
         end
         return link.html_safe
       else
         link = book_jacket_tag(manifestation)
-        unless link
-          link = screenshot_tag(manifestation)
-        end
+        link ||= screenshot_tag(manifestation)
       end
 
-      unless link
-        link = link_to image_tag('unknown_resource.png', width: '100', height: '100', alt: '*', itemprop: 'image'), manifestation
-      end
+      link ||= link_to image_tag('unknown_resource.png', width: '100', height: '100', alt: '*', itemprop: 'image'), manifestation
       link
     end
 
     def screenshot_tag(manifestation, generator = LibraryGroup.site_config.screenshot_generator)
       return nil unless manifestation.try(:access_address)
+
       case generator
       when "mozshot"
         link_to image_tag("http://mozshot.nemui.org/shot?#{manifestation.access_address}", width: 128, height: 128, alt: manifestation.original_title, border: 0, itemprop: 'image'), manifestation.access_address
@@ -42,9 +39,11 @@ module EnjuManifestationViewer
 
     def book_jacket_tag(manifestation, generator = LibraryGroup.site_config.book_jacket_source)
       return nil unless manifestation
+
       case generator
       when "amazon"
         return nil unless LibraryGroup.site_config.settings[:amazon_hostname]
+
         book_jacket = manifestation.amazon_book_jacket
         if book_jacket
           link_to image_tag(book_jacket[:url], width: book_jacket[:width], height: book_jacket[:height], alt: manifestation.original_title, class: 'book_jacket', itemprop: 'image'), "http://#{LibraryGroup.site_config.settings[:amazon_hostname]}/dp/#{book_jacket[:asin]}"
@@ -60,6 +59,7 @@ module EnjuManifestationViewer
 
     def amazon_link(asin, hostname = LibraryGroup.site_config.settings[:amazon_hostname])
       return nil if asin.blank?
+
       "http://#{hostname}/dp/#{asin}"
     end
 

data/app/helpers/enju_manifestation_viewer/manifestation_viewer_helper.rb

@@ -0,0 +1,5 @@
+module EnjuManifestationViewer
+  module ManifestationViewerHelper
+    extend ApplicationHelper
+  end
+end

data/lib/enju_manifestation_viewer.rb

@@ -16,7 +16,7 @@ module EnjuManifestationViewer
     def youtube_id
       if access_address
         url = ::Addressable::URI.parse(access_address)
-        if url.host =~ /youtube\.com$/ and url.path == "/watch"
+        if url.host =~ /youtube\.com$/ && (url.path == "/watch")
           return CGI.parse(url.query)["v"][0]
         end
       end
@@ -25,7 +25,7 @@ module EnjuManifestationViewer
     def nicovideo_id
       if access_address
         url = ::Addressable::URI.parse(access_address)
-        if url.host =~ /nicovideo\.jp$/ and url.path =~ /^\/watch/
+        if url.host =~ /nicovideo\.jp$/ && url.path =~ /^\/watch/
           return url.path.split("/")[2]
         end
       end
@@ -35,7 +35,7 @@ module EnjuManifestationViewer
       if access_address
         url = ::Addressable::URI.parse(access_address)
         paths = url.path.split('/')
-        if url.host =~ /^www\.flickr\.com$/ and paths[1] == 'photos' and paths[2]
+        if url.host =~ /^www\.flickr\.com$/ && (paths[1] == 'photos') && paths[2]
           info = {}
           if paths[3] == "sets"
             info[:user] = paths[2]

data/lib/enju_manifestation_viewer/version.rb

@@ -1,3 +1,3 @@
 module EnjuManifestationViewer
-  VERSION = "0.3.0.beta.1"
+  VERSION = "0.3.0.beta.2"
 end

data/spec/dummy/db/migrate/001_create_agents.rb

@@ -32,13 +32,13 @@ class CreateAgents < ActiveRecord::Migration[4.2]
       t.text :region
       t.datetime :date_of_birth
       t.datetime :date_of_death
-      t.integer :language_id, :default => 1, :null => false
-      t.integer :country_id, :default => 1, :null => false
-      t.integer :agent_type_id, :default => 1, :null => false
-      t.integer :lock_version, :default => 0, :null => false
+      t.integer :language_id, default: 1, null: false
+      t.integer :country_id, default: 1, null: false
+      t.integer :agent_type_id, default: 1, null: false
+      t.integer :lock_version, default: 0, null: false
       t.text :note
-      t.integer :required_role_id, :default => 1, :null => false
-      t.integer :required_score, :default => 0, :null => false
+      t.integer :required_role_id, default: 1, null: false
+      t.integer :required_score, default: 0, null: false
       t.text :email
       t.text :url
     end

data/spec/dummy/db/migrate/002_devise_create_users.rb

@@ -2,8 +2,8 @@ class DeviseCreateUsers < ActiveRecord::Migration[4.2]
   def change
     create_table(:users) do |t|
       ## Database authenticatable
-      t.string :email,              :null => false, :default => ""
-      t.string :encrypted_password, :null => false, :default => ""
+      t.string :email,              null: false, default: ""
+      t.string :encrypted_password, null: false, default: ""
 
       ## Recoverable
       t.string   :reset_password_token
@@ -13,7 +13,7 @@ class DeviseCreateUsers < ActiveRecord::Migration[4.2]
       t.datetime :remember_created_at
 
       ## Trackable
-      t.integer  :sign_in_count, :default => 0
+      t.integer  :sign_in_count, default: 0
       t.datetime :current_sign_in_at
       t.datetime :last_sign_in_at
       t.string   :current_sign_in_ip
@@ -33,12 +33,11 @@ class DeviseCreateUsers < ActiveRecord::Migration[4.2]
       ## Token authenticatable
       # t.string :authentication_token
 
-
       t.timestamps
     end
 
-    add_index :users, :email,                :unique => true
-    add_index :users, :reset_password_token, :unique => true
+    add_index :users, :email,                unique: true
+    add_index :users, :reset_password_token, unique: true
     # add_index :users, :confirmation_token,   :unique => true
     # add_index :users, :unlock_token,         :unique => true
     # add_index :users, :authentication_token, :unique => true

data/spec/dummy/db/migrate/005_create_manifestations.rb

@@ -1,7 +1,7 @@
 class CreateManifestations < ActiveRecord::Migration[4.2]
   def change
     create_table :manifestations do |t|
-      t.text :original_title, :null => false
+      t.text :original_title, null: false
       t.text :title_alternative
       t.text :title_transcription
       t.string :classification_number
@@ -11,8 +11,8 @@ class CreateManifestations < ActiveRecord::Migration[4.2]
       t.timestamps
       t.datetime :deleted_at
       t.string :access_address
-      t.integer :language_id, :default => 1, :null => false
-      t.integer :carrier_type_id, :default => 1, :null => false
+      t.integer :language_id, default: 1, null: false
+      t.integer :carrier_type_id, default: 1, null: false
       t.integer :start_page
       t.integer :end_page
       t.integer :height
@@ -25,17 +25,17 @@ class CreateManifestations < ActiveRecord::Migration[4.2]
       t.string :serial_number_string
       t.integer :edition
       t.text :note
-      t.boolean :repository_content, :default => false, :null => false
-      t.integer :lock_version, :default => 0, :null => false
-      t.integer :required_role_id, :default => 1, :null => false
-      t.integer :required_score, :default => 0, :null => false
-      t.integer :frequency_id, :default => 1, :null => false
-      t.boolean :subscription_master, :default => false, :null => false
+      t.boolean :repository_content, default: false, null: false
+      t.integer :lock_version, default: 0, null: false
+      t.integer :required_role_id, default: 1, null: false
+      t.integer :required_score, default: 0, null: false
+      t.integer :frequency_id, default: 1, null: false
+      t.boolean :subscription_master, default: false, null: false
     end
-    #add_index :manifestations, :carrier_type_id
-    #add_index :manifestations, :required_role_id
+    # add_index :manifestations, :carrier_type_id
+    # add_index :manifestations, :required_role_id
     add_index :manifestations, :access_address
-    #add_index :manifestations, :frequency_id
+    # add_index :manifestations, :frequency_id
     add_index :manifestations, :manifestation_identifier
     add_index :manifestations, :updated_at
     add_index :manifestations, :date_of_publication

data/spec/dummy/db/migrate/006_create_items.rb

@@ -5,14 +5,14 @@ class CreateItems < ActiveRecord::Migration[4.2]
       t.string :item_identifier
       t.timestamps
       t.datetime :deleted_at
-      t.integer :shelf_id, :default => 1, :null => false
-      t.boolean :include_supplements, :default => false, :null => false
+      t.integer :shelf_id, default: 1, null: false
+      t.boolean :include_supplements, default: false, null: false
       t.text :note
       t.string :url
       t.integer :price
-      t.integer :lock_version, :default => 0, :null => false
-      t.integer :required_role_id, :default => 1, :null => false
-      t.integer :required_score, :default => 0, :null => false
+      t.integer :lock_version, default: 0, null: false
+      t.integer :required_role_id, default: 1, null: false
+      t.integer :required_score, default: 0, null: false
     end
     add_index :items, :shelf_id
     add_index :items, :item_identifier

data/spec/dummy/db/migrate/012_create_owns.rb

@@ -1,8 +1,8 @@
 class CreateOwns < ActiveRecord::Migration[4.2]
   def change
     create_table :owns do |t|
-      t.references :agent, :null => false
-      t.references :item, :null => false
+      t.references :agent, null: false
+      t.references :item, null: false
       t.integer :position
       t.timestamps
     end

data/spec/dummy/db/migrate/015_create_creates.rb

@@ -1,8 +1,8 @@
 class CreateCreates < ActiveRecord::Migration[4.2]
   def change
     create_table :creates do |t|
-      t.references :agent, :null => false
-      t.references :work, :null => false
+      t.references :agent, null: false
+      t.references :work, null: false
       t.integer :position
       t.timestamps
     end

data/spec/dummy/db/migrate/041_create_roles.rb

@@ -1,12 +1,12 @@
 class CreateRoles < ActiveRecord::Migration[4.2]
   def change
     create_table "roles" do |t|
-      t.column :name, :string, :null => false
+      t.column :name, :string, null: false
       t.column :display_name, :string
       t.column :note, :text
       t.column :created_at, :datetime
       t.column :updated_at, :datetime
-      t.integer :score, :default => 0, :null => false
+      t.integer :score, default: 0, null: false
       t.integer :position
     end
   end

data/spec/dummy/db/migrate/047_create_produces.rb

@@ -1,8 +1,8 @@
 class CreateProduces < ActiveRecord::Migration[4.2]
   def change
     create_table :produces do |t|
-      t.references :agent, :null => false
-      t.references :manifestation, :null => false
+      t.references :agent, null: false
+      t.references :manifestation, null: false
       t.integer :position
       t.timestamps
     end

data/spec/dummy/db/migrate/073_create_carrier_types.rb

@@ -1,7 +1,7 @@
 class CreateCarrierTypes < ActiveRecord::Migration[4.2]
   def change
     create_table :carrier_types do |t|
-      t.string :name, :null => false
+      t.string :name, null: false
       t.text :display_name
       t.text :note
       t.integer :position

data/spec/dummy/db/migrate/077_create_user_groups.rb

@@ -1,7 +1,7 @@
 class CreateUserGroups < ActiveRecord::Migration[4.2]
   def change
     create_table :user_groups do |t|
-      t.string :name, :not_null => true
+      t.string :name, not_null: true
       t.text :display_name
       t.text :note
       t.integer :position

data/spec/dummy/db/migrate/112_create_frequencies.rb

@@ -1,7 +1,7 @@
 class CreateFrequencies < ActiveRecord::Migration[4.2]
   def change
     create_table :frequencies do |t|
-      t.string :name, :null => false
+      t.string :name, null: false
       t.text :display_name
       t.text :note
       t.integer :position

data/spec/dummy/db/migrate/117_create_form_of_works.rb

@@ -1,7 +1,7 @@
 class CreateFormOfWorks < ActiveRecord::Migration[4.2]
   def change
     create_table :form_of_works do |t|
-      t.string :name, :null => false
+      t.string :name, null: false
       t.text :display_name
       t.text :note
       t.integer :position

data/spec/dummy/db/migrate/124_create_bookstores.rb

@@ -1,7 +1,7 @@
 class CreateBookstores < ActiveRecord::Migration[4.2]
   def change
     create_table :bookstores do |t|
-      t.text :name, :null => false
+      t.text :name, null: false
       t.string :zip_code
       t.text :address
       t.text :note

data/spec/dummy/db/migrate/125_create_donates.rb

@@ -1,8 +1,8 @@
 class CreateDonates < ActiveRecord::Migration[4.2]
   def change
     create_table :donates do |t|
-      t.integer :agent_id, :null => false
-      t.integer :item_id, :null => false
+      t.integer :agent_id, null: false
+      t.integer :item_id, null: false
 
       t.timestamps
     end

data/spec/dummy/db/migrate/130_create_request_status_types.rb

@@ -1,7 +1,7 @@
 class CreateRequestStatusTypes < ActiveRecord::Migration[4.2]
   def change
     create_table :request_status_types do |t|
-      t.string :name, :null => false
+      t.string :name, null: false
       t.text :display_name
       t.text :note
       t.integer :position

data/spec/dummy/db/migrate/131_create_request_types.rb

@@ -1,7 +1,7 @@
 class CreateRequestTypes < ActiveRecord::Migration[4.2]
   def change
     create_table :request_types do |t|
-      t.string :name, :null => false
+      t.string :name, null: false
       t.text :display_name
       t.text :note
       t.integer :position

data/spec/dummy/db/migrate/133_create_agent_merges.rb

@@ -1,7 +1,7 @@
 class CreateAgentMerges < ActiveRecord::Migration[4.2]
   def self.up
     create_table :agent_merges do |t|
-      t.integer :agent_id, :agent_merge_list_id, :null => false
+      t.integer :agent_id, :agent_merge_list_id, null: false
 
       t.timestamps
     end

data/spec/dummy/db/migrate/20080830154109_create_realizes.rb

@@ -1,8 +1,8 @@
 class CreateRealizes < ActiveRecord::Migration[4.2]
   def change
     create_table :realizes do |t|
-      t.references :agent, :null => false
-      t.references :expression, :null => false
+      t.references :agent, null: false
+      t.references :expression, null: false
       t.integer :position
 
       t.timestamps

data/spec/dummy/db/migrate/20080830172106_create_exemplifies.rb

@@ -1,13 +1,13 @@
 class CreateExemplifies < ActiveRecord::Migration[4.2]
   def change
     create_table :exemplifies do |t|
-      t.integer :manifestation_id, :null => false
-      t.integer :item_id, :null => false
+      t.integer :manifestation_id, null: false
+      t.integer :item_id, null: false
       t.integer :position
 
       t.timestamps
     end
     add_index :exemplifies, :manifestation_id
-    add_index :exemplifies, :item_id, :unique => true
+    add_index :exemplifies, :item_id, unique: true
   end
 end

data/spec/dummy/db/migrate/20080905191442_create_agent_types.rb

@@ -1,7 +1,7 @@
 class CreateAgentTypes < ActiveRecord::Migration[4.2]
   def change
     create_table :agent_types do |t|
-      t.string :name, :null => false
+      t.string :name, null: false
       t.text :display_name
       t.text :note
       t.integer :position

data/spec/dummy/db/migrate/20081023092436_create_search_engines.rb

@@ -1,12 +1,12 @@
 class CreateSearchEngines < ActiveRecord::Migration[4.2]
   def change
     create_table :search_engines do |t|
-      t.string :name, :null => false
+      t.string :name, null: false
       t.text :display_name
-      t.string :url, :null => false
-      t.text :base_url, :null => false
-      t.text :http_method, :null => false
-      t.text :query_param, :null => false
+      t.string :url, null: false
+      t.text :base_url, null: false
+      t.text :http_method, null: false
+      t.text :query_param, null: false
       t.text :additional_param
       t.text :note
       t.integer :position

data/spec/dummy/db/migrate/20081025083323_create_countries.rb

@@ -1,22 +1,21 @@
 class CreateCountries < ActiveRecord::Migration[4.2]
-
   # ISO 3166 is the International Standard for country codes.
   #
-  # ISO 3166-1:2006 Codes for the representation of names of countries and their subdivisions - Part 1: 
-  # Country codes which is what most users know as ISO's country codes. First published in 1974, it is has since 
-  # then become one of the world's most popular and most widely used standard solution for coding country names. 
-  # It contains a two-letter code which is recommended as the general purpose code, a three-letter code which has 
+  # ISO 3166-1:2006 Codes for the representation of names of countries and their subdivisions - Part 1:
+  # Country codes which is what most users know as ISO's country codes. First published in 1974, it is has since
+  # then become one of the world's most popular and most widely used standard solution for coding country names.
+  # It contains a two-letter code which is recommended as the general purpose code, a three-letter code which has
   # better mnenomic properties and a numeric-3 code which can be useful if script independence of the codes is important.
   #
   # http://www.iso.org/iso/country_codes/background_on_iso_3166/what_is_iso_3166.htm
 
   def change
     create_table :countries do |t|
-      t.string :name,         :size => 80, :null => false
+      t.string :name, size: 80, null: false
       t.text :display_name
-      t.string :alpha_2,      :size => 2
-      t.string :alpha_3,      :size => 3
-      t.string :numeric_3, :size => 3
+      t.string :alpha_2,      size: 2
+      t.string :alpha_3,      size: 3
+      t.string :numeric_3, size: 3
       t.text :note
       t.integer :position
     end

data/spec/dummy/db/migrate/20081025083905_create_languages.rb

@@ -1,21 +1,20 @@
 class CreateLanguages < ActiveRecord::Migration[4.2]
-
   # ISO 639 is the set of international standards that lists short codes for language names.
   # Note this doesn't include macrolanguages (dialects)
   # Information on macrolanguages http://en.wikipedia.org/wiki/ISO_639_macrolanguage
 
   def change
     create_table :languages do |t|
-      t.string :name, :null => false
+      t.string :name, null: false
       t.string :native_name
       t.text :display_name
-      t.string :iso_639_1,        :size => 3
-      t.string :iso_639_2,        :size => 3
-      t.string :iso_639_3,        :size => 3
+      t.string :iso_639_1,        size: 3
+      t.string :iso_639_2,        size: 3
+      t.string :iso_639_3,        size: 3
       t.text :note
       t.integer :position
     end
-    add_index :languages, :name, :unique => true
+    add_index :languages, :name, unique: true
     add_index :languages, :iso_639_1
     add_index :languages, :iso_639_2
     add_index :languages, :iso_639_3

data/spec/dummy/db/migrate/20081027150907_create_picture_files.rb

@@ -10,6 +10,6 @@ class CreatePictureFiles < ActiveRecord::Migration[4.2]
 
       t.timestamps
     end
-    add_index :picture_files, [:picture_attachable_id, :picture_attachable_type], :name => "index_picture_files_on_picture_attachable_id_and_type"
+    add_index :picture_files, %i[picture_attachable_id picture_attachable_type], name: "index_picture_files_on_picture_attachable_id_and_type"
   end
 end

data/spec/dummy/db/migrate/20090720091106_create_medium_of_performances.rb

@@ -1,7 +1,7 @@
 class CreateMediumOfPerformances < ActiveRecord::Migration[4.2]
   def change
     create_table :medium_of_performances do |t|
-      t.string :name, :null => false
+      t.string :name, null: false
       t.text :display_name
       t.text :note
       t.integer :position

data/spec/dummy/db/migrate/20090720091429_create_content_types.rb

@@ -1,7 +1,7 @@
 class CreateContentTypes < ActiveRecord::Migration[4.2]
   def change
     create_table :content_types do |t|
-      t.string :name, :null => false
+      t.string :name, null: false
       t.text :display_name
       t.text :note
       t.integer :position

data/spec/dummy/db/migrate/20090812151902_create_agent_relationship_types.rb

@@ -1,7 +1,7 @@
 class CreateAgentRelationshipTypes < ActiveRecord::Migration[4.2]
   def change
     create_table :agent_relationship_types do |t|
-      t.string :name, :null => false
+      t.string :name, null: false
       t.text :display_name
       t.text :note
       t.integer :position

data/spec/dummy/db/migrate/20091025080447_create_licenses.rb

@@ -1,7 +1,7 @@
 class CreateLicenses < ActiveRecord::Migration[4.2]
   def change
     create_table :licenses do |t|
-      t.string :name, :null => false
+      t.string :name, null: false
       t.string :display_name
       t.text :note
       t.integer :position

data/spec/dummy/db/migrate/20091202124834_create_versions.rb

@@ -1,18 +1,18 @@
 class CreateVersions < ActiveRecord::Migration[4.2]
   def self.up
     create_table :versions do |t|
-      t.string   :item_type, :null => false
-      t.integer  :item_id,   :null => false
-      t.string   :event,     :null => false
+      t.string   :item_type, null: false
+      t.integer  :item_id,   null: false
+      t.string   :event,     null: false
       t.string   :whodunnit
       t.text     :object
       t.datetime :created_at
     end
-    add_index :versions, [:item_type, :item_id]
+    add_index :versions, %i[item_type item_id]
   end
 
   def self.down
-    remove_index :versions, [:item_type, :item_id]
+    remove_index :versions, %i[item_type item_id]
     drop_table :versions
   end
 end

data/spec/dummy/db/migrate/20100607044753_create_manifestation_relationship_types.rb

@@ -1,7 +1,7 @@
 class CreateManifestationRelationshipTypes < ActiveRecord::Migration[4.2]
   def change
     create_table :manifestation_relationship_types do |t|
-      t.string :name, :null => false
+      t.string :name, null: false
       t.text :display_name
       t.text :note
       t.integer :position

data/spec/dummy/db/migrate/20110222073537_add_url_to_library_group.rb

@@ -1,6 +1,6 @@
 class AddUrlToLibraryGroup < ActiveRecord::Migration[4.2]
   def self.up
-    add_column :library_groups, :url, :string, :default => 'http://localhost:3000/'
+    add_column :library_groups, :url, :string, default: 'http://localhost:3000/'
   end
 
   def self.down

data/spec/dummy/db/migrate/20110318183304_add_valid_period_for_new_user_to_user_group.rb

@@ -1,6 +1,6 @@
 class AddValidPeriodForNewUserToUserGroup < ActiveRecord::Migration[4.2]
   def self.up
-    add_column :user_groups, :valid_period_for_new_user, :integer, :default => 0, :null => false
+    add_column :user_groups, :valid_period_for_new_user, :integer, default: 0, null: false
     add_column :user_groups, :expired_at, :timestamp
   end
 

data/spec/dummy/db/migrate/20110627035057_create_series_statement_merges.rb

@@ -1,8 +1,8 @@
 class CreateSeriesStatementMerges < ActiveRecord::Migration[4.2]
   def change
     create_table :series_statement_merges do |t|
-      t.integer :series_statement_id, :null => false
-      t.integer :series_statement_merge_list_id, :null => false
+      t.integer :series_statement_id, null: false
+      t.integer :series_statement_merge_list_id, null: false
 
       t.timestamps
     end

data/spec/dummy/db/migrate/20120125152919_add_title_subseries_transcription_to_series_statement.rb

@@ -1,6 +1,5 @@
 class AddTitleSubseriesTranscriptionToSeriesStatement < ActiveRecord::Migration[4.2]
   def change
     add_column :series_statements, :title_subseries_transcription, :text
-
   end
 end

data/spec/dummy/db/migrate/20120129020544_add_budget_type_id_to_item.rb

@@ -1,6 +1,5 @@
 class AddBudgetTypeIdToItem < ActiveRecord::Migration[4.2]
   def change
     add_column :items, :budget_type_id, :integer
-
   end
 end

data/spec/dummy/db/migrate/20120319120638_add_content_type_id_to_manifestation.rb

@@ -1,6 +1,5 @@
 class AddContentTypeIdToManifestation < ActiveRecord::Migration[4.2]
   def change
-    add_column :manifestations, :content_type_id, :integer, :default => 1
-
+    add_column :manifestations, :content_type_id, :integer, default: 1
   end
 end

data/spec/dummy/db/migrate/20120510140958_add_closed_to_shelf.rb

@@ -1,5 +1,5 @@
 class AddClosedToShelf < ActiveRecord::Migration[4.2]
   def change
-    add_column :shelves, :closed, :boolean, :default => false, :null => false
+    add_column :shelves, :closed, :boolean, default: false, null: false
   end
 end

data/spec/dummy/db/migrate/20130221154434_add_additional_attributes_to_user.rb

@@ -4,13 +4,13 @@ class AddAdditionalAttributesToUser < ActiveRecord::Migration[4.2]
     add_column :users, :deleted_at, :datetime
     add_column :users, :expired_at, :datetime
 
-    add_column :users, :failed_attempts, :integer, :default => 0
+    add_column :users, :failed_attempts, :integer, default: 0
     add_column :users, :unlock_token, :string
     add_column :users, :locked_at, :datetime
 
     add_column :users, :confirmed_at, :datetime
 
-    add_index :users, :username, :unique => true
-    add_index :users, :unlock_token,         :unique => true
+    add_index :users, :username, unique: true
+    add_index :users, :unlock_token, unique: true
   end
 end

data/spec/dummy/db/migrate/20130506175834_create_identifiers.rb

@@ -1,15 +1,15 @@
 class CreateIdentifiers < ActiveRecord::Migration[4.2]
   def change
     create_table :identifiers do |t|
-      t.string :body, :null => false
-      t.integer :identifier_type_id, :null => false
+      t.string :body, null: false
+      t.integer :identifier_type_id, null: false
       t.integer :manifestation_id
       t.boolean :primary
       t.integer :position
 
       t.timestamps
     end
-    add_index :identifiers, [:body, :identifier_type_id]
+    add_index :identifiers, %i[body identifier_type_id]
     add_index :identifiers, :manifestation_id
   end
 end

data/spec/dummy/db/migrate/20140122054321_create_profiles.rb

@@ -15,6 +15,6 @@ class CreateProfiles < ActiveRecord::Migration[4.2]
     end
 
     add_index :profiles, :user_id
-    add_index :profiles, :user_number, :unique => true
+    add_index :profiles, :user_number, unique: true
   end
 end

data/spec/dummy/db/migrate/20140519170214_create_resource_import_file_transitions.rb

@@ -13,6 +13,6 @@ class CreateResourceImportFileTransitions < ActiveRecord::Migration[4.2]
     end
 
     add_index :resource_import_file_transitions, :resource_import_file_id, name: "index_resource_import_file_transitions_on_file_id"
-    add_index :resource_import_file_transitions, [:sort_key, :resource_import_file_id], unique: true, name: "index_resource_import_file_transitions_on_sort_key_and_file_id"
+    add_index :resource_import_file_transitions, %i[sort_key resource_import_file_id], unique: true, name: "index_resource_import_file_transitions_on_sort_key_and_file_id"
   end
 end

data/spec/dummy/db/migrate/20140519171220_create_import_request_transitions.rb

@@ -13,6 +13,6 @@ class CreateImportRequestTransitions < ActiveRecord::Migration[4.2]
     end
 
     add_index :import_request_transitions, :import_request_id
-    add_index :import_request_transitions, [:sort_key, :import_request_id], unique: true, name: "index_import_request_transitions_on_sort_key_and_request_id"
+    add_index :import_request_transitions, %i[sort_key import_request_id], unique: true, name: "index_import_request_transitions_on_sort_key_and_request_id"
   end
 end

data/spec/dummy/db/migrate/20140524020735_create_agent_import_file_transitions.rb

@@ -13,6 +13,6 @@ class CreateAgentImportFileTransitions < ActiveRecord::Migration[4.2]
     end
 
     add_index :agent_import_file_transitions, :agent_import_file_id
-    add_index :agent_import_file_transitions, [:sort_key, :agent_import_file_id], unique: true, name: "index_agent_import_file_transitions_on_sort_key_and_file_id"
+    add_index :agent_import_file_transitions, %i[sort_key agent_import_file_id], unique: true, name: "index_agent_import_file_transitions_on_sort_key_and_file_id"
   end
 end

data/spec/dummy/db/migrate/20140524074813_create_user_import_file_transitions.rb

@@ -12,6 +12,6 @@ class CreateUserImportFileTransitions < ActiveRecord::Migration[4.2]
       t.timestamps
     end
 
-    add_index :user_import_file_transitions, [:sort_key, :user_import_file_id], unique: true, name: "index_user_import_file_transitions_on_sort_key_and_file_id"
+    add_index :user_import_file_transitions, %i[sort_key user_import_file_id], unique: true, name: "index_user_import_file_transitions_on_sort_key_and_file_id"
   end
 end

data/spec/dummy/db/migrate/20140614141500_create_resource_export_file_transitions.rb

@@ -13,6 +13,6 @@ class CreateResourceExportFileTransitions < ActiveRecord::Migration[4.2]
     end
 
     add_index :resource_export_file_transitions, :resource_export_file_id, name: "index_resource_export_file_transitions_on_file_id"
-    add_index :resource_export_file_transitions, [:sort_key, :resource_export_file_id], unique: true, name: "index_resource_export_file_transitions_on_sort_key_and_file_id"
+    add_index :resource_export_file_transitions, %i[sort_key resource_export_file_id], unique: true, name: "index_resource_export_file_transitions_on_sort_key_and_file_id"
   end
 end

data/spec/dummy/db/migrate/20140709113905_create_user_export_file_transitions.rb

@@ -13,6 +13,6 @@ class CreateUserExportFileTransitions < ActiveRecord::Migration[4.2]
     end
 
     add_index :user_export_file_transitions, :user_export_file_id, name: "index_user_export_file_transitions_on_file_id"
-    add_index :user_export_file_transitions, [:sort_key, :user_export_file_id], unique: true, name: "index_user_export_file_transitions_on_sort_key_and_file_id"
+    add_index :user_export_file_transitions, %i[sort_key user_export_file_id], unique: true, name: "index_user_export_file_transitions_on_sort_key_and_file_id"
   end
 end

data/spec/dummy/db/migrate/20151213070943_add_translation_table_to_library_group.rb

@@ -3,10 +3,9 @@ class AddTranslationTableToLibraryGroup < ActiveRecord::Migration[4.2]
   def up
     if defined?(Globalize)
       LibraryGroup.create_translation_table!({
-        login_banner: :text
-      }, {
-        migrate_data: true
-      })
+                                               login_banner: :text
+                                             },
+                                             migrate_data: true)
     end
 
     if defined?(AwesomeHstoreTranslate)

data/spec/dummy/db/migrate/20180107160726_add_constraints_to_most_recent_for_user_import_file_transitions.rb

@@ -2,7 +2,7 @@ class AddConstraintsToMostRecentForUserImportFileTransitions < ActiveRecord::Mig
   disable_ddl_transaction!
 
   def up
-    add_index :user_import_file_transitions, [:user_import_file_id, :most_recent], unique: true, where: "most_recent", name: "index_user_import_file_transitions_parent_most_recent" #, algorithm: :concurrently
+    add_index :user_import_file_transitions, %i[user_import_file_id most_recent], unique: true, where: "most_recent", name: "index_user_import_file_transitions_parent_most_recent" # , algorithm: :concurrently
     change_column_null :user_import_file_transitions, :most_recent, false
   end
 

data/spec/dummy/db/migrate/20180107160740_add_constraints_to_most_recent_for_user_export_file_transitions.rb

@@ -2,7 +2,7 @@ class AddConstraintsToMostRecentForUserExportFileTransitions < ActiveRecord::Mig
   disable_ddl_transaction!
 
   def up
-    add_index :user_export_file_transitions, [:user_export_file_id, :most_recent], unique: true, where: "most_recent", name: "index_user_export_file_transitions_parent_most_recent" #, algorithm: :concurrently
+    add_index :user_export_file_transitions, %i[user_export_file_id most_recent], unique: true, where: "most_recent", name: "index_user_export_file_transitions_parent_most_recent" # , algorithm: :concurrently
     change_column_null :user_export_file_transitions, :most_recent, false
   end
 

data/spec/dummy/db/migrate/20180107161311_add_constraints_to_most_recent_for_agent_import_file_transitions.rb

@@ -2,7 +2,7 @@ class AddConstraintsToMostRecentForAgentImportFileTransitions < ActiveRecord::Mi
   disable_ddl_transaction!
 
   def up
-    add_index :agent_import_file_transitions, [:agent_import_file_id, :most_recent], unique: true, where: "most_recent", name: "index_agent_import_file_transitions_parent_most_recent" #, algorithm: :concurrently
+    add_index :agent_import_file_transitions, %i[agent_import_file_id most_recent], unique: true, where: "most_recent", name: "index_agent_import_file_transitions_parent_most_recent" # , algorithm: :concurrently
     change_column_null :agent_import_file_transitions, :most_recent, false
   end
 

data/spec/dummy/db/migrate/20180107161331_add_constraints_to_most_recent_for_resource_import_file_transitions.rb

@@ -2,7 +2,7 @@ class AddConstraintsToMostRecentForResourceImportFileTransitions < ActiveRecord:
   disable_ddl_transaction!
 
   def up
-    add_index :resource_import_file_transitions, [:resource_import_file_id, :most_recent], unique: true, where: "most_recent", name: "index_resource_import_file_transitions_parent_most_recent" #, algorithm: :concurrently
+    add_index :resource_import_file_transitions, %i[resource_import_file_id most_recent], unique: true, where: "most_recent", name: "index_resource_import_file_transitions_parent_most_recent" # , algorithm: :concurrently
     change_column_null :resource_import_file_transitions, :most_recent, false
   end
 

data/spec/dummy/db/migrate/20180107161347_add_constraints_to_most_recent_for_resource_export_file_transitions.rb

@@ -2,7 +2,7 @@ class AddConstraintsToMostRecentForResourceExportFileTransitions < ActiveRecord:
   disable_ddl_transaction!
 
   def up
-    add_index :resource_export_file_transitions, [:resource_export_file_id, :most_recent], unique: true, where: "most_recent", name: "index_resource_export_file_transitions_parent_most_recent" #, algorithm: :concurrently
+    add_index :resource_export_file_transitions, %i[resource_export_file_id most_recent], unique: true, where: "most_recent", name: "index_resource_export_file_transitions_parent_most_recent" # , algorithm: :concurrently
     change_column_null :resource_export_file_transitions, :most_recent, false
   end
 

data/spec/dummy/db/migrate/20180107161410_add_constraints_to_most_recent_for_import_request_transitions.rb

@@ -2,7 +2,7 @@ class AddConstraintsToMostRecentForImportRequestTransitions < ActiveRecord::Migr
   disable_ddl_transaction!
 
   def up
-    add_index :import_request_transitions, [:import_request_id, :most_recent], unique: true, where: "most_recent", name: "index_import_request_transitions_parent_most_recent" #, algorithm: :concurrently
+    add_index :import_request_transitions, %i[import_request_id most_recent], unique: true, where: "most_recent", name: "index_import_request_transitions_parent_most_recent" # , algorithm: :concurrently
     change_column_null :import_request_transitions, :most_recent, false
   end
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: enju_manifestation_viewer
 version: !ruby/object:Gem::Version
-  version: 0.3.0.beta.1
+  version: 0.3.0.beta.2
 platform: ruby
 authors:
 - Kosuke Tanabe
@@ -149,6 +149,7 @@ files:
 - app/assets/config/enju_manifestation_viewer_manifest.js
 - app/helpers/enju_manifestation_viewer/application_helper.rb
 - app/helpers/enju_manifestation_viewer/book_jacket_helper.rb
+- app/helpers/enju_manifestation_viewer/manifestation_viewer_helper.rb
 - app/views/manifestations/_flickr.html.erb
 - app/views/manifestations/_google_book_search.html.erb
 - app/views/manifestations/_google_book_thumbnail.html.erb