enju_leaf 1.1.0.rc20 → 1.1.0.rc21

data/app/views/user_import_files/_results.html.erb

@@ -0,0 +1,22 @@
+<h2><%= t('activerecord.models.user_import_result') %></h2>
+<table class="table table-striped index">
+  <tr>
+    <th><%= t('activerecord.attributes.user_import_result.lineno') %></th>
+    <th><%= t('activerecord.attributes.user_import_result.body') %></th>
+    <th><%= t('activerecord.models.user') %></th>
+    <th></th>
+  </tr>
+<% @user_import_results.each_with_index do |user_import_result, idx| %>
+  <tr class="line<%= cycle("0", "1") -%>">
+    <td><%= @user_import_results.offset_value + idx + 1 %></td>
+    <td><%= user_import_result.body.inspect %></td>
+    <td><%= link_to user_import_result.user.username, user_import_result.user.profile if user_import_result.user.try(:profile) %></td>
+    <td>
+      <%= link_to t('page.show'), user_import_result %>
+      <%= link_to t('page.destroy'), user_import_result, data: {confirm: t('page.are_you_sure')}, method: :delete %>
+    </td>
+  </tr>
+<% end %>
+</table>
+<%= paginate(@user_import_results) %>
+

data/app/views/user_import_files/show.html.erb

@@ -8,25 +8,18 @@
   <%= @user_import_file.id -%>
 </p>
 
+<% if @user_import_file.user_import_file_name %>
 <p>
   <strong><%= t('activerecord.attributes.user_import_file.user_import_file_name') -%>:</strong>
-  <%= link_to @user_import_file.user_import_file_name, user_import_file_path(@user_import_file, format: :download) if @user_import_file.user_import_file_name %>
-</p>
-
-<p>
-  <strong><%= t('activerecord.attributes.user_import_file.user_import_content_type') -%>:</strong>
-  <%= @user_import_file.user_import_content_type -%>
+  <%= link_to @user_import_file.user_import_file_name, user_import_file_path(@user_import_file, format: :download) %>
+  (<%= @user_import_file.user_import_content_type -%>)
 </p>
 
 <p>
   <strong><%= t('activerecord.attributes.user_import_file.user_import_file_size') -%>:</strong>
   <%= @user_import_file.user_import_file_size -%>
 </p>
-
-<p>
-  <strong><%= t('activerecord.attributes.user_import_file.user_import_fingerprint') -%>:</strong>
-  <%= @user_import_file.user_import_fingerprint -%>
-</p>
+<% end %>
 
 <p>
   <strong><%= t('page.created_at') -%>:</strong>
@@ -73,13 +66,16 @@
   <%= link_to @user_import_file.user.username, @user_import_file.user.profile if @user_import_file.user.try(:profile) -%>
 </p>
 
+<% if @user_import_results %>
+<%= render 'results' %>
+<% end %>
+
 </div>
 </div>
 
 <div id="submenu" class="ui-corner-all ui-widget-content">
   <%= form_for(@user_import_file) do |f| %>
   <ul>
-    <li><%= link_to t('activerecord.models.user_import_result'), user_import_file_user_import_results_path(@user_import_file) -%></li>
     <li><%= back_to_index(flash[:page_info]) -%></li>
     <% if current_user.has_role?('Administrator') %>
       <li><%= link_to 'Resque', resque_path %></li>

data/app/views/user_import_results/_list.html.erb

@@ -0,0 +1,22 @@
+<table class="table table-striped index">
+  <tr>
+    <th><%= t('activerecord.models.user_import_file') %></th>
+    <th><%= t('activerecord.models.user') %></th>
+    <th></th>
+  </tr>
+
+<% @user_import_results.each do |user_import_result| %>
+  <tr class="line<%= cycle("0", "1") -%>">
+    <td>
+      <%= link_to user_import_result.user_import_file.user_import_file_name, user_import_result.user_import_file %>
+      <br />
+      <%=l user_import_result.created_at %>
+    </td>
+    <td><%= link_to user_import_result.user.username, user_import_result.user.profile if user_import_result.user.try(:profile) %></td>
+    <td>
+      <%= link_to t('page.show'), user_import_result %>
+      <%= link_to t('page.destroy'), user_import_result, data: {confirm: t('page.are_you_sure')}, method: :delete %>
+    </td>
+  </tr>
+<% end %>
+</table>

data/app/views/user_import_results/_list_lines.html.erb

@@ -0,0 +1,24 @@
+<ul>
+  <li><%= t('activerecord.models.user_import_file') %>: <%= link_to @user_import_file.user_import_file_name, @user_import_file %>
+  <li><%= t('page.created_at') %>: <%= l(@user_import_file.created_at) %>
+</h2>
+<table class="table table-striped index">
+  <tr>
+    <th><%= t('activerecord.attributes.user_import_result.lineno') %></th>
+    <th><%= t('activerecord.attributes.user_import_result.body') %></th>
+    <th><%= t('activerecord.models.user') %></th>
+    <th></th>
+  </tr>
+
+<% @user_import_results.each_with_index do |user_import_result, idx| %>
+  <tr class="line<%= cycle("0", "1") -%>">
+    <td><%= @user_import_results.offset_value + idx + 1 %></td>
+    <td><%= user_import_result.body.inspect %></td>
+    <td><%= link_to user_import_result.user.username, user_import_result.user.profile if user_import_result.user.try(:profile) %></td>
+    <td>
+      <%= link_to t('page.show'), user_import_result %>
+      <%= link_to t('page.destroy'), user_import_result, data: {confirm: t('page.are_you_sure')}, method: :delete %>
+    </td>
+  </tr>
+<% end %>
+</table>

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

@@ -2,36 +2,11 @@
 <h1 class="title"><%= t('page.listing', model: t('activerecord.models.user_import_result')) -%></h1>
 <div id="content_list">
   <% if @user_import_file %>
-    <h2 class="resource_title">
-      <%= link_to @user_import_file.user_import_file_name, @user_import_file %>
-    </h2>
+    <%= render 'list_lines' %>
+  <% else %>
+    <%= render 'list' %>
   <% end %>
-
-<table class="table table-striped index">
-  <tr>
-    <th><%= t('activerecord.models.user_import_file') %></th>
-    <th><%= t('activerecord.models.user') %></th>
-    <th></th>
-  </tr>
-
-<% @user_import_results.each do |user_import_result| %>
-  <tr class="line<%= cycle("0", "1") -%>">
-    <td>
-      <%= link_to user_import_result.user_import_file.user_import_file_name, user_import_result.user_import_file %>
-      <br />
-      <%=l user_import_result.created_at %>
-    </td>
-    <td><%= link_to user_import_result.user.username, user_import_result.user.profile if user_import_result.user.try(:profile) %></td>
-    <td>
-      <%= link_to t('page.show'), user_import_result %>
-      <%= link_to t('page.destroy'), user_import_result, data: {confirm: t('page.are_you_sure')}, method: :delete %>
-    </td>
-  </tr>
-<% end %>
-</table>
-
-<%= paginate(@user_import_results) %>
-
+  <%= paginate(@user_import_results) %>
 </div>
 </div>
 
@@ -40,7 +15,7 @@
     <li>
     <% if @user_import_file %>
       <%= link_to((image_tag 'icons/page_white_excel.png', size: '16x16', alt: 'TSV', class: 'icon'), user_import_file_user_import_results_path(@user_import_file, format: :txt, locale: @locale.to_s)) -%>
-      (<%= link_to 'TSV', user_import_file_user_import_results_path(format: :txt, locale: @locale.to_s) -%>)
+      (<%= link_to 'TSV', user_import_file_user_import_results_path(@user_import_file, format: :txt, locale: @locale.to_s) -%>)
     <% else %>
       <%= link_to((image_tag 'icons/page_white_excel.png', size: '16x16', alt: 'TSV', class: 'icon'), user_import_results_path(format: :txt, locale: @locale.to_s)) -%>
       (<%= link_to 'TSV', user_import_results_path(format: :txt, locale: @locale.to_s) -%>)

data/app/views/user_import_results/show.html.erb

@@ -15,7 +15,7 @@
 
 <p>
   <b><%= t('activerecord.attributes.user_import_result.body') %></b>
-  <%= @user_import_result.body %>
+  <%= @user_import_result.body.inspect %>
 </p>
 
 <p>

data/config/locales/devise.ja.yml

@@ -51,8 +51,8 @@ ja:
       already_confirmed: "was already confirmed, please try signing in"
       confirmation_period_expired: "needs to be confirmed within %{period}, please request a new one"
       expired: "有効期限が切れています。新しく登録してください。"
-      not_found: "not found"
+      not_found: "は登録されていません。"
       not_locked: "was not locked"
       not_saved:
-        one: "%{resource} を保存するときに1個のエラーが発生しました:"
-        other: "%{resource} を保存するときに%{count}個のエラーが発生しました:"
+        one: "%{resource} の情報に1個のエラーがあります:"
+        other: "%{resource} の情報に%{count}個のエラーがあります:"

data/config/locales/translation_en.yml

@@ -75,7 +75,8 @@ en:
         default_user_group_id: Default user group
         default_library_id: Default library
       user_import_result:
-        body: Body
+        body: Contents
+        lineno: Line
       user_export_file:
         user_export_file_name: Filename
         user_export_file_size: Size
@@ -150,6 +151,7 @@ en:
     start_import: "Start import"
     import_record: "Import records"
     export: "Export"
+    other_format: "Other formats"
     detail: "Detail"
     acquisition: "Acquisition"
     configuration: "Configuration"
@@ -288,6 +290,7 @@ en:
     cannot_parse_yaml_header: "has an invalid YAML header. You must add a space after the header's colon."
     notice: Notice
     system_information: System information
+    default_locale: Default locale
     nosolr_error: 'There is an error on connecting to the search server.'
     nosolr_error_description: 'This error may be caused by a delay when the sever is still waiting for launching the search server. In that case, please wait for a while, and go back to this page later.'
   title:
@@ -304,6 +307,7 @@ en:
     to: "to"
     from_to: "-"
     all_libraries: "All libraries"
+    check_all: "Check/Uncheck All"
   work:
     agents_work: "%{agent_name}'s works"
     library_group_work: "Works in %{library_group_name}"

data/config/locales/translation_ja.yml

@@ -75,7 +75,8 @@ ja:
         default_user_group_id: 既定の利用者グループ
         default_library_id: 既定の図書館
       user_import_result:
-        body: 本文
+        body: 内容
+        lineno: 行数
       user_export_file:
         user_export_file_name: ファイル名
         user_export_file_size: ファイルサイズ
@@ -141,6 +142,7 @@ ja:
     start_import: "インポートを開始"
     import_record: "書誌レコードのインポート"
     export: "エクスポート"
+    other_format: "書き出し"
     detail: "詳細"
     acquisition: "資料の受入"
     configuration: "システムの設定"
@@ -273,6 +275,7 @@ ja:
     cannot_parse_yaml_header: "のYAMLヘッダを読み込めません。ヘッダのコロンの後には、半角スペースを含める必要があります。"
     notice: 注意
     system_information: システムの基本的な情報の表示
+    default_locale: "既定の言語"
     nosolr_error: '検索サーバに不具合があるようです'
     nosolr_error_description: '検索サーバーの起動に時間がかかっている場合もこのエラーとなります。少し時間を置いてアクセスするとエラーが解消される場合もありますのでお試しください。'
   title:
@@ -288,6 +291,7 @@ ja:
     from: ""
     to: "から"
     all_libraries: "全ての図書館"
+    check_all: "全て選択/解除"
   work:
     agent_work: "%{agent_name} の書いた資料"
     library_group_work: "%{library_group_name} にある書いた資料"

data/lib/enju_leaf/calculate_stat.rb

@@ -10,6 +10,7 @@ module CalculateStat
   end
 
   module ClassMethods
+    # 利用統計の集計を開始します。
     def calculate_stat
       self.not_calculated.each do |stat|
         stat.transition_to!(:started)
@@ -18,6 +19,7 @@ module CalculateStat
   end
 
   module InstanceMethods
+    # 利用統計の日付をチェックします。
     def check_date
       if self.start_date and self.end_date
         if self.start_date >= self.end_date
@@ -27,6 +29,7 @@ module CalculateStat
       end
     end
 
+    # 利用統計の集計完了メッセージを送信します。
     def send_message
       sender = User.find(1) #system
       message_template = MessageTemplate.localized_template('counting_completed', user.profile.locale)

data/lib/enju_leaf/engine.rb

@@ -17,7 +17,6 @@ require 'rails_autolink'
 require 'devise-encryptable'
 require 'sitemap_generator'
 require 'redis-rails'
-require 'jquery-rails'
 require 'jquery-ui-rails'
 require 'statesman'
 require 'resque/scheduler/server'

data/lib/enju_leaf/helper.rb

@@ -1,5 +1,6 @@
 module EnjuLeaf
   module EnjuLeafHelper
+    # 使用中のデータベースのアダプタ名を表示します。
     def database_adapter
       case ActiveRecord::Base.connection.adapter_name
       when 'PostgreSQL'
@@ -10,7 +11,8 @@ module EnjuLeaf
         link_to 'SQLite', 'http://www.sqlite.org/'
       end
     end
-  
+
+    # HTMLのtitleに表示されるアクション名を設定します。  
     def title_action_name
       case controller.action_name
       when 'index'
@@ -39,7 +41,9 @@ module EnjuLeaf
     def move_position(object)
       render partial: 'page/position', locals: {object: object}
     end
-  
+
+    # I18nに対応した状態名を表示します。
+    # @param [String] state 状態名
     def localized_state(state)
       case state
       when 'pending'
@@ -57,6 +61,8 @@ module EnjuLeaf
       end
     end
   
+    # I18nに対応した状態名を表示します。
+    # @param [Boolean] bool 状態名
     def localized_boolean(bool)
       case bool.to_s
       when nil
@@ -66,11 +72,13 @@ module EnjuLeaf
         t('page.boolean.false')
       end
     end
-  
+
+    # ログイン中のユーザの権限名を表示します。
     def current_user_role_name
       current_user.try(:role).try(:name) || 'Guest'
     end
   
+    # HTMLのtitleを表示します。
     def title(controller_name)
       string = ''
       unless ['page', 'routing_error', 'my_accounts'].include?(controller_name)
@@ -82,7 +90,9 @@ module EnjuLeaf
       string << LibraryGroup.system_name + ' - Next-L Enju Leaf'
       string.html_safe
     end
-  
+
+    # 前の画面に戻るリンクを表示します。 
+    # @param [Hash] options
     def back_to_index(options = {})
       if options == nil
         options = {}
@@ -91,14 +101,17 @@ module EnjuLeaf
         options.delete(:page) if options[:page].to_i == 1
       end
       unless controller_name == 'test'
-        link_to t('page.listing', model: t("activerecord.models.#{controller_name.singularize}")), url_for(params.merge(controller: controller_name, action: :index, id: nil, only_path: true).merge(options))
+        link_to t('page.listing', model: t("activerecord.models.#{controller_name.singularize}")), url_for(params.merge(controller: controller_name, action: :index, page: nil, id: nil, only_path: true).merge(options))
       end
     end
-  
+
+    # 検索フォームにフォーカスを移動するJavaScriptを表示します。 
     def set_focus_on_search_form
       javascript_tag("$(function(){$('#search_form').focus()})") if @query.blank?
     end
-  
+
+    # Markdownの文字列をパースして表示します。
+    # @param [String] string Markdownの文字列 
     def markdown_helper(string)
       return unless string
       if defined?(Redcarpet)
@@ -112,6 +125,8 @@ module EnjuLeaf
       end
     end
 
+    # ユーザの未読メッセージ数を表示します。
+    # @param [User] user ユーザ
     def messages_count(user)
       Message.search do
         with(:receiver_id).equal_to user.id

data/lib/enju_leaf/import_file.rb

