diff --git a/app/views/spree/admin/orders/_form.html.haml b/app/views/spree/admin/orders/_form.html.haml index 1b3c17a72f..bebe2dea1d 100644 --- a/app/views/spree/admin/orders/_form.html.haml +++ b/app/views/spree/admin/orders/_form.html.haml @@ -5,7 +5,7 @@ = render :partial => "spree/admin/orders/shipment", :collection => @order.shipments, :locals => { :order => order } = render :partial => "spree/admin/orders/_form/adjustments", :locals => { :adjustments => @order.price_adjustments, :order => order, :title => Spree.t(:line_item_adjustments)} - = render :partial => "spree/admin/orders/_form/adjustments", :locals => { :adjustments => @order.adjustments, :order => order, :title => Spree.t(:order_adjustments)} + = render :partial => "spree/admin/orders/_form/adjustments", :locals => { :adjustments => @order.adjustments.eligible, :order => order, :title => Spree.t(:order_adjustments)} - if order.line_items.exists? %fieldset#order-total.no-border-bottom{"data-hook" => "order_details_total"} diff --git a/spec/features/admin/orders_spec.rb b/spec/features/admin/orders_spec.rb index 48072ddad8..dc8b654be8 100644 --- a/spec/features/admin/orders_spec.rb +++ b/spec/features/admin/orders_spec.rb @@ -283,6 +283,19 @@ feature %q{ end end + scenario "shows only eligible adjustments" do + adjustment = create( + :adjustment, + adjustable: @order, + label: "invalid adjustment", + amount: 0 + ) + + visit spree.edit_admin_order_path(@order) + + expect(page).to have_no_content adjustment.label + end + scenario "cannot split the order in different stock locations" do # There's only 1 stock location in OFN, so the split functionality that comes with spree should be hidden expect(page).to_not have_selector '.split-item'