enju_leaf 1.1.0.rc12 → 1.1.0.rc13

data/app/models/enju_leaf/ability.rb

@@ -6,12 +6,12 @@ module EnjuLeaf
   
       case user.try(:role).try(:name)
       when 'Administrator'
-        can :index, User
-        can [:read, :create, :update], User
-        can :destroy, User do |u|
-          if u != user and u.id != 1
+        can :index, Profile
+        can [:read, :create, :update], [User, Profile]
+        can :destroy, Profile do |profile|
+          if profile != user.profile and profile.user.id != 1
             if defined?(EnjuCirculation)
-               true if u.checkouts.not_returned.empty?
+               true if profile.user.checkouts.not_returned.empty?
             else
               true
             end
@@ -19,12 +19,13 @@ module EnjuLeaf
         end
         can [:read, :create, :update], UserGroup
         can :destroy, UserGroup do |user_group|
-          user_group.users.empty?
+          user_group.profiles.empty?
         end
         can :manage, [
           UserHasRole
         ]
         can :manage, [
+          UserExportFile,
           UserImportFile
         ] if LibraryGroup.site_config.network_access_allowed?(ip_address)
         can :update, [
@@ -35,11 +36,11 @@ module EnjuLeaf
           UserImportResult
         ] if LibraryGroup.site_config.network_access_allowed?(ip_address)
       when 'Librarian'
-        can [:read, :create, :update], User
-        can :destroy, User do |u|
-          if u.role.name == 'User' and u != user
+        can [:read, :create, :update], Profile
+        can :destroy, Profile do |profile|
+          if profile.user.role.name == 'User' and profile != user.profile
             if defined?(EnjuCirculation)
-               true if u.checkouts.not_returned.empty?
+               true if profile.user.checkouts.not_returned.empty?
             else
               true
             end
@@ -56,9 +57,9 @@ module EnjuLeaf
           UserImportResult
         ] if LibraryGroup.site_config.network_access_allowed?(ip_address)
       when 'User'
