Revert "Fix 9 Rubocop violations (Rails/Presence and Rails/RedirectBackOrTo)Fix rubocop violations"

.rubocop_todo.yml

@@ -1,6 +1,6 @@
 # This configuration was generated by
 # `rubocop --auto-gen-config --auto-gen-only-exclude --exclude-limit 1400 --no-auto-gen-timestamp`
-# using RuboCop version 1.84.2.
+# using RuboCop version 1.81.7.
 # The point is for the user to remove these configuration records
 # one by one as the offenses are removed from the code base.
 # Note that changes in the inspected code, or installation of new
@@ -35,7 +35,6 @@ Lint/UselessConstantScoping:
     - 'lib/reporting/report_metadata_builder.rb'
 
 # Offense count: 1
-# This cop supports unsafe autocorrection (--autocorrect-all).
 Lint/UselessOr:
   Exclude:
     - 'app/models/product_import/entry_validator.rb'
@@ -63,13 +62,14 @@ Metrics/AbcSize:
     - 'lib/spree/core/controller_helpers/order.rb'
     - 'spec/services/orders/checkout_restart_service_spec.rb'
 
-# Offense count: 7
+# Offense count: 9
 # Configuration parameters: CountComments, Max, CountAsOne, AllowedMethods, AllowedPatterns, inherit_mode.
 # AllowedMethods: refine
 Metrics/BlockLength:
   Exclude:
     - 'app/models/spree/order/checkout.rb'
     - 'app/models/spree/payment.rb'
+    - 'app/models/spree/payment/processing.rb'
     - 'app/models/spree/shipment.rb'
     - 'lib/spree/core/controller_helpers/common.rb'
     - 'lib/tasks/data.rake'
@@ -168,7 +168,7 @@ Metrics/CyclomaticComplexity:
     - 'lib/spree/localized_number.rb'
     - 'spec/models/product_importer_spec.rb'
 
-# Offense count: 20
+# Offense count: 22
 # Configuration parameters: CountComments, Max, CountAsOne, AllowedMethods, AllowedPatterns.
 Metrics/MethodLength:
   Exclude:
@@ -178,6 +178,7 @@ Metrics/MethodLength:
     - 'app/models/spree/ability.rb'
     - 'app/models/spree/gateway/pay_pal_express.rb'
     - 'app/models/spree/order/checkout.rb'
+    - 'app/models/spree/payment/processing.rb'
     - 'app/models/spree/preferences/preferable_class_methods.rb'
     - 'lib/open_food_network/order_cycle_form_applicator.rb'
     - 'lib/open_food_network/order_cycle_permissions.rb'
@@ -237,7 +238,7 @@ Naming/MethodParameterName:
 # Offense count: 60
 # Configuration parameters: Mode, AllowedMethods, AllowedPatterns, AllowBangMethods, WaywardPredicates.
 # AllowedMethods: call
-# WaywardPredicates: infinite?, nonzero?
+# WaywardPredicates: nonzero?
 Naming/PredicateMethod:
   Exclude:
     - 'app/controllers/admin/product_import_controller.rb'
@@ -335,6 +336,24 @@ Rails/OrderArguments:
     - 'spec/services/orders/generate_invoice_service_spec.rb'
     - 'spec/system/admin/order_cycles/simple_spec.rb'
 
+# Offense count: 3
+# This cop supports safe autocorrection (--autocorrect).
+Rails/Presence:
+  Exclude:
+    - 'app/controllers/admin/enterprises_controller.rb'
+    - 'app/models/spree/product.rb'
+
+# Offense count: 6
+# This cop supports safe autocorrection (--autocorrect).
+# Configuration parameters: Severity.
+Rails/RedirectBackOrTo:
+  Exclude:
+    - 'app/controllers/admin/order_cycles_controller.rb'
+    - 'app/controllers/locales_controller.rb'
+    - 'app/controllers/spree/admin/invoices_controller.rb'
+    - 'app/controllers/spree/admin/orders_controller.rb'
+    - 'app/controllers/spree/admin/return_authorizations_controller.rb'
+
 # Offense count: 1
 # Configuration parameters: TransactionMethods.
 Rails/TransactionExitStatement:

app/controllers/admin/enterprises_controller.rb

@@ -177,7 +177,7 @@ module Admin
     protected
 
     def delete_custom_tab
-      @object.custom_tab.presence&.destroy
+      @object.custom_tab.destroy if @object.custom_tab.present?
       enterprise_params.delete(:custom_tab_attributes)
     end
 
@@ -240,7 +240,9 @@ module Admin
         enterprises = OpenFoodNetwork::OrderCyclePermissions.new(spree_current_user, @order_cycle)
           .visible_enterprises
 
-        enterprises.presence&.includes(supplied_products: [:variants, :image])
+        if enterprises.present?
+          enterprises.includes(supplied_products: [:variants, :image])
+        end
       when :index
         if spree_current_user.admin?
           OpenFoodNetwork::Permissions.new(spree_current_user).

app/controllers/admin/order_cycles_controller.rb

@@ -209,7 +209,7 @@ module Admin
       elsif params[:context] == "checkout_options" && params[:save_and_back_to_list]
         redirect_to main_app.admin_order_cycles_path
       else
-        redirect_back_or_to(root_path)
+        redirect_back(fallback_location: root_path)
       end
     end
 

app/controllers/locales_controller.rb

@@ -3,6 +3,6 @@
 class LocalesController < BaseController
   def show
     UserLocaleSetter.new(spree_current_user, params[:id], cookies).set_locale
-    redirect_back_or_to(main_app.root_url)
+    redirect_back fallback_location: main_app.root_url
   end
 end

app/controllers/spree/admin/invoices_controller.rb

@@ -28,7 +28,7 @@ module Spree
           flash[:error] = t(:must_have_valid_business_number,
                             enterprise_name: @order.distributor.name)
         end
-        redirect_back_or_to(spree.admin_dashboard_path)
+        redirect_back(fallback_location: spree.admin_dashboard_path)
       end
 
       private

app/controllers/spree/admin/orders_controller.rb

@@ -80,7 +80,7 @@ module Spree
       rescue Spree::Core::GatewayError => e
         flash[:error] = e.message.to_s
       ensure
-        redirect_back_or_to(spree.admin_dashboard_path)
+        redirect_back fallback_location: spree.admin_dashboard_path
       end
 
       def resend
@@ -88,7 +88,7 @@ module Spree
         flash[:success] = t('admin.orders.order_email_resent')
 
         respond_with(@order) do |format|
-          format.html { redirect_back_or_to(spree.admin_dashboard_path) }
+          format.html { redirect_back(fallback_location: spree.admin_dashboard_path) }
         end
       end
 

app/controllers/spree/admin/return_authorizations_controller.rb

@@ -10,7 +10,7 @@ module Spree
       def fire
         @return_authorization.public_send("#{params[:e]}!")
         flash[:success] = Spree.t(:return_authorization_updated)
-        redirect_back_or_to(spree.admin_dashboard_path)
+        redirect_back fallback_location: spree.admin_dashboard_path
       end
 
       protected

app/models/spree/product.rb

@@ -295,7 +295,7 @@ module Spree
       # The variant is invalid if no supplier is present, but this method can be triggered when
       # importing product. In this scenario the variant has not been updated with the supplier yet
       # hence the check.
-      first_variant.supplier.presence&.touch
+      first_variant.supplier.touch if first_variant.supplier.present?
     end
 
     def validate_image