diff --git a/app/helpers/spree/admin/base_helper_decorator.rb b/app/helpers/spree/admin/base_helper_decorator.rb index c1fd1ab223..85e2f83781 100644 --- a/app/helpers/spree/admin/base_helper_decorator.rb +++ b/app/helpers/spree/admin/base_helper_decorator.rb @@ -31,8 +31,7 @@ module Spree html_options = {class: "remove_fields #{options[:class]}", data: {action: 'remove'}, title: t(:remove)} html_options.merge!(options[:html]) if options.key? :html - link_to_with_icon('icon-trash', name, '#', html_options).gsub('href="#" ', '').html_safe + - f.hidden_field(:_destroy) + link_to_with_icon('icon-trash', name, '#', html_options).gsub('href="#" ', '') + f.hidden_field(:_destroy) end end end diff --git a/spec/helpers/spree/admin/base_helper_spec.rb b/spec/helpers/spree/admin/base_helper_spec.rb new file mode 100644 index 0000000000..5daaaf8c28 --- /dev/null +++ b/spec/helpers/spree/admin/base_helper_spec.rb @@ -0,0 +1,15 @@ +require 'spec_helper' + +describe Spree::BaseHelper, type: :helper do + describe "#link_to_remove_fields_without_url" do + let(:name) { 'Hola' } + let(:form) { double('form_for', hidden_field: '') } + let(:options) { {} } + + subject { helper.link_to_remove_fields_without_url(name, form, options) } + + it 'returns an `a` tag followed by a hidden `input` tag' do + expect(subject).to eq("Hola") + end + end +end