SchrodingersGat
commented
3 weeks ago @realstudent01 you will have to provide an example here, but I think that the code is doing what it should be doing...
def get_bom_price_range(self, quantity=1, internal=False, purchase=False):
"""Return the price range of the BOM for this part.
Adds the minimum price for all components in the BOM.
Note: If the BOM contains items without pricing information,
these items cannot be included in the BOM!
"""
min_price = None
max_price = None
for item in self.get_bom_items().select_related('sub_part'):
if item.sub_part.pk == self.pk:
logger.warning('WARNING: BomItem ID %s contains itself in BOM', item.pk)
continue
q = decimal.Decimal(quantity)
i = decimal.Decimal(item.quantity)
prices = item.sub_part.get_price_range(
q * i, internal=internal, purchase=purchase
)
if prices is None:
continue
low, high = prices
if min_price is None:
min_price = 0
if max_price is None:
max_price = 0
min_price += low
max_price += high
if min_price is None or max_price is None:
return None
min_price = normalize(min_price)
max_price = normalize(max_price)
return (min_price, max_price)The BOM pricing uses the quantity of the BOM line item to determine pricing. Is this not what you are asking for?
realstudent01
jesseyu1984918
motusRP
matmair
Please verify that this feature request has NOT been suggested before.
Problem statement
When building assemblies out of assemblies, the price data does not propagate up the levels.
Suggested solution
When an assembly is added as a component to another assembly, multiply the total price of the assembly's BOM by the quantity of that assembly being used in the higher assembly.
Describe alternatives you've considered
Another solution would be to not allow assemblies to be components of other assemblies, but that seems opposed to the purpose of inventree
Examples of other systems
No response
Do you want to develop this?