Undo order import changes (#5725)

Author: diebas

app/models/order_import.rb

@@ -14,8 +14,8 @@ class OrderImport < ApplicationRecord
 
   delegate :download_url, to: :error_file, prefix: true, allow_nil: true
 
-  def fetch_or_create_order(row_importer)
-    order_cache[row_importer.order_key] || create_order_from_imported_row(row_importer)
+  def fetch_or_create_order!(row_importer)
+    order_cache[row_importer.order_key] || create_order_from_imported_row!(row_importer)
   end
 
   #
@@ -74,8 +74,8 @@ def to_s
 
   private
 
-  def create_order_from_imported_row(row_importer)
-    Order.create(
+  def create_order_from_imported_row!(row_importer)
+    Order.create!(
       facility:,
       account: row_importer.account,
       user: row_importer.user,

app/services/order_row_importer.rb

@@ -141,7 +141,7 @@ def add_error(message, options = {})
   def add_to_order
     ActiveRecord::Base.transaction do
       begin
-        @order = field(:order_number).present? ? existing_order : @order_import.fetch_or_create_order(self)
+        @order = field(:order_number).present? ? existing_order : @order_import.fetch_or_create_order!(self)
 
         # The order adding feature has some quirky behavior because of the "order form"
         # feature: if you add multiple of a timed service, it creates multiple line items