solidus_admin 0.3.1 → 0.4.0

data/app/controllers/solidus_admin/store_credits_controller.rb

@@ -0,0 +1,231 @@
+# frozen_string_literal: true
+
+module SolidusAdmin
+  class StoreCreditsController < SolidusAdmin::ResourcesController
+    before_action :set_user
+    before_action :set_store_credit, only: [:show, :edit_amount, :update_amount, :edit_memo, :update_memo, :edit_validity, :invalidate]
+    before_action :set_store_credit_reasons, only: [:edit_amount, :update_amount, :edit_validity, :invalidate]
+    before_action :set_store_credit_events, only: [:show]
+    before_action :set_store_credit_categories, only: [:new]
+
+    def index
+      @store_credits = Spree::StoreCredit.where(user_id: @user.id).order(id: :desc)
+
+      respond_to do |format|
+        format.html { render component("users/store_credits/index").new(user: @user, store_credits: @store_credits) }
+      end
+    end
+
+    def show
+      respond_to do |format|
+        format.html { render component("users/store_credits/show").new(user: @user, store_credit: @store_credit, events: @store_credit_events) }
+      end
+    end
+
+    def new
+      @store_credit ||= Spree::StoreCredit.new
+
+      respond_to do |format|
+        format.html {
+          render component("users/store_credits/new").new(
+            user: @user,
+            store_credit: @store_credit,
+            categories: @store_credit_categories
+          )
+        }
+      end
+    end
+
+    def create
+      @store_credit = @user.store_credits.build(
+        permitted_resource_params.merge({
+          created_by: spree_current_user,
+          action_originator: spree_current_user
+        })
+      )
+
+      return unless ensure_amount { render_new_with_errors }
+      return unless ensure_store_credit_category { render_new_with_errors }
+
+      if @store_credit.save
+        flash[:notice] = t('.success')
+        redirect_to after_create_path, status: :see_other
+      else
+        render_new_with_errors
+      end
+    end
+
+    def edit_amount
+      respond_to do |format|
+        format.html {
+          render component("users/store_credits/edit_amount").new(
+            user: @user,
+            store_credit: @store_credit,
+            reasons: @store_credit_reasons
+          )
+        }
+      end
+    end
+
+    def update_amount
+      return unless ensure_amount { render_edit_with_errors }
+      return unless ensure_store_credit_reason { render_edit_with_errors }
+
+      if @store_credit.update_amount(permitted_resource_params[:amount], @store_credit_reason, spree_current_user)
+        flash[:notice] = t('.success')
+        redirect_to after_update_path, status: :see_other
+      else
+        render_edit_with_errors
+      end
+    end
+
+    def edit_memo
+      respond_to do |format|
+        format.html {
+          render component("users/store_credits/edit_memo").new(
+            user: @user,
+            store_credit: @store_credit,
+          )
+        }
+      end
+    end
+
+    def update_memo
+      if @store_credit.update(memo: permitted_resource_params[:memo])
+        flash[:notice] = t('.success')
+      else
+        # Memo update failures are nearly impossible to trigger due to lack of validation.
+        flash[:error] = t('.failure')
+      end
+
+      redirect_to after_update_path, status: :see_other
+    end
+
+    def edit_validity
+      respond_to do |format|
+        format.html {
+          render component("users/store_credits/edit_validity").new(
+            user: @user,
+            store_credit: @store_credit,
+            reasons: @store_credit_reasons
+          )
+        }
+      end
+    end
+
+    def invalidate
+      return unless ensure_store_credit_reason { render_edit_with_errors }
+
+      if @store_credit.invalidate(@store_credit_reason, spree_current_user)
+        flash[:notice] = t('.success')
+      else
+        # Ensure store_credit_reason handles invalid param/form submissions and modal re-rendering.
+        # This is just a fallback error state in case anything goes wrong with StoreCredit#invalidate.
+        flash[:error] = t('.failure')
+      end
+
+      redirect_to after_update_path, status: :see_other
+    end
+
+    private
+
+    def resource_class = Spree::StoreCredit
+
+    def set_store_credit
+      @store_credit ||= Spree::StoreCredit.find(params[:id])
+    end
+
+    def set_user
+      @user = Spree.user_class.find(params[:user_id])
+    end
+
+    def set_store_credit_reasons
+      @store_credit_reasons = Spree::StoreCreditReason.active.order(:name)
+    end
+
+    def set_store_credit_categories
+      @store_credit_categories = Spree::StoreCreditCategory.all.order(:name)
+    end
+
+    def set_store_credit_events
+      @store_credit_events = @store_credit.store_credit_events.chronological
+    end
+
+    def permitted_resource_params
+      permitted_params = [:amount, :currency, :category_id, :memo]
+      permitted_params << :category_id if action_name.to_sym == :create
+      permitted_params << :store_credit_reason_id if [:update_amount, :invalidate].include?(action_name.to_sym)
+
+      params.require(:store_credit).permit(permitted_params).merge(created_by: spree_current_user)
+    end
+
+    def render_new_with_errors
+      set_store_credit_categories
+
+      page_component = component("users/store_credits/new").new(
+        user: @user,
+        store_credit: @store_credit,
+        categories: @store_credit_categories
+      )
+
+      render_resource_form_with_errors(page_component)
+    end
+
+    def render_edit_with_errors
+      set_store_credit_events
+
+      template = if action_name.to_sym == :invalidate
+        "edit_validity"
+      else
+        "edit_amount"
+      end
+
+      page_component = component("users/store_credits/#{template}").new(
+        user: @user,
+        store_credit: @store_credit,
+        reasons: @store_credit_reasons
+      )
+
+      render_resource_form_with_errors(page_component)
+    end
+
+    def ensure_amount
+      if permitted_resource_params[:amount].blank?
+        @store_credit.errors.add(:amount, :greater_than, count: 0, value: permitted_resource_params[:amount])
+        yield if block_given? # Block is for error template rendering on a per-action basis so this can be re-used.
+        return false
+      end
+      true
+    end
+
+    def ensure_store_credit_reason
+      @store_credit_reason = Spree::StoreCreditReason.find_by(id: permitted_resource_params[:store_credit_reason_id])
+
+      if @store_credit_reason.blank?
+        @store_credit.errors.add(:store_credit_reason_id, "Store Credit reason must be provided")
+        yield if block_given? # Block is for error template rendering on a per-action basis so this can be re-used.
+        return false
+      end
+      true
+    end
+
+    def ensure_store_credit_category
+      @store_credit_category = Spree::StoreCreditCategory.find_by(id: permitted_resource_params[:category_id])
+
+      if @store_credit_category.blank?
+        @store_credit.errors.add(:category_id, "Store Credit category must be provided")
+        yield if block_given? # Block is for error template rendering on a per-action basis so this can be re-used.
+        return false
+      end
+      true
+    end
+
+    def after_create_path
+      solidus_admin.user_store_credits_path(@user)
+    end
+
+    def after_update_path
+      solidus_admin.user_store_credit_path(@user, @store_credit)
+    end
+  end
+end