-        can :show, User
-        can :update, User do |u|
-          u == user
+        can :show, Profile
+        can :update, Profile do |profile|
+          profile == user.profile
         end
         can :read, [
           UserGroup

data/app/models/profile.rb

@@ -1,16 +1,61 @@
 class Profile < ActiveRecord::Base
+  attr_accessible :full_name, :keyword_list, :locale
+  attr_accessible :full_name, :user_number, :library_id, :keyword_list, :note,
+    :user_group_id, :user_id, :locale, :required_role_id, :expired_at,
+    :user_attributes,
+    :save_checkout_history,
+    as: :admin
+
+  enju_circulation_profile_model if defined?(EnjuCirculation)
+  enju_search_log_profile_model if defined?(EnjuSearchLog)
+
   belongs_to :user
-  belongs_to :library, :validate => true
+  belongs_to :library, validate: true
   belongs_to :user_group
-  belongs_to :required_role, :class_name => 'Role', :foreign_key => 'required_role_id' #, :validate => true
+  belongs_to :required_role, class_name: 'Role', foreign_key: 'required_role_id' #, validate: true
 
-  validates_associated :user_group, :library #, :agent
+  validates_associated :user_group, :library
+  validates_associated :user
   validates_presence_of :user_group, :library, :locale #, :user_number
-  validates :user_number, :uniqueness => true, :format => {:with => /\A[0-9A-Za-z_]+\Z/}, :allow_blank => true
+  validates :user_number, uniqueness: true, format: {with: /\A[0-9A-Za-z_]+\Z/}, allow_blank: true
 
   searchable do
     text :user_number, :full_name, :note
     string :user_number
+    text :username do
+      user.try(:username)
+    end
+    text :email do
+      user.try(:email)
+    end
+    string :username do
+      user.try(:username)
+    end
+    string :email do
+      user.try(:email)
+    end
+    time :created_at
+    time :updated_at
+    boolean :active do
+      user.try(:active_for_authentication?)
+    end
+  end
+
+  before_validation :set_role_and_agent, on: :create
+  before_save :set_expired_at
+  accepts_nested_attributes_for :user
+
+  def set_role_and_agent
+    self.required_role = Role.where(name: 'Librarian').first
+    self.locale = I18n.default_locale.to_s unless locale
+  end
+
+  def set_expired_at
+    if expired_at.blank?
+      if user_group.valid_period_for_new_user > 0
+        self.expired_at = user_group.valid_period_for_new_user.days.from_now.end_of_day
+      end
+    end
   end
 end
 
@@ -18,16 +63,18 @@ end
 #
 # Table name: profiles
 #
-#  id               :integer          not null, primary key
-#  user_id          :integer
-#  user_group_id    :integer
-#  library_id       :integer
-#  locale           :string(255)
-#  user_number      :string(255)
-#  full_name        :text
-#  note             :text
-#  keyword_list     :text
-#  required_role_id :integer
-#  created_at       :datetime         not null
-#  updated_at       :datetime         not null
+#  id                       :integer          not null, primary key
+#  user_id                  :integer
+#  user_group_id            :integer
+#  library_id               :integer
+#  locale                   :string(255)
+#  user_number              :string(255)
+#  full_name                :text
+#  note                     :text
+#  keyword_list             :text
+#  required_role_id         :integer
+#  created_at               :datetime         not null
+#  updated_at               :datetime         not null
+#  checkout_icalendar_token :string(255)
+#  save_checkout_history    :boolean          default(FALSE), not null
 #

data/app/models/role.rb

@@ -1,9 +1,9 @@
 class Role < ActiveRecord::Base
   attr_accessible :name, :display_name, :note
   include MasterModel
-  default_scope :order => "roles.position"
+  default_scope order: "roles.position"
   has_many :user_has_roles
-  has_many :users, :through => :user_has_roles
+  has_many :users, through: :user_has_roles
   after_save :clear_all_cache
   after_destroy :clear_all_cache
 

data/app/models/user_export_file.rb

@@ -0,0 +1,52 @@
+class UserExportFile < ActiveRecord::Base
+  include Statesman::Adapters::ActiveRecordModel
+  include ExportFile
+  enju_export_file_model
+  has_attached_file :user_export
+  validates_attachment_content_type :user_export, :content_type => /\Atext\/plain\Z/
+
+  has_many :user_export_file_transitions
+
+  def state_machine
+    UserExportFileStateMachine.new(self, transition_class: UserExportFileTransition)
+  end
+
+  delegate :can_transition_to?, :transition_to!, :transition_to, :current_state,
+    to: :state_machine
+
+  def export!
+    transition_to!(:started)
+    tempfile = Tempfile.new(['user_export_file_', '.txt'])
+    file = User.export(format: :txt)
+    tempfile.puts(file)
+    tempfile.close
+    self.user_export = File.new(tempfile.path, "r")
+    if save
+      send_message
+    end
+    transition_to!(:completed)
+  rescue => e
+    transition_to!(:failed)
+    raise e
+  end
+
+  private
+  def self.transition_class
+    UserExportFileTransition
+  end
+end
+
+# == Schema Information
+#
+# Table name: user_export_files
+#
+#  id                       :integer          not null, primary key
+#  user_id                  :integer
+#  user_export_file_name    :string(255)
+#  user_export_content_type :string(255)
+#  user_export_file_size    :integer
+#  user_export_updated_at   :datetime
+#  executed_at              :datetime
+#  created_at               :datetime         not null
+#  updated_at               :datetime         not null
+#

data/app/models/user_export_file_state_machine.rb

@@ -0,0 +1,15 @@
+class UserExportFileStateMachine
+  include Statesman::Machine
+
+  state :pending, initial: true
+  state :started
+  state :completed
+  state :failed
+
+  transition from: :pending, to: :started
+  transition from: :started, to: [:completed, :failed]
+
+  after_transition(from: :pending, to: :started) do |user_export_file|
+    user_export_file.update_column(:executed_at, Time.zone.now)
+  end
+end

data/app/models/user_export_file_transition.rb

@@ -0,0 +1,20 @@
+class UserExportFileTransition < ActiveRecord::Base
+  include Statesman::Adapters::ActiveRecordTransition
+
+  
+  belongs_to :user_export_file, inverse_of: :user_export_file_transitions
+  attr_accessible :to_state, :sort_key, :metadata
+end
+
+# == Schema Information
+#
+# Table name: user_export_file_transitions
+#
+#  id                  :integer          not null, primary key
+#  to_state            :string(255)
+#  metadata            :text             default("{}")
+#  sort_key            :integer
+#  user_export_file_id :integer
+#  created_at          :datetime         not null
+#  updated_at          :datetime         not null
+#

data/app/models/user_group.rb

@@ -7,12 +7,12 @@ class UserGroup < ActiveRecord::Base
     :number_of_time_to_notify_overdue
 
   include MasterModel
-  default_scope :order => "user_groups.position"
-  has_many :users
+  default_scope order: "user_groups.position"
+  has_many :profiles
 
   validates_numericality_of :valid_period_for_new_user,
     :greater_than_or_equal_to => 0,
-    :allow_blank => true
+    allow_blank: true
 
   paginates_per 10
 

data/app/models/user_has_role.rb

@@ -1,11 +1,11 @@
 class UserHasRole < ActiveRecord::Base
   attr_accessible :user_id, :role_id
-  attr_accessible :user_id, :role_id, :as => :admin
+  attr_accessible :user_id, :role_id, as: :admin
   belongs_to :user
   belongs_to :role
   accepts_nested_attributes_for :role
 
-#  validates_uniqueness_of :role_id, :scope => :user_id
+#  validates_uniqueness_of :role_id, scope: :user_id
 #  validates_presence_of :role_id, :user_id
 end
 

data/app/models/user_import_file.rb

@@ -1,5 +1,6 @@
 class UserImportFile < ActiveRecord::Base
-  attr_accessible :user_import, :edit_mode, :user_encoding, :mode
+  attr_accessible :user_import, :edit_mode, :user_encoding, :mode,
+    :default_user_group_id, :default_library_id
   include Statesman::Adapters::ActiveRecordModel
   include ImportFile
   default_scope {order('user_import_files.id DESC')}
@@ -12,7 +13,7 @@ class UserImportFile < ActiveRecord::Base
       :s3_permissions => :private
   else
     has_attached_file :user_import,
-      :path => ":rails_root/private/system/:class/:attachment/:id_partition/:style/:filename"
+      path: ":rails_root/private/system/:class/:attachment/:id_partition/:style/:filename"
   end
   validates_attachment_content_type :user_import, :content_type => [
     'text/csv',
@@ -22,7 +23,9 @@ class UserImportFile < ActiveRecord::Base
     'application/vnd.ms-excel'
   ]
   validates_attachment_presence :user_import
-  belongs_to :user, :validate => true
+  belongs_to :user, validate: true
+  belongs_to :default_user_group, class_name: 'UserGroup'
+  belongs_to :default_library, class_name: 'Library'
   has_many :user_import_results
 
   has_many :user_import_file_transitions
@@ -40,7 +43,7 @@ class UserImportFile < ActiveRecord::Base
   def import
     transition_to!(:started)
     num = {:user_imported => 0, :user_found => 0, :failed => 0}
-    rows = open_import_file(create_import_temp_file)
+    rows = open_import_file(create_import_temp_file(user_import))
     row_num = 1
 
     field = rows.first
@@ -52,7 +55,7 @@ class UserImportFile < ActiveRecord::Base
       row_num += 1
       next if row['dummy'].to_s.strip.present?
       import_result = UserImportResult.create!(
-        :user_import_file_id => id, :body => row.fields.join("\t")
+        :user_import_file_id => id, body: row.fields.join("\t")
       )
 
       username = row['username']
@@ -63,7 +66,7 @@ class UserImportFile < ActiveRecord::Base
         num[:user_found] += 1
       else
         new_user = User.new
-        new_user.role = Role.where(:name => row['role']).first
+        new_user.role = Role.where(name: row['role']).first
         if new_user.role
           unless user.has_role?(new_user.role.name)
             num[:failed] += 1
@@ -74,42 +77,29 @@ class UserImportFile < ActiveRecord::Base
         end
         new_user.operator = user
         new_user.username = username
-        new_user.email = row['email'] if row['email'].present?
-        if row['user_group'].present?
-          new_user.user_group = UserGroup.where(:name => row['user_group']).first
-        end
-        new_user.user_number = row['user_number']
-        if row['expired_at'].present?
-          new_user.expired_at = Time.zone.parse(row['expired_at']).end_of_day
-        end
-        new_user.note = row['note']
-
-        if I18n.available_locales.include?(row['locale'].to_s.to_sym)
-          new_user.locale = row['locale']
-        end
-
-        library = Library.where(:name => row['library'].to_s.strip).first || Library.web
-        new_user.library = library
-
-        if row['password'].present?
-          new_user.password = row['password']
-        else
-          new_user.set_auto_generated_password
-        end
+        new_user.assign_attributes(set_user_params(new_user, row), as: :admin)
+        profile = Profile.new
+        profile.assign_attributes(set_profile_params(row), as: :admin)
 
         if new_user.save
-          num[:user_imported] += 1
-          import_result.user = new_user
-          import_result.save!
+          new_user.profile = profile
+          if profile.save
+            num[:user_imported] += 1
+            import_result.user = new_user
+            import_result.save!
+          else
+            num[:failed] += 1
+          end
         else
           num[:failed] += 1
         end
       end
     end
 
+    Sunspot.commit
     rows.close
     transition_to!(:completed)
-    Sunspot.commit
+    send_message
     num
   rescue => e
     self.error_message = "line #{row_num}: #{e.message}"
@@ -119,8 +109,42 @@ class UserImportFile < ActiveRecord::Base
 
   def modify
     transition_to!(:started)
-    transition_to!(:completed)
+    num = {:user_updated => 0, :user_not_found => 0, :failed => 0}
+    rows = open_import_file(create_import_temp_file(user_import))
     row_num = 1
+
+    field = rows.first
+    if [field['username']].reject{|f| f.to_s.strip == ""}.empty?
+      raise "username column is not found"
+    end
+
+    rows.each do |row|
+      row_num += 1
+      next if row['dummy'].to_s.strip.present?
+      import_result = UserImportResult.create!(
+        :user_import_file_id => id, body: row.fields.join("\t")
+      )
+
+      username = row['username']
+      new_user = User.where(:username => username).first
+      if new_user
+        new_user.assign_attributes(set_user_params(new_user, row), as: :admin)
+        if new_user.save
+          num[:user_updated] += 1
+          import_result.user = new_user
+          import_result.save!
+        else
+          num[:failed] += 1
+        end
+      else
+        num[:user_not_found] += 1
+      end
+    end
+
+    rows.close
+    transition_to!(:completed)
+    Sunspot.commit
+    num
   rescue => e
     self.error_message = "line #{row_num}: #{e.message}"
     transition_to!(:failed)
@@ -130,7 +154,7 @@ class UserImportFile < ActiveRecord::Base
   def remove
     transition_to!(:started)
     row_num = 1
-    rows = open_import_file(create_import_temp_file)
+    rows = open_import_file(create_import_temp_file(user_import))
 
     field = rows.first
     if [field['username']].reject{|field| field.to_s.strip == ""}.empty?
@@ -155,27 +179,30 @@ class UserImportFile < ActiveRecord::Base
     UserImportFileTransition
   end
 
-  def create_import_temp_file
-    tempfile = Tempfile.new(self.class.name.underscore)
-    if Setting.uploaded_file.storage == :s3
-      uploaded_file_path = user_import.expiring_url(10)
-    else
-      uploaded_file_path = user_import.path
-    end
-    open(uploaded_file_path){|f|
-      f.each{|line|
-        tempfile.puts(convert_encoding(line))
-      }
-    }
-    tempfile.close
-    tempfile
-  end
-
   def open_import_file(tempfile)
-    file = CSV.open(tempfile.path, 'r:utf-8', :col_sep => "\t")
+    file = CSV.open(tempfile.path, 'r:utf-8', col_sep: "\t")
+    header_columns = %w(
+      username role email password user_group user_number expired_at
+      keyword_list note locale library dummy
+    )
+    if defined?(EnjuCirculation)
+      header_columns += %w(checkout_icalendar_token save_checkout_history)
+    end
+    if defined?(EnjuSearchLog)
+      header_columns += %w(save_search_history)
+    end
+    if defined?(EnjuBookmark)
+      header_columns += %w(share_bookmarks)
+    end
+
     header = file.first
-    rows = CSV.open(tempfile.path, 'r:utf-8', :headers => header, :col_sep => "\t")
-    UserImportResult.create!(:user_import_file_id => self.id, :body => header.join("\t"))
+    ignored_columns = header - header_columns
+    unless ignored_columns.empty?
+      self.error_message = I18n.t('import.following_column_were_ignored', column: ignored_columns.join(', '))
+      save!
+    end
+    rows = CSV.open(tempfile.path, 'r:utf-8', headers: header, col_sep: "\t")
+    UserImportResult.create!(:user_import_file_id => self.id, body: header.join("\t"))
     tempfile.close(true)
     file.close
     rows
@@ -188,6 +215,63 @@ class UserImportFile < ActiveRecord::Base
   rescue
     Rails.logger.info "#{Time.zone.now} importing resources failed!"
   end
+
+  private
+  def set_user_params(new_user, row)
+    params = {}
+    profile_params = {}
+    params[:email] = row['email'] if row['email'].present?
+
+    if row['password'].present?
+      params[:password] = row['password']
+    else
+      params[:password] = Devise.friendly_token[0..7]
+    end
+    params
+  end
+
+  def set_profile_params(row)
+    params = {}
+    user_group = UserGroup.where(name: row['user_group']).first
+    unless user_group
+      user_group = default_user_group
+    end
+    params[:user_group_id] = user_group.id if user_group
+
+    params[:user_number] = row['user_number']
+
+    if row['expired_at'].present?
+      params[:expired_at] = Time.zone.parse(row['expired_at']).end_of_day
+    end
+
+    if row['keyword_list'].present?
+      params[:keyword_list] = row['keyword_list'].split('//').join('\n')
+    end
+
+    params[:note] = row['note']
+
+    if I18n.available_locales.include?(row['locale'].to_s.to_sym)
+      params[:locale] = row['locale']
+    end
+
+    library = Library.where(name: row['library'].to_s.strip).first
+    unless library
+      library = default_library || Library.web
+    end
+    params[:library_id] = library.id if library
+
+    if defined?(EnjuCirculation)
+      params[:checkout_icalendar_token] = row['checkout_icalendar_token'] if row['checkout_icalendar_token'].present?
+      params[:save_checkout_history] = row['save_checkout_history'] if row['save_checkout_history'].present?
+    end
+    if defined?(EnjuSearchLog)
+      params[:save_search_history] = row['save_search_history'] if row['save_search_history'].present?
+    end
+    if defined?(EnjuBookmark)
+      params[:share_bookmarks] = row['share_bookmarks'] if row['share_bookmarks'].present?
+    end
+    params
+  end
 end
 
 # == Schema Information
@@ -208,4 +292,6 @@ end
 #  created_at               :datetime         not null
 #  updated_at               :datetime         not null
 #  user_encoding            :string(255)
+#  default_library_id       :integer
+#  default_user_group_id    :integer
 #

data/app/views/kaminari/_first_page.html.erb

@@ -6,4 +6,4 @@
     per_page:      number of items to fetch per page
     remote:        data-remote
 -%>
-  <%= link_to_unless current_page.first?, raw(t 'views.pagination.first'), url, :remote => remote %>
+  <%= link_to_unless current_page.first?, raw(t 'views.pagination.first'), url, remote: remote %>

data/app/views/kaminari/_last_page.html.erb

@@ -6,4 +6,4 @@
     per_page:      number of items to fetch per page
     remote:        data-remote
 -%>
-  <%= link_to_unless current_page.last?, raw(t 'views.pagination.last'), url, {:remote => remote} %>
+  <%= link_to_unless current_page.last?, raw(t 'views.pagination.last'), url, {remote: remote} %>

data/app/views/kaminari/_next_page.html.erb

@@ -6,4 +6,4 @@
     per_page:      number of items to fetch per page
     remote:        data-remote
 -%>
-  <%= link_to_unless current_page.last?, raw(t 'views.pagination.next'), url, :rel => 'next', :remote => remote %>
+  <%= link_to_unless current_page.last?, raw(t 'views.pagination.next'), url, rel: 'next', remote: remote %>

data/app/views/kaminari/_page.html.erb

@@ -8,5 +8,5 @@
     remote:        data-remote
 -%>
 <% if page.current? %><span class="current"><% end %>
-  <%= link_to_unless page.current?, page, url, opts = {:remote => remote, :rel => page.next? ? 'next' : page.prev? ? 'prev' : nil} %>
+  <%= link_to_unless page.current?, page, url, opts = {remote: remote, rel: page.next? ? 'next' : page.prev? ? 'prev' : nil} %>
 <% if page.current? %></span><% end %>

data/app/views/kaminari/_prev_page.html.erb

@@ -6,4 +6,4 @@
     per_page:      number of items to fetch per page
     remote:        data-remote
 -%>
-  <%= link_to_unless current_page.first?, raw(t 'views.pagination.previous'), url, :rel => 'prev', :remote => remote %>
+  <%= link_to_unless current_page.first?, raw(t 'views.pagination.previous'), url, rel: 'prev', remote: remote %>

data/app/views/kaminari/enju_leaf_mobile/_next_page.html.erb

@@ -8,6 +8,6 @@
 -%>
 <% unless current_page.last? %>
 <li class="next_page mobile_pagination ui-block-c">
-  <%= link_to_unless current_page.last?, raw(t 'views.pagination.next'), url, :rel => 'next', :remote => remote %>
+  <%= link_to_unless current_page.last?, raw(t 'views.pagination.next'), url, rel: 'next', remote: remote %>
 </li>
 <% end %>

data/app/views/kaminari/enju_leaf_mobile/_page.html.erb

@@ -8,5 +8,5 @@
     remote:        data-remote
 -%>
 <li class="page<%= ' active' if page.current? %> mobile_pagination ui-block-b">
-<%= link_to "#{page} / #{total_pages}", url, opts = {:remote => remote, :rel => page.next? ? 'next' : page.prev? ? 'prev' : nil} %>
+<%= link_to "#{page} / #{total_pages}", url, opts = {remote: remote, rel: page.next? ? 'next' : page.prev? ? 'prev' : nil} %>
 </li>

data/app/views/kaminari/enju_leaf_mobile/_prev_page.html.erb

@@ -8,6 +8,6 @@
 -%>
 <% unless current_page.first? %>
 <li class="prev mobile_pagination ui-block-a">
-  <%= link_to_unless current_page.first?, raw(t 'views.pagination.previous'), url, :rel => 'prev', :remote => remote %>
+  <%= link_to_unless current_page.first?, raw(t 'views.pagination.previous'), url, rel: 'prev', remote: remote %>
 </li>
 <% end %>

data/app/views/layouts/application.html.erb

@@ -5,11 +5,11 @@
   <%= render 'page/include' %>
   <title><%= title(controller.controller_name) %></title>
   <%- if controller.controller_name == 'manifestations' and controller.action_name == 'index' -%>
-    <%= auto_discovery_link_tag(:rss, url_for(params.merge(:format => 'rss', :page => nil, :commit => nil, :only_path => true)), :title => (t('manifestation.library_group_manifestation', :library_group_name => @library_group.display_name.localize) + " (RSS)")) %>
-    <%= auto_discovery_link_tag(:atom, url_for(params.merge(:format => 'atom', :page => nil, :commit => nil, :only_path => true)), :title => (t('manifestation.library_group_manifestation', :library_group_name => @library_group.display_name.localize) + " (Atom)")) %>
+    <%= auto_discovery_link_tag(:rss, url_for(params.merge(format: 'rss', page: nil, commit: nil, only_path: true)), title: (t('manifestation.library_group_manifestation', library_group_name: @library_group.display_name.localize) + " (RSS)")) %>
+    <%= auto_discovery_link_tag(:atom, url_for(params.merge(format: 'atom', page: nil, commit: nil, only_path: true)), title: (t('manifestation.library_group_manifestation', library_group_name: @library_group.display_name.localize) + " (Atom)")) %>
     <% if defined?(EnjuNews) %>
-      <%= auto_discovery_link_tag(:rss, news_posts_url(:format => :rss, :locale => @locale), :title => (t('news_post.library_group_news_post', :library_group_name => @library_group.display_name.localize) + " (RSS)")) %>
-      <%= auto_discovery_link_tag(:atom, news_posts_url(:format => :atom, :locale => @locale), :title => (t('news_post.library_group_news_post', :library_group_name => @library_group.display_name.localize) + " (Atom)")) %>
+      <%= auto_discovery_link_tag(:rss, news_posts_url(format: :rss, locale: @locale), title: (t('news_post.library_group_news_post', library_group_name: @library_group.display_name.localize) + " (RSS)")) %>
+      <%= auto_discovery_link_tag(:atom, news_posts_url(format: :atom, locale: @locale), title: (t('news_post.library_group_news_post', library_group_name: @library_group.display_name.localize) + " (Atom)")) %>
     <% end %>
   <% end %>
   <%= book_jacket_header(Setting.book_jacket.source) %>