rattail/rattail
2
0
Fork 0
Code Issues 7 Pull requests Actions Packages Projects Releases Wiki Activity

Flag invoice case quantity diffs for receiving, when applicable

Browse source
This commit is contained in:
Lance Edgar 2023-12-19 13:08:27 -06:00
parent 1be88bb667
commit 1daf4022fb
1 changed files with 16 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

16
rattail/batch/purchase.py
View file

@ -2111,11 +2111,27 @@ class PurchaseBatchHandler(BatchHandler):
if batch.order_quantities_known and ( if batch.order_quantities_known and (
self.get_units_shipped(row) != self.get_units_accounted_for(row)): self.get_units_shipped(row) != self.get_units_accounted_for(row)):
row.status_code = row.STATUS_ORDERED_RECEIVED_DIFFER row.status_code = row.STATUS_ORDERED_RECEIVED_DIFFER
elif case_quantity_differs: elif case_quantity_differs:
cost = row.product.cost_for_vendor(batch.vendor) cost = row.product.cost_for_vendor(batch.vendor)
row.status_code = row.STATUS_CASE_QUANTITY_DIFFERS row.status_code = row.STATUS_CASE_QUANTITY_DIFFERS
row.status_text = "batch has {} but master cost has {}".format( row.status_text = "batch has {} but master cost has {}".format(
repr(row.case_quantity), repr(cost.case_size)) repr(row.case_quantity), repr(cost.case_size))
# the canonical "case size" for the row will always
# match the master cost record on file, but the
# invoice may have a different value. there will be
# false alarms e.g. for 6-pack soda, so we only flag
# these if *totals* also do not match, *and* if the
# case UOM is being received
elif (row.invoice_case_size is not None
and row.invoice_case_size != row.case_quantity
and row.invoice_total is not None
and row.invoice_total_calculated != row.invoice_total
and (row.cases_ordered or row.cases_shipped)):
row.status_code = row.STATUS_CASE_QUANTITY_DIFFERS
row.status_text = f"master has {row.case_quantity} but invoice has {row.invoice_case_size}"
# TODO: is this right? should out of stock just be a filter for # TODO: is this right? should out of stock just be a filter for
# the user to specify, or should it affect status? # the user to specify, or should it affect status?
elif row.out_of_stock: elif row.out_of_stock: