dscf-marketplace 0.16.0 → 0.16.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 3cc1c9aaf986ce406e613ac0b3dc31e9a230a8a4e996ef61df353ef51003cdd6
-  data.tar.gz: ca71e2b9a1310eb873818fdeede519b86af08ca7db17764d0a123ab287c23376
+  metadata.gz: be5ac84a6073e85553af278f37aed96cab1d39d3284817361e76060ca789f638
+  data.tar.gz: 2dfbf53884b07ea2e894e46b8dc3c99f8843b997ff6174a9a154ee1299ce6993
 SHA512:
-  metadata.gz: 59c39334fa8624f99730ef5bf36c9bb18f17e3ecb2c653af7be70cea8068c45fe0dad28b3a7cd632398ef14eed830f1f57111d36bae5ce9d1c63bc6961685ad2
-  data.tar.gz: 688c0ab34c504eaf3e393ef5e82a4ec0c2047d8ed0114126776fd6af88a27156fadf87b532bbb91458b6a63d8d43d2ec568477f15f2c8eb6cae9fb4bf58e2437
+  metadata.gz: 3e6c001c6f0af07cda0207acda610c20cb6eabd41c2a3fd24f8c4ea026c46e1b0dc8b6c7a84b5d3fcf22ec4b0bf974c434cdfe668a50049d4320260005d2b34c
+  data.tar.gz: 5115dad31f5607464892aea0b3dbee5d6761b84f21c81e26cc73febe091a13a1c7358911f89b21c37c618f78cbbd5f32a437ea131627502ae0872f5be486a538

data/app/controllers/dscf/marketplace/orders_controller.rb

@@ -44,11 +44,17 @@ module Dscf
         resolved_items, skipped, adjusted = resolve_reorder_items(source_order)
 
         if resolved_items.empty?
-          return render_error(
-            errors: "None of the items on order ##{source_order.id} are available to reorder",
-            status: :unprocessable_entity,
-            skipped_items: skipped
-          )
+          # Distinguish "nothing left to reorder" from "everything is
+          # unavailable" — an order whose lines were all cancelled has no
+          # availability problem, and reporting one sends the caller hunting
+          # through listings for a cause that isn't there.
+          message = if skipped.empty?
+            "Order ##{source_order.id} has no reorderable items (every line was cancelled or the order has no items)"
+          else
+            "None of the items on order ##{source_order.id} are available to reorder"
+          end
+
+          return render_error(errors: message, status: :unprocessable_entity, skipped_items: skipped)
         end
 
         parent, orders = place_direct_listing_cart(resolved_items, base_attrs: reorder_base_attributes(source_order))
@@ -70,6 +76,10 @@ module Dscf
         end
 
         render_success(data: new_order, serializer_options: options, status: :created, meta: meta)
+      rescue ActiveRecord::RecordNotFound
+        # An unknown order id is a 404, not a business-rule failure — the broad
+        # rescue below would otherwise report it as 422.
+        raise
       rescue ActiveRecord::RecordInvalid => e
         errors = e.record&.errors&.full_messages&.presence || [ e.message ]
         render_error(errors: errors.join(", "), status: :unprocessable_entity)
@@ -644,8 +654,15 @@ module Dscf
 
         reorder_source_lines(source_order).each do |item, seller|
           product_name = item.product&.name
-          wanted = item.quantity.to_i
-          next if wanted <= 0
+          # Quantities are decimal(15,6) — goods sold by weight have fractional
+          # lines, and an aggregator resolving a line can set any decimal. to_i
+          # would silently truncate 0.5 to 0 and drop the line.
+          wanted = item.quantity.to_d
+
+          if wanted <= 0
+            skipped << {product_id: item.product_id, product_name: product_name, reason: "invalid_quantity"}
+            next
+          end
 
           listing, aggregator_listing = find_reorder_listing(item, seller)
 
@@ -654,7 +671,7 @@ module Dscf
             next
           end
 
-          available = (listing || aggregator_listing).quantity.to_i
+          available = (listing || aggregator_listing).quantity.to_d
           if available <= 0
             skipped << {product_id: item.product_id, product_name: product_name, reason: "out_of_stock"}
             next
@@ -680,12 +697,18 @@ module Dscf
 
       # The lines worth reordering, each paired with the business it was bought
       # from. A parent holds no lines of its own, so a reorder of a whole cart
-      # collects them from its sub-orders (and may split again). Cancelled lines
-      # were never fulfilled, so they don't come along.
+      # collects them from its sub-orders (and may split again).
+      #
+      # Whether a cancelled line comes along depends on what was cancelled. A
+      # cancelled ORDER is precisely what a retailer wants to re-place — every
+      # line reads as cancelled, and dropping them all would leave nothing to
+      # reorder. On a live order a cancelled LINE was deliberately removed
+      # during validation or refused by its supplier, and that decision stands.
       def reorder_source_lines(source_order)
         orders = source_order.parent? ? source_order.sub_orders.to_a : [ source_order ]
         orders.flat_map do |order|
-          order.order_items.reject(&:cancelled?).map { |item| [ item, order.ordered_to ] }
+          items = order.cancelled? ? order.order_items.to_a : order.order_items.reject(&:cancelled?)
+          items.map { |item| [ item, order.ordered_to ] }
         end
       end
 

data/lib/dscf/marketplace/version.rb

@@ -1,5 +1,5 @@
 module Dscf
   module Marketplace
-    VERSION = "0.16.0".freeze
+    VERSION = "0.16.2".freeze
   end
 end

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification
 name: dscf-marketplace
 version: !ruby/object:Gem::Version
-  version: 0.16.0
+  version: 0.16.2
 platform: ruby
 authors:
 - Asrat
 bindir: bin
 cert_chain: []
-date: 2026-07-29 00:00:00.000000000 Z
+date: 2026-07-30 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails