Extract out a OrderCycle#simple? method so it can be used in models

app/helpers/order_cycles_helper.rb

@@ -56,10 +56,6 @@ module OrderCyclesHelper
     @simple_index ||= !OpenFoodNetwork::Permissions.new(spree_current_user).can_manage_complex_order_cycles?
   end
 
-  def order_cycles_simple_form
-    @order_cycles_simple_form ||= @order_cycle.coordinator.sells == 'own'
-  end
-
   def pickup_time(order_cycle = current_order_cycle)
     order_cycle.exchanges.to_enterprises(current_distributor).outgoing.first.pickup_time
   end

app/models/order_cycle.rb

@@ -274,6 +274,10 @@ class OrderCycle < ApplicationRecord
     items.each { |li| scoper.scope(li.variant) }
   end
 
+  def simple?
+    coordinator.sells == 'own'
+  end
+
   private
 
   def opening?

app/views/admin/order_cycles/edit.html.haml

@@ -13,20 +13,20 @@
 - content_for :page_title do
   = t :edit_order_cycle
 
-- ng_controller = order_cycles_simple_form ? 'AdminSimpleEditOrderCycleCtrl' : 'AdminEditOrderCycleCtrl'
+- ng_controller = @order_cycle.simple? ? 'AdminSimpleEditOrderCycleCtrl' : 'AdminEditOrderCycleCtrl'
 = admin_inject_order_cycle_instance
 = form_for [main_app, :admin, @order_cycle], :url => '', :html => {:class => 'ng order_cycle', 'ng-app' => 'admin.orderCycles', 'ng-controller' => ng_controller, name: 'order_cycle_form'} do |f|
 
   %save-bar{ dirty: "order_cycle_form.$dirty", persist: "true" }
     %input.red{ type: "button", value: t('.save'), ng: { click: "submit($event, null)", disabled: "!order_cycle_form.$dirty || order_cycle_form.$invalid" } }
-    - if order_cycles_simple_form
+    - if @order_cycle.simple?
       %input.red{ type: "button", value: t('.save_and_back_to_list'), ng: { click: "submit($event, '#{main_app.admin_order_cycles_path}')", disabled: "!order_cycle_form.$dirty || order_cycle_form.$invalid" } }
     - else
       %input.red{ type: "button", value: t('.save_and_next'), ng: { click: "submit($event, '#{main_app.admin_order_cycle_incoming_path(@order_cycle)}')", disabled: "!order_cycle_form.$dirty || order_cycle_form.$invalid" } }
       %input{ type: "button", value: t('.next'), ng: { click: "cancel('#{main_app.admin_order_cycle_incoming_path(@order_cycle)}')", disabled: "order_cycle_form.$dirty" } }
     %input{ type: "button", ng: { value: "order_cycle_form.$dirty ? '#{t('.cancel')}' : '#{t('.back_to_list')}'", click: "cancel('#{main_app.admin_order_cycles_path}')" } }
 
-  - if order_cycles_simple_form
+  - if @order_cycle.simple?
     = render 'simple_form', f: f
   - else
     = render 'form', f: f

app/views/admin/order_cycles/new.html.haml

@@ -1,18 +1,18 @@
 - content_for :page_title do
   =t('new_order_cycle')
 
-- ng_controller = order_cycles_simple_form ? 'AdminSimpleCreateOrderCycleCtrl' : 'AdminCreateOrderCycleCtrl'
+- ng_controller = @order_cycle.simple? ? 'AdminSimpleCreateOrderCycleCtrl' : 'AdminCreateOrderCycleCtrl'
 = admin_inject_order_cycle_instance
 
 = form_for [main_app, :admin, @order_cycle], :url => '', :html => {:class => 'ng order_cycle', 'ng-app' => 'admin.orderCycles', 'ng-controller' => ng_controller, name: 'order_cycle_form'} do |f|
 
   %save-bar{ dirty: "order_cycle_form.$dirty", persist: "true" }
-    - if order_cycles_simple_form
+    - if @order_cycle.simple?
       - custom_redirect_path = main_app.admin_order_cycles_path
     %input.red{ type: "button", value: t('.create'), ng: { click: "submit($event, '#{custom_redirect_path}')", disabled: "!order_cycle_form.$dirty || order_cycle_form.$invalid" } }
     %input{ type: "button", ng: { value: "order_cycle_form.$dirty ? '#{t('.cancel')}' : '#{t('.back_to_list')}'", click: "cancel('#{main_app.admin_order_cycles_path}')" } }
 
-  - if order_cycles_simple_form
+  - if @order_cycle.simple?
     = render 'simple_form', f: f
   - else
     = render 'form', f: f

spec/models/order_cycle_spec.rb

@@ -610,6 +610,20 @@ describe OrderCycle do
     end
   end
 
+  describe "#simple?" do
+    it "returns true if the coordinator sells their own products i.e. shops" do
+      order_cycle = build(:simple_order_cycle, coordinator: build(:enterprise, sells: "own"))
+
+      expect(order_cycle).to be_simple
+    end
+
+    it "returns false if the coordinator can sell other people's products i.e. hubs" do
+      order_cycle = build(:simple_order_cycle, coordinator: build(:enterprise, sells: "any"))
+
+      expect(order_cycle).not_to be_simple
+    end
+  end
+
   def core_exchange_attributes(exchange)
     exterior_attribute_keys = %w(id order_cycle_id created_at updated_at)
     exchange.attributes.