diff --git a/app/controllers/admin/resource_controller.rb b/app/controllers/admin/resource_controller.rb index b15a576f40..31a7b657f2 100644 --- a/app/controllers/admin/resource_controller.rb +++ b/app/controllers/admin/resource_controller.rb @@ -61,7 +61,7 @@ module Admin def destroy if @object.destroy - flash[:success] = flash_message_for(@object, :successfully_removed) + flash[:success] = Spree.t(:successfully_removed) respond_with(@object) do |format| format.html { redirect_to collection_url } format.js { render partial: "spree/admin/shared/destroy" } @@ -76,7 +76,7 @@ module Admin protected def resource_not_found - flash[:error] = flash_message_for(model_class.new, :not_found) + flash[:error] = Spree.t(:not_found) redirect_to collection_url end diff --git a/app/controllers/admin/tag_rules_controller.rb b/app/controllers/admin/tag_rules_controller.rb index 620de16ee3..79bd815405 100644 --- a/app/controllers/admin/tag_rules_controller.rb +++ b/app/controllers/admin/tag_rules_controller.rb @@ -30,7 +30,7 @@ module Admin status = :ok if @rule.destroy - flash[:success] = Spree.t(:successfully_removed, resource: "Tag Rule") + flash[:success] = Spree.t(:successfully_removed, resource: Spree.t(:tag_rule)) else flash.now[:error] = t(".destroy_error") status = :internal_server_error diff --git a/app/controllers/admin/vouchers_controller.rb b/app/controllers/admin/vouchers_controller.rb index 72bb759e10..272de55966 100644 --- a/app/controllers/admin/vouchers_controller.rb +++ b/app/controllers/admin/vouchers_controller.rb @@ -14,7 +14,7 @@ module Admin ) if @voucher.save - flash[:success] = I18n.t(:successfully_created, resource: "Voucher") + flash[:success] = I18n.t(:successfully_created, resource: Spree.t(:voucher)) redirect_to edit_admin_enterprise_path(@enterprise, anchor: :vouchers_panel) else render_error diff --git a/app/controllers/spree/admin/images_controller.rb b/app/controllers/spree/admin/images_controller.rb index cb1ebe40ac..193b013562 100644 --- a/app/controllers/spree/admin/images_controller.rb +++ b/app/controllers/spree/admin/images_controller.rb @@ -68,7 +68,7 @@ module Spree destroy_before if @object.destroy - flash[:success] = flash_message_for(@object, :successfully_removed) + flash[:success] = Spree.t(:successfully_removed) end redirect_to location_after_save diff --git a/app/controllers/spree/admin/product_properties_controller.rb b/app/controllers/spree/admin/product_properties_controller.rb index aebdff556e..b92208c924 100644 --- a/app/controllers/spree/admin/product_properties_controller.rb +++ b/app/controllers/spree/admin/product_properties_controller.rb @@ -16,7 +16,7 @@ module Spree @url_filters = ::ProductFilters.new.extract(request.query_parameters) if @object.destroy - flash[:success] = flash_message_for(@object, :successfully_removed) + flash[:success] = Spree.t(:successfully_removed) end # if destroy fails it won't show any errors to the user redirect_to spree.admin_product_product_properties_url(params[:product_id], @url_filters) diff --git a/app/controllers/spree/admin/shipping_methods_controller.rb b/app/controllers/spree/admin/shipping_methods_controller.rb index 25433b213f..8b24f0cc09 100644 --- a/app/controllers/spree/admin/shipping_methods_controller.rb +++ b/app/controllers/spree/admin/shipping_methods_controller.rb @@ -36,7 +36,7 @@ module Spree end @object.touch :deleted_at - flash[:success] = flash_message_for(@object, :successfully_removed) + flash[:success] = Spree.t(:successfully_removed) respond_with(@object) do |format| format.html { redirect_to collection_url } diff --git a/app/controllers/spree/admin/tax_categories_controller.rb b/app/controllers/spree/admin/tax_categories_controller.rb index e9d6547f48..7d7e82c86a 100644 --- a/app/controllers/spree/admin/tax_categories_controller.rb +++ b/app/controllers/spree/admin/tax_categories_controller.rb @@ -5,7 +5,7 @@ module Spree class TaxCategoriesController < ::Admin::ResourceController def destroy if @object.destroy - flash[:success] = flash_message_for(@object, :successfully_removed) + flash[:success] = Spree.t(:successfully_removed) respond_with(@object) do |format| format.html { redirect_to collection_url } format.js { render partial: "spree/admin/shared/destroy" } diff --git a/config/locales/en.yml b/config/locales/en.yml index d3fe23123d..8732c9df20 100644 --- a/config/locales/en.yml +++ b/config/locales/en.yml @@ -4195,6 +4195,8 @@ en: logourl: "Logourl" are_you_sure_delete: "Are you sure you want to delete this record?" confirm_delete: "Confirm Deletion" + tag_rule: "Tag Rule" + voucher: "Voucher" configurations: "Configurations" general_settings: "General Settings" diff --git a/spec/system/admin/configuration/tax_categories_spec.rb b/spec/system/admin/configuration/tax_categories_spec.rb index ea5074d8bf..56605a5848 100644 --- a/spec/system/admin/configuration/tax_categories_spec.rb +++ b/spec/system/admin/configuration/tax_categories_spec.rb @@ -55,4 +55,15 @@ RSpec.describe "Tax Categories" do expect(page).to have_content("desc 99") end end + + context "admin deleting a tax category" do + it "should be able to delete an existing tax category" do + create(:tax_category, name: "To be removed") + click_link "Tax Categories" + accept_confirm('Are you sure?') do + within_row(1) { find(".icon-trash").click } + end + expect(page).not_to have_content("To be removed") + end + end end