Save option to display only own label formats in users (issue hitobit…

…o#76)

app/abilities/person_ability.rb

@@ -13,7 +13,8 @@ class PersonAbility < AbilityDsl::Base
     class_side(:index, :query).everybody
 
     permission(:any).may(:show, :show_full, :history, :update,
-                         :update_email, :primary_group, :log).
+                         :update_email, :primary_group, :log,
+                         :label_format_settings).
                      herself
 
     permission(:contact_data).may(:show).other_with_contact_data

app/controllers/label_format/settings_controller.rb

@@ -0,0 +1,21 @@
+class LabelFormat::SettingsController < ApplicationController
+  before_action :authorize
+
+  def update
+    if params[:display_only_own_label_formats].blank?
+      user.update!(display_only_own_label_formats: false)
+    else
+      user.update!(display_only_own_label_formats: true)
+    end
+  end
+
+  private
+
+  def user
+    current_user
+  end
+
+  def authorize
+    authorize!(:label_format_settings, current_user)
+  end
+end

app/helpers/dropdown/people_export.rb

@@ -74,7 +74,11 @@ def add_last_used_format_item(parent)
     end
 
     def add_label_format_items(parent)
-      LabelFormat.list.for_user(user).each do |label_format|
+      label_formats = LabelFormat.all
+      if user.display_only_own_label_formats?
+        label_formats = label_formats.where(user: user)
+      end
+      label_formats.list.for_user(user).each do |label_format|
         parent.sub_items << Item.new(label_format, export_label_format_path(label_format.id),
                                      target: :new, class: 'export-label-format')
       end

app/models/label_format.rb

@@ -33,6 +33,8 @@ def available_page_sizes
 
   has_many :people, foreign_key: :last_label_format_id, dependent: :nullify
 
+  belongs_to :user
+
 
   validates :name, presence: true, length: { maximum: 255, allow_nil: true }
   validates :page_size, inclusion: available_page_sizes

app/views/label_format/settings/update.js.haml

@@ -0,0 +1,2 @@
+var onlyOwn = $('#display_only_own_label_formats').get(0).checked;
+// do something?

app/views/label_formats/index.html.haml

@@ -7,9 +7,17 @@
 
 #main
   -if can?(:manage_global, LabelFormat)
-    %label
-      %input#admin_toggle{ type: 'checkbox', data: {hide: 'admin_labels'} }
-        = I18n.t('label_formats.see_own_labels')
+    = check_box_tag(:display_only_own_label_formats,
+      true,
+      current_user.display_only_own_label_formats,
+      id: 'display_only_own_label_formats',
+      class: 'switcher',
+      data: { remote: true,
+              url: label_format_settings_path,
+              method: :put })
+    %label{for: 'display_only_own_label_formats'}
+     
+    %strong= t('label_formats.see_own_labels')
 
   %h2= I18n.t('label_formats.own_labels')
   = render 'list'

config/routes.rb

@@ -176,7 +176,11 @@
 
     resources :qualification_kinds
 
-    resources :label_formats
+    resources :label_formats do
+      collection do
+        resource :settings, controller: 'label_format/settings', as: 'label_format_settings'
+      end
+    end
 
     resources :custom_contents, only: [:index, :edit, :update]
     get 'custom_contents/:id' => 'custom_contents#edit'

db/migrate/20161214145725_add_display_only_own_label_formats_to_people.rb

@@ -0,0 +1,5 @@
+class AddDisplayOnlyOwnLabelFormatsToPeople < ActiveRecord::Migration
+  def change
+    add_column :people, :display_only_own_label_formats, :boolean, default: false
+  end
+end