From 7158d801ac3de74b586ad319df8e49cd2bbd5bd0 Mon Sep 17 00:00:00 2001 From: Rainer Dema Date: Tue, 5 Dec 2023 18:13:20 +0100 Subject: [PATCH] Add `stores/index` component with dedicated actions --- .../stores/index/component.html.erb | 34 +++++++++++ .../solidus_admin/stores/index/component.rb | 59 +++++++++++++++++++ .../solidus_admin/stores/index/component.yml | 4 ++ .../solidus_admin/stores_controller.rb | 40 +++++++++++++ admin/config/locales/stores.en.yml | 6 ++ admin/config/routes.rb | 1 + admin/spec/features/stores_spec.rb | 24 ++++++++ 7 files changed, 168 insertions(+) create mode 100644 admin/app/components/solidus_admin/stores/index/component.html.erb create mode 100644 admin/app/components/solidus_admin/stores/index/component.rb create mode 100644 admin/app/components/solidus_admin/stores/index/component.yml create mode 100644 admin/app/controllers/solidus_admin/stores_controller.rb create mode 100644 admin/config/locales/stores.en.yml create mode 100644 admin/spec/features/stores_spec.rb diff --git a/admin/app/components/solidus_admin/stores/index/component.html.erb b/admin/app/components/solidus_admin/stores/index/component.html.erb new file mode 100644 index 00000000000..fcd454b2e0b --- /dev/null +++ b/admin/app/components/solidus_admin/stores/index/component.html.erb @@ -0,0 +1,34 @@ +<%= page do %> + <%= page_header do %> + <%= page_header_title title %> + <%= page_header_actions do %> + <%= render component("ui/button").new( + tag: :a, + text: t('.add'), + href: spree.new_admin_store_path, + icon: "add-line", + ) %> + <% end %> + <% end %> + + <%= render component('ui/table').new( + id: stimulus_id, + data: { + class: Spree::Store, + rows: @page.records, + url: ->(store) { spree.edit_admin_store_path(store) }, + prev: prev_page_path, + next: next_page_path, + columns: columns, + batch_actions: batch_actions, + }, + search: { + name: :q, + value: params[:q], + url: solidus_admin.stores_path, + searchbar_key: :name_or_url_or_code_cont, + filters: filters, + scopes: scopes, + }, + ) %> +<% end %> diff --git a/admin/app/components/solidus_admin/stores/index/component.rb b/admin/app/components/solidus_admin/stores/index/component.rb new file mode 100644 index 00000000000..486dee8d77c --- /dev/null +++ b/admin/app/components/solidus_admin/stores/index/component.rb @@ -0,0 +1,59 @@ +# frozen_string_literal: true + +class SolidusAdmin::Stores::Index::Component < SolidusAdmin::BaseComponent + include SolidusAdmin::Layout::PageHelpers + + def initialize(page:) + @page = page + end + + def title + Spree::Store.model_name.human.pluralize + end + + def prev_page_path + solidus_admin.url_for(**request.params, page: @page.number - 1, only_path: true) unless @page.first? + end + + def next_page_path + solidus_admin.url_for(**request.params, page: @page.next_param, only_path: true) unless @page.last? + end + + def batch_actions + [ + { + display_name: t('.batch_actions.delete'), + action: solidus_admin.stores_path, + method: :delete, + icon: 'delete-bin-7-line', + }, + ] + end + + def filters + [] + end + + def scopes + [] + end + + def columns + [ + :name, + :url, + { + header: :slug, + data: ->(store) do + content_tag :div, store.code + end + }, + { + header: :default, + data: ->(store) do + store.default? ? component('ui/badge').yes : component('ui/badge').no + end + }, + ] + end +end diff --git a/admin/app/components/solidus_admin/stores/index/component.yml b/admin/app/components/solidus_admin/stores/index/component.yml new file mode 100644 index 00000000000..54418702ad3 --- /dev/null +++ b/admin/app/components/solidus_admin/stores/index/component.yml @@ -0,0 +1,4 @@ +en: + add: 'Add new' + batch_actions: + delete: 'Delete' diff --git a/admin/app/controllers/solidus_admin/stores_controller.rb b/admin/app/controllers/solidus_admin/stores_controller.rb new file mode 100644 index 00000000000..ef60bcf8298 --- /dev/null +++ b/admin/app/controllers/solidus_admin/stores_controller.rb @@ -0,0 +1,40 @@ +# frozen_string_literal: true + +module SolidusAdmin + class StoresController < SolidusAdmin::BaseController + include SolidusAdmin::ControllerHelpers::Search + + def index + stores = apply_search_to( + Spree::Store.order(id: :desc), + param: :q + ) + + set_page_and_extract_portion_from(stores) + + respond_to do |format| + format.html { render component('stores/index').new(page: @page) } + end + end + + def destroy + @stores = Spree::Store.where(id: params[:id]) + + Spree::Store.transaction { @stores.destroy_all } + + flash[:notice] = t('.success') + redirect_back_or_to stores_path, status: :see_other + end + + private + + def load_store + @store = Spree::Store.find_by!(number: params[:id]) + authorize! action_name, @store + end + + def store_params + params.require(:store).permit(:store_id, permitted_store_attributes) + end + end +end diff --git a/admin/config/locales/stores.en.yml b/admin/config/locales/stores.en.yml new file mode 100644 index 00000000000..4f901a50d6d --- /dev/null +++ b/admin/config/locales/stores.en.yml @@ -0,0 +1,6 @@ +en: + solidus_admin: + stores: + title: "Stores" + destroy: + success: "Stores were successfully removed." diff --git a/admin/config/routes.rb b/admin/config/routes.rb index fd7dcdaeda5..f1c9f46868e 100644 --- a/admin/config/routes.rb +++ b/admin/config/routes.rb @@ -44,4 +44,5 @@ admin_resources :shipping_methods, only: [:index, :destroy] admin_resources :shipping_categories, only: [:index, :destroy] admin_resources :stock_locations, only: [:index, :destroy] + admin_resources :stores, only: [:index, :destroy] end diff --git a/admin/spec/features/stores_spec.rb b/admin/spec/features/stores_spec.rb new file mode 100644 index 00000000000..921393a584d --- /dev/null +++ b/admin/spec/features/stores_spec.rb @@ -0,0 +1,24 @@ +# frozen_string_literal: true + +require 'spec_helper' + +describe "Stores", :js, type: :feature do + before { sign_in create(:admin_user, email: 'admin@example.com') } + + it "lists stores and allows deleting them" do + create(:store, name: "B2C Store") + create(:store, name: "B2B Store") + + visit "/admin/stores" + expect(page).to have_content("B2C Store") + expect(page).to have_content("B2B Store") + expect(page).to be_axe_clean + + select_row("B2C Store") + click_on "Delete" + expect(page).to have_content("Stores were successfully removed.") + expect(page).not_to have_content("B2C Store") + expect(Spree::Store.count).to eq(1) + expect(page).to be_axe_clean + end +end