effective_qb_online 0.9.9 → 0.9.10

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e9c2123d98959752c1fc3282eaed56393c472270627852362f18f481df53effb
-  data.tar.gz: 5337ce9494f3677d72d3d8b5fb95ed43612d3a1580631cc6954d0f0dbac20f83
+  metadata.gz: e6a3e221e9f86bab6fec4bbdf698d486a12924850960c797e897a79a9dde137e
+  data.tar.gz: eceb1df9ab98fd686e43e23572ef6be5ace97720bddaf35df4878dee8c5fafbe
 SHA512:
-  metadata.gz: ba8008ddd33147f3cf9f793a9f8a2477ed231ba91588cbaa47246ce571b91107eff55298d4a1eef5fb28144279ee011d35c7b5fe5b0ca1680cb1eaa6a3714f26
-  data.tar.gz: 3ab35742a9bd30d10248a8a8efeaf4511c42bc9a1390283f6a5387a69ec76e95e3faeb39673e97b31065d336a5c6e8d5229597dcbfc529d13afe610661aecbc7
+  metadata.gz: 481a68ddeda1c7dd2fc07f6c5a44758df77f7bed84a84313f8f33850c3b061a59444c5f3f9654e35f114677752e12b750909f0fb45e5298a8d6a7282e59a19ab
+  data.tar.gz: 6840d8757d7d237283957334a868f447348395972f42e09c036bd6b6f16ac6e7baf5b84dea43419d9653e2e9968527c8dfba2a10424ee4d41f87978c1800fa2c

data/app/models/effective/qb_receipt.rb

@@ -66,6 +66,12 @@ module Effective
         sales_receipt = Effective::QbSalesReceipt.build_from_receipt!(self, api: api)
         sales_receipt = api.create_sales_receipt(sales_receipt: sales_receipt)
 
+        # The SalesReceipt now exists in QuickBooks. Persist its id immediately so a
+        # later failure (e.g. the total sanity checks below) doesn't orphan it. Without
+        # this, a re-sync would try to create it again and QuickBooks rejects the
+        # reused DocNumber with a Duplicate Document Number Error.
+        update_column(:sales_receipt_id, sales_receipt.id)
+
         # Sanity check
         if EffectiveOrders.try(:credit_card_surcharge_qb_item_name).present? && !EffectiveOrders.try(:fee_saver?)
           if (expected = api.price_to_amount(order.total)) != sales_receipt.total

data/app/models/effective/qb_sales_receipt.rb

@@ -38,8 +38,11 @@ module Effective
       end
 
       receipt.qb_receipt_items.each do |receipt_item|
+        # The purchasable may have been deleted. When it's missing we can't auto-match
+        # the QuickBooks item from its qb_item_id/qb_item_name, so we fall back to the
+        # QbReceiptItem's item_id (mapped manually in the admin). The SalesReceipt line
+        # items are built from the OrderItem, so a missing purchasable is not fatal.
         purchasable = receipt_item.order_item.purchasable
-        raise("Expected a purchasable for Effective::OrderItem #{receipt_item.order_item.id}") unless purchasable.present?
 
         # Either of these could match
         purchasable_id_name = [
@@ -59,7 +62,7 @@ module Effective
 
         if item.blank?
           purchasable_id_name = [purchasable.try(:qb_item_id), purchasable.try(:qb_item_name)].compact
-          raise("Unknown Item #{purchasable_id_name.join(' or ')} from #{purchasable} (#{purchasable.class.name} ##{purchasable.id})")
+          raise("Unknown Item #{purchasable_id_name.join(' or ')} from #{purchasable} (#{purchasable.class.name} ##{purchasable&.id})")
         end
 
         receipt_item.update!(item_id: item.id)

data/lib/effective_qb_online/version.rb

@@ -1,3 +1,3 @@
 module EffectiveQbOnline
-  VERSION = '0.9.9'.freeze
+  VERSION = '0.9.10'.freeze
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: effective_qb_online
 version: !ruby/object:Gem::Version
-  version: 0.9.9
+  version: 0.9.10
 platform: ruby
 authors:
 - Code and Effect
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2026-06-18 00:00:00.000000000 Z
+date: 2026-07-20 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails