diff --git a/app/controllers/template_folders_controller.rb b/app/controllers/template_folders_controller.rb index 25e489ce..03f58bf5 100644 --- a/app/controllers/template_folders_controller.rb +++ b/app/controllers/template_folders_controller.rb @@ -55,11 +55,10 @@ class TemplateFoldersController < ApplicationController def selected_order @selected_order ||= - if cookies.permanent[:dashboard_templates_order].blank? || - (cookies.permanent[:dashboard_templates_order] == 'used_at' && can?(:manage, :countless)) + if can?(:manage, :countless) 'created_at' else - cookies.permanent[:dashboard_templates_order] + cookies.permanent[:dashboard_templates_order].presence || 'created_at' end end diff --git a/app/controllers/templates_dashboard_controller.rb b/app/controllers/templates_dashboard_controller.rb index 470833be..787d3a24 100644 --- a/app/controllers/templates_dashboard_controller.rb +++ b/app/controllers/templates_dashboard_controller.rb @@ -102,11 +102,10 @@ class TemplatesDashboardController < ApplicationController def selected_order @selected_order ||= - if cookies.permanent[:dashboard_templates_order].blank? || - (cookies.permanent[:dashboard_templates_order] == 'used_at' && can?(:manage, :countless)) + if can?(:manage, :countless) 'created_at' else - cookies.permanent[:dashboard_templates_order] + cookies.permanent[:dashboard_templates_order].presence || 'created_at' end end diff --git a/app/views/template_folders/show.html.erb b/app/views/template_folders/show.html.erb index c51879d8..527af6ca 100644 --- a/app/views/template_folders/show.html.erb +++ b/app/views/template_folders/show.html.erb @@ -54,8 +54,8 @@ <%= render partial: 'templates/template', collection: @templates %> <% templates_order_select_html = capture do %> - <% if params[:q].blank? && @pagy.pages > 1 %> - <%= render 'shared/templates_order_select', with_recently_used: @pagy.count.present? && @pagy.count < 10_000 && !can?(:manage, :countless), selected_order: %> + <% if params[:q].blank? && @pagy.pages > 1 && !can?(:manage, :countless) %> + <%= render 'shared/templates_order_select', with_recently_used: @pagy.count.present? && @pagy.count < 10_000, selected_order: %> <% end %> <% end %> <%= render 'shared/pagination', pagy: @pagy, items_name: @templates.present? ? 'templates' : 'template_folders', right_additional_html: templates_order_select_html %> diff --git a/app/views/templates_dashboard/index.html.erb b/app/views/templates_dashboard/index.html.erb index b11c3c7c..b09f077d 100644 --- a/app/views/templates_dashboard/index.html.erb +++ b/app/views/templates_dashboard/index.html.erb @@ -45,8 +45,8 @@ <% end %> <% end %> <% templates_order_select_html = capture do %> - <% if params[:q].blank? && @pagy.pages > 1 %> - <%= render 'shared/templates_order_select', with_recently_used: @pagy.count.present? && @pagy.count < 10_000 && !can?(:manage, :countless), selected_order: %> + <% if params[:q].blank? && @pagy.pages > 1 && !can?(:manage, :countless) %> + <%= render 'shared/templates_order_select', with_recently_used: @pagy.count.present? && @pagy.count < 10_000, selected_order: %> <% end %> <% end %> <% if @template_folders.present? || @show_default_folder || @show_shared_folder %>