data/app/controllers/solidus_admin/tax_categories_controller.rb

@@ -1,119 +1,13 @@
 # frozen_string_literal: true
 
 module SolidusAdmin
-  class TaxCategoriesController < SolidusAdmin::BaseController
-    include SolidusAdmin::ControllerHelpers::Search
-
-    before_action :set_tax_category, only: %i[edit update]
-
-    def new
-      @tax_category = Spree::TaxCategory.new
-
-      set_index_page
-
-      respond_to do |format|
-        format.html { render component('tax_categories/new').new(page: @page, tax_category: @tax_category) }
-      end
-    end
-
-    def edit
-      @tax_category = Spree::TaxCategory.find(params[:id])
-
-      set_index_page
-
-      respond_to do |format|
-        format.html { render component('tax_categories/edit').new(page: @page, tax_category: @tax_category) }
-      end
-    end
-
-    def create
-      @tax_category = Spree::TaxCategory.new(tax_category_params)
-
-      if @tax_category.save
-        respond_to do |format|
-          flash[:notice] = t('.success')
-
-          format.html do
-            redirect_to solidus_admin.tax_categories_path, status: :see_other
-          end
-
-          format.turbo_stream do
-            # we need to explicitly write the refresh tag for now.
-            # See https://github.com/hotwired/turbo-rails/issues/579
-            render turbo_stream: '<turbo-stream action="refresh" />'
-          end
-        end
-      else
-        set_index_page
-
-        respond_to do |format|
-          format.html do
-            page_component = component('tax_categories/new').new(page: @page, tax_category: @tax_category)
-            render page_component, status: :unprocessable_entity
-          end
-        end
-      end
-    end
-
-    def update
-      if @tax_category.update(tax_category_params)
-        respond_to do |format|
-          flash[:notice] = t('.success')
-
-          format.html do
-            redirect_to solidus_admin.tax_categories_path, status: :see_other
-          end
-
-          format.turbo_stream do
-            render turbo_stream: '<turbo-stream action="refresh" />'
-          end
-        end
-      else
-        set_index_page
-
-        respond_to do |format|
-          format.html do
-            page_component = component('tax_categories/edit').new(page: @page, tax_category: @tax_category)
-            render page_component, status: :unprocessable_entity
-          end
-        end
-      end
-    end
-
-    def index
-      set_index_page
-
-      respond_to do |format|
-        format.html { render component('tax_categories/index').new(page: @page) }
-      end
-    end
-
-    def destroy
-      @tax_categories = Spree::TaxCategory.where(id: params[:id])
-
-      Spree::TaxCategory.transaction { @tax_categories.destroy_all }
-
-      flash[:notice] = t('.success')
-      redirect_back_or_to tax_categories_path, status: :see_other
-    end
-
+  class TaxCategoriesController < SolidusAdmin::ResourcesController
     private
 
-    def set_tax_category
-      @tax_category = Spree::TaxCategory.find(params[:id])
-    end
+    def resource_class = Spree::TaxCategory
 
-    def tax_category_params
+    def permitted_resource_params
       params.require(:tax_category).permit(:name, :description, :is_default, :tax_code)
     end
-
-    def set_index_page
-      tax_categories = apply_search_to(
-        Spree::TaxCategory.order(created_at: :desc, id: :desc),
-        param: :q,
-      )
-
-      set_page_and_extract_portion_from(tax_categories)
-    end
   end
 end

data/app/helpers/solidus_admin/void_elements_helper.rb

@@ -0,0 +1,13 @@
+# frozen_string_literal: true
+
+module SolidusAdmin
+  module VoidElementsHelper
+    # https://github.com/rails/rails/blob/194d697036c61af0caa66de5659721ded2478ce9/actionview/lib/action_view/helpers/tag_helper.rb#L84
+    HTML_VOID_ELEMENTS = %i(area base br col embed hr img input keygen link meta source track wbr)
+
+    # @param element [Symbol]
+    def void_element?(element)
+      HTML_VOID_ELEMENTS.include?(element)
+    end
+  end
+end

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

@@ -5,6 +5,9 @@
 
     <title><%= solidus_admin_title %></title>
     <meta name="viewport" content="width=device-width,initial-scale=1">
+    <meta name="turbo-cache-control" content="no-cache">
+    <meta name="turbo-prefetch" content="false">
+
     <%= csrf_meta_tags %>
     <%= csp_meta_tag %>
 

data/config/locales/menu_item.en.yml

@@ -6,7 +6,6 @@ en:
       option_types: Option Types
       property_types: Properties
       taxonomies: Taxonomies
-      taxons: Display Order
       promotions: Promotions
       promotion_categories: Promotion Categories
       stock: Stock

data/config/locales/properties.en.yml

@@ -2,5 +2,9 @@ en:
   solidus_admin:
     properties:
       title: "Properties"
+      create:
+        success: "Property was successfully created."
+      update:
+        success: "Property was successfully updated."
       destroy:
         success: "Properties were successfully removed."

data/config/locales/store_credits.en.yml

@@ -0,0 +1,16 @@
+en:
+  solidus_admin:
+    store_credits:
+      title: "Store Credits"
+      destroy:
+        success: "Store credit was successfully removed."
+      create:
+        success: "Store credit was successfully created."
+      update_amount:
+        success: "Store credit was successfully updated."
+      update_memo:
+        success: "Store credit was successfully updated."
+        failure: "Something went wrong. Store credit could not be updated."
+      invalidate:
+        success: "Store credit was successfully invalidated."
+        failure: "Something went wrong. Store credit could not be invalidated."

data/config/routes.rb

@@ -52,10 +52,21 @@ SolidusAdmin::Engine.routes.draw do
       get :orders
       get :items
     end
+
+    resources :store_credits, only: [:index, :show, :new, :create], controller: "store_credits" do
+      member do
+        get :edit_amount
+        put :update_amount
+        get :edit_memo
+        put :update_memo
+        get :edit_validity
+        put :invalidate
+      end
+    end
   end
 
   admin_resources :promotions, only: [:index, :destroy]
-  admin_resources :properties, only: [:index, :destroy]
+  admin_resources :properties, except: [:show]
   admin_resources :option_types, only: [:index, :destroy], sortable: true
   admin_resources :taxonomies, only: [:index, :destroy], sortable: true
   admin_resources :promotion_categories, only: [:index, :destroy]

data/lib/generators/solidus_admin/install/install_generator.rb

@@ -38,6 +38,8 @@ module SolidusAdmin
           gem "actioncable"
         end
 