@@ -8,6 +8,7 @@ module ImportFile
       include InstanceMethods
     end
 
+    # 失敗したインポート処理を一括削除します。
     def expire
       self.stucked.find_each do |file|
         file.destroy
@@ -29,6 +30,8 @@ module ImportFile
       end
     end
 
+    # インポートするファイルの文字コードをUTF-8に変換します。
+    # @param [String] line 変換する文字列
     def convert_encoding(line)
       if defined?(CharlockHolmes::EncodingDetector)
         begin
@@ -49,6 +52,7 @@ module ImportFile
       end
     end
 
+    # インポート完了時のメッセージを送信します。
     def send_message
       sender = User.find(1)
       message_template = MessageTemplate.localized_template('import_completed', user.profile.locale)

data/lib/enju_leaf/master_model.rb

@@ -19,6 +19,7 @@ module MasterModel
   end
 
   module InstanceMethods
+    # 表示名を設定します。
     def set_display_name
       self.display_name = "#{I18n.locale}: #{name}" if display_name.blank?
     end

data/lib/enju_leaf/user.rb

@@ -69,61 +69,70 @@ module EnjuLeaf
 
         paginates_per 10
 
+        # 有効期限切れのユーザを一括で使用不可にします。
         def lock_expired_users
           User.find_each do |user|
             user.lock_access! if user.expired? and user.active_for_authentication?
           end
         end
 
+        # ユーザの情報をエクスポートします。
+        # @param [Hash] options
         def export(options = {format: :txt})
           header = %w(
             username
+            full_name
+            full_name_transcription
             email
             user_number
             role
             user_group
             library
             locale
+            locked
+            required_role
             created_at
             updated_at
             expired_at
             keyword_list
             note
+	  )
+	  header += %w(
             checkout_icalendar_token
             save_checkout_history
-            save_search_history
-            share_bookmarks
-          ).join("\t")
+	  ) if defined? EnjuCirculation
+	  header << "save_search_history" if defined? EnjuSearchLog
+	  header << "share_bookmarks" if defined? EnjuBookmark
+	  header = header.join("\t")
           users = User.all.map{|u|
             lines = []
             lines << u.username
+            lines << u.try(:profile).try(:full_name)
+            lines << u.try(:profile).try(:full_name_transcription)
             lines << u.email
             lines << u.try(:profile).try(:user_number)
             lines << u.role.name
             lines << u.try(:profile).try(:user_group).try(:name)
             lines << u.try(:profile).try(:library).try(:name)
             lines << u.try(:profile).try(:locale)
+            lines << u.access_locked?
+            lines << u.try(:profile).try(:required_role).try(:name)
             lines << u.created_at
             lines << u.updated_at
-            lines << u.expired_at
+            lines << u.try(:profile).try(:expired_at)
             lines << u.try(:profile).try(:keyword_list).try(:split).try(:join, "//")
             lines << u.try(:profile).try(:note)
-            if defined?(EnjuCirculation)
+	    if defined? EnjuCirculation
               lines << u.try(:profile).try(:checkout_icalendar_token)
               lines << u.try(:profile).try(:save_checkout_history)
-            else
-              lines << nil
-            end
-            if defined?(EnjuSearchLog)
-              lines << u.try(:save_search_history)
-            else
-              lines << nil
-            end
-            if defined?(EnjuBookmark)
-              lines << u.try(:share_bookmarks)
-            else
-              lines << nil
-            end
+	    end
+	    if defined? EnjuSearchLog
+              lines << u.try(:profile).try(:save_search_history)
+	    end
+	    if defined? EnjuBookmark
+              lines << u.try(:profile).try(:share_bookmarks)
+	    end
+	    lines
           }
           if options[:format] == :txt
             users.map{|u| u.join("\t")}.unshift(header).join("\r\n")
@@ -135,12 +144,17 @@ module EnjuLeaf
     end
 
     module InstanceMethods
+      # ユーザにパスワードが必要かどうかをチェックします。
+      # @return [Boolean]
       def password_required?
         if Devise.mappings[:user].modules.include?(:database_authenticatable)
           !persisted? || !password.nil? || !password_confirmation.nil?
         end
       end
 
