Sale prices form should only show buyable product

[twnaing]

Sale prices form should only show buyable product (like stock view).

Currently it is including product.master in the form and table.

spree_sales/app/controllers/spree/admin/sale_prices_controller.rb

Lines 49 to 52 in 9fe999c

	@variants = @product.variants_including_master.map do |variant|
	[variant.id, variant.sku_and_options_text]
	end
	@variants.insert(0, [:all_variants, Spree.t(:all_variants)])

        @variants = @product.variants.map do |variant|
          [variant.id, variant.sku_and_options_text]
        end
        if @variants.variants.size > 1
          @variants.push [@product.master.id, @product.master.sku_and_options_text]
        else
          @variants.insert(0, [:all_variants, Spree.t(:all_variants)])
        end

spree_sales/app/views/spree/admin/sale_prices/_form.html.erb

Line 22 in 9fe999c

<%= select_tag "sale_price[variant]", options_from_collection_for_select(@variants, :first, :last, @product.master.id), class: 'select2' %>

        <%= select_tag "sale_price[variant]", options_from_collection_for_select(@variants, :first, :last), class: 'select2' %>