Allow producer to edit their products on hubs' orders

app/models/enterprise.rb

@@ -375,7 +375,7 @@ def is_hub
     sells == 'any'
   end
 
-  def is_producer
+  def is_producer_only
     is_primary_producer && sells == 'none'
   end
 

app/models/spree/user.rb

@@ -156,9 +156,9 @@ def can_manage_orders?
     # any of order distributors allow them to edit their orders.
     def can_manage_line_items_in_orders?
       @can_manage_line_items_in_orders ||= begin
-        has_any_producer = enterprises.any?(&:is_producer)
-        has_producer_editable_orders = Spree::Order.editable_by_producers(enterprises).exists?
-        has_any_producer && has_producer_editable_orders
+        return unless enterprises.any?(&:is_producer_only)
+
+        Spree::Order.editable_by_producers(enterprises).exists?
       end
     end
 

spec/models/enterprise_spec.rb

@@ -1013,25 +1013,25 @@ def should_have_enterprise_relationship(opts = {})
     end
   end
 
-  describe "#is_producer" do
+  describe "#is_producer_only" do
     context "when enterprise is_primary_producer and sells none" do
       it "returns true" do
         enterprise = build(:supplier_enterprise)
-        expect(enterprise.is_producer).to be true
+        expect(enterprise.is_producer_only).to be true
       end
     end
 
     context "when enterprise is_primary_producer and sells any" do
       it "returns false" do
         enterprise = build(:enterprise, is_primary_producer: true, sells: "any")
-        expect(enterprise.is_producer).to be false
+        expect(enterprise.is_producer_only).to be false
       end
     end
 
     context "when enterprise is_primary_producer and sells own" do
       it "returns false" do
         enterprise = build(:enterprise, is_primary_producer: true, sells: "own")
-        expect(enterprise.is_producer).to be false
+        expect(enterprise.is_producer_only).to be false
       end
     end
   end