Mirrored_Repos/InvenTree
1
0
Fork 0
mirror of https://github.com/inventree/InvenTree synced 2024-08-30 18:33:04 +00:00
Code Issues Packages Projects Releases Wiki Activity

Add try/except blocks for calls to conert_money

Browse Source
This commit is contained in:
Oliver 2022-01-12 11:57:33 +11:00
parent 2255619b40
commit 7ce55f4195
2 changed files with 19 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
InvenTree/part/models.py
View File

@ -29,6 +29,8 @@ from markdownx.models import MarkdownxField
from django_cleanup import cleanup from django_cleanup import cleanup
from djmoney.contrib.exchange.exceptions import MissingRate
from mptt.models import TreeForeignKey, MPTTModel from mptt.models import TreeForeignKey, MPTTModel
from mptt.exceptions import InvalidMove from mptt.exceptions import InvalidMove
from mptt.managers import TreeManager from mptt.managers import TreeManager
@ -1832,9 +1834,14 @@ class Part(MPTTModel):
def get_purchase_price(self, quantity): def get_purchase_price(self, quantity):
currency = currency_code_default() currency = currency_code_default()
try:
prices = [convert_money(item.purchase_price, currency).amount for item in self.stock_items.all() if item.purchase_price] prices = [convert_money(item.purchase_price, currency).amount for item in self.stock_items.all() if item.purchase_price]
except MissingRate:
prices = None
if prices: if prices:
return min(prices) * quantity, max(prices) * quantity return min(prices) * quantity, max(prices) * quantity
return None return None
@transaction.atomic @transaction.atomic

9
InvenTree/part/views.py
View File

@ -20,6 +20,7 @@ from django.contrib import messages
from moneyed import CURRENCIES from moneyed import CURRENCIES
from djmoney.contrib.exchange.models import convert_money from djmoney.contrib.exchange.models import convert_money
from djmoney.contrib.exchange.exceptions import MissingRate
from PIL import Image from PIL import Image
@ -425,7 +426,11 @@ class PartDetail(InvenTreeRoleMixin, DetailView):
continue continue
# convert purchase price to current currency - only one currency in the graph # convert purchase price to current currency - only one currency in the graph
try:
price = convert_money(stock_item.purchase_price, default_currency) price = convert_money(stock_item.purchase_price, default_currency)
except MissingRate:
continue
line = { line = {
'price': price.amount, 'price': price.amount,
'qty': stock_item.quantity 'qty': stock_item.quantity
@ -487,7 +492,11 @@ class PartDetail(InvenTreeRoleMixin, DetailView):
if None in [sale_item.purchase_price, sale_item.quantity]: if None in [sale_item.purchase_price, sale_item.quantity]:
continue continue
try:
price = convert_money(sale_item.purchase_price, default_currency) price = convert_money(sale_item.purchase_price, default_currency)
except MissingRate:
continue
line = { line = {
'price': price.amount if price else 0, 'price': price.amount if price else 0,
'qty': sale_item.quantity, 'qty': sale_item.quantity,