+      # ユーザが特定の権限を持っているかどうかをチェックします。
+      # @param [String] role_in_question 権限名
+      # @return [Boolean]
       def has_role?(role_in_question)
         return false unless role
         return true if role.name == role_in_question
@@ -154,6 +168,7 @@ module EnjuLeaf
         end
       end
 
+      # ユーザに使用不可の設定を反映させます。
       def set_lock_information
         if locked == '1' and self.active_for_authentication?
           lock_access!
@@ -169,6 +184,8 @@ module EnjuLeaf
         end
       end
 
+      # ユーザが有効期限切れかどうかをチェックし、期限切れであれば使用不可に設定します。
+      # @return [Object]
       def check_expiration
         return if has_role?('Administrator')
         if expired_at
@@ -178,6 +195,8 @@ module EnjuLeaf
         end
       end
 
+      # ユーザの削除前に、管理者ユーザが不在にならないかどうかをチェックします。
+      # @return [Object]
       def check_role_before_destroy
         if has_role?('Administrator')
           if Role.where(name: 'Administrator').first.users.count == 1
@@ -186,26 +205,36 @@ module EnjuLeaf
         end
       end
 
+      # ユーザに自動生成されたパスワードを設定します。
+      # @return [String]
       def set_auto_generated_password
         password = Devise.friendly_token[0..7]
         self.password = password
         self.password_confirmation = password
       end
 
+      # ユーザが有効期限切れかどうかをチェックします。
+      # @return [Boolean]
       def expired?
         if expired_at
           true if expired_at.beginning_of_day < Time.zone.now.beginning_of_day
         end
       end
 
+      # ユーザが管理者かどうかをチェックします。
+      # @return [Boolean]
       def is_admin?
-        true if has_role?('Administrator')
+        return true if has_role?('Administrator')
+        false
       end
 
+      # ユーザがシステム上の最後のLibrarian権限ユーザかどうかをチェックします。
+      # @return [Boolean]
       def last_librarian?
         if has_role?('Librarian')
           role = Role.where(name: 'Librarian').first
-          true if role.users.size == 1
+          return true if role.users.count == 1
+          false
         end
       end
 
@@ -213,6 +242,9 @@ module EnjuLeaf
         Devise::Mailer.confirmation_instructions(self).deliver if email.present?
       end
 
+      # ユーザが削除可能かどうかをチェックします。
+      # @param [User] current_user ユーザ
+      # @return [Object]
       def deletable_by?(current_user)
         return nil unless current_user
         if defined?(EnjuCirculation)

data/lib/enju_leaf/version.rb

@@ -1,3 +1,3 @@
 module EnjuLeaf
-  VERSION = "1.1.0.rc20"
+  VERSION = "1.1.0.rc21"
 end

data/lib/generators/enju_leaf/setup/setup_generator.rb

@@ -114,7 +114,7 @@ EOS
       "config.serve_static_assets = true"
     gsub_file 'config/environments/production.rb',
       /# config.cache_store = :mem_cache_store$/,
-      "config.cache_store = :dalli_store, {:compress => true}"
+      "config.cache_store = :redis_store, ENV['REDIS_URL'], { expires_in: 1.day }"
     gsub_file 'config/environments/production.rb',
       /# config.assets.precompile \+= %w\( search.js \)$/,
       "config.assets.precompile += %w( mobile.js mobile.css print.css )"

data/spec/controllers/my_accounts_controller_spec.rb

@@ -1,5 +1,5 @@
 # -*- encoding: utf-8 -*-
-require 'spec_helper'
+require 'rails_helper'
 
 describe MyAccountsController do
   fixtures :all
@@ -29,7 +29,7 @@ describe MyAccountsController do
     describe "When logged in as Administrator" do
       before(:each) do
         profile = FactoryGirl.create(:profile)
-        @user = User.find('enjuadmin')
+        @user = FactoryGirl.create(:admin)
         @user.profile = profile
         sign_in @user
       end