+        execute_command :bundle, :install
+
         route "mount Lookbook::Engine, at: '#{solidus_mount_point}lookbook' if Rails.env.development?"
       end
 

data/lib/solidus_admin/component_registry.rb

@@ -28,7 +28,7 @@ module SolidusAdmin
       prefix = "#{SolidusAdmin::Configuration::ENGINE_ROOT}/app/components/solidus_admin/"
       suffix = "/component.rb"
       dictionary = Dir["#{prefix}**#{suffix}"].map { _1.delete_prefix(prefix).delete_suffix(suffix) }
-      corrections = DidYouMean::SpellChecker.new(dictionary: dictionary).correct(key.to_s)
+      corrections = DidYouMean::SpellChecker.new(dictionary:).correct(key.to_s)
 
       raise ComponentNotFoundError.new(
         "Unknown component #{key}#{DidYouMean.formatter.message_for(corrections)}",

data/lib/solidus_admin/configuration.rb

@@ -102,11 +102,6 @@ module SolidusAdmin
               key: "taxonomies",
               route: -> { spree.admin_taxonomies_path },
               position: 30
-            },
-            {
-              key: "taxons",
-              route: -> { spree.admin_taxons_path },
-              position: 40
             }
           ]
         },
@@ -169,10 +164,10 @@ module SolidusAdmin
         {
           position: index,
           key: item.label,
-          icon: icon,
-          route: route,
+          icon:,
+          route:,
           children: item.children.map.with_index(&menu_item_to_hash),
-          match_path: match_path,
+          match_path:,
         }
       end
 

data/lib/solidus_admin/engine.rb

@@ -63,5 +63,11 @@ module SolidusAdmin
         SolidusAdmin::Engine.root.join("app/components"),
       ]
     end
+
+    initializer "solidus_admin.routing_proxies" do |app|
+      ActiveSupport.on_load(:after_routes_loaded) do
+        SolidusAdmin::BaseComponent.include app.routes.mounted_helpers
+      end
+    end
   end
 end

data/lib/solidus_admin/install_tailwindcss.rb

@@ -8,7 +8,7 @@ input_path = "app/assets/stylesheets/solidus_admin/application.tailwind.css"
 output_path = "app/assets/builds/solidus_admin/tailwind.css"
 
 unless bundle_command "show tailwindcss-rails"
-  bundle_command "add tailwindcss-rails"
+  bundle_command "add tailwindcss-rails --version '~>3.0'"
 end
 
 # Copy the Tailwind CSS main file.

data/lib/solidus_admin/testing_support/component_helpers.rb

@@ -12,15 +12,9 @@ module SolidusAdmin
       #      "Rendered"
       #    end
       #  end
-      def mock_component(&definition)
-        location = caller(1, 1).first
-        component_class = Class.new(SolidusAdmin::BaseComponent)
-        # ViewComponent will complain if we don't fake a class name:
-        # @see https://github.com/ViewComponent/view_component/blob/5decd07842c48cbad82527daefa3fe9c65a4226a/lib/view_component/base.rb#L371
-        component_class.define_singleton_method(:name) { "Foo" }
-        component_class.define_singleton_method(:to_s) { "#{name} (#{location})" }
-        component_class.class_eval(&definition) if definition
-        component_class
+      def mock_component(class_name = "Foo::Component", &definition)
+        component_class = stub_const(class_name, Class.new(described_class, &definition))
+        component_class.new
       end
     end
   end

data/lib/solidus_admin/testing_support/feature_helpers.rb

@@ -19,7 +19,7 @@ module SolidusAdmin
       end
 
       def find_row(text)
-        find('table tbody tr', text: text)
+        find('table tbody tr', text:)
       end
 
       def find_row_checkbox(text)

data/lib/solidus_admin/testing_support/shared_examples/bulk_delete_resources.rb

@@ -0,0 +1,20 @@
+# frozen_string_literal: true
+
+RSpec.shared_examples_for 'feature: bulk delete resources' do
+  it 'allows to bulk delete resources' do
+    create(resource_factory, name: 'Bulk delete item 1')
+    create(resource_factory, name: 'Bulk delete item 2')
+
+    visit index_path
+    expect(page).to have_content('Bulk delete item 1')
+    expect(page).to have_content('Bulk delete item 2')
+
+    select_row('Bulk delete item 1')
+    select_row('Bulk delete item 2')
+    click_on 'Delete'
+
+    expect(page).to have_content('were successfully removed.')
+    expect(page).not_to have_content('Bulk delete item 1')
+    expect(page).not_to have_content('Bulk delete item 2')
+  end
+end