From 4768c9cbb3a4298a9d8e34ca26526b19a8b7cc8f Mon Sep 17 00:00:00 2001 From: eeintech Date: Thu, 19 Nov 2020 15:31:39 -0500 Subject: [PATCH 1/2] Fixed validation of SupplierPart edit form by forcing the value of single_pricing field --- InvenTree/company/views.py | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/InvenTree/company/views.py b/InvenTree/company/views.py index 2f734a7cc5..a98df503b2 100644 --- a/InvenTree/company/views.py +++ b/InvenTree/company/views.py @@ -274,6 +274,12 @@ class SupplierPartEdit(AjaxUpdateView): def get_form(self): form = super().get_form() + supplier_part = self.get_object() + + # It appears that hiding a MoneyField fails validation + # Therefore the idea to set the value before hiding + if form.is_valid(): + form.cleaned_data['single_pricing'] = supplier_part.unit_pricing # Hide the single-pricing field (only for creating a new SupplierPart!) form.fields['single_pricing'].widget = HiddenInput() From 11745ebd6c315ca4295ede55cefa76b08c2fdd3c Mon Sep 17 00:00:00 2001 From: eeintech Date: Thu, 19 Nov 2020 15:32:42 -0500 Subject: [PATCH 2/2] Removed one too many indent --- InvenTree/company/views.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/InvenTree/company/views.py b/InvenTree/company/views.py index a98df503b2..2720f4ccac 100644 --- a/InvenTree/company/views.py +++ b/InvenTree/company/views.py @@ -279,7 +279,7 @@ class SupplierPartEdit(AjaxUpdateView): # It appears that hiding a MoneyField fails validation # Therefore the idea to set the value before hiding if form.is_valid(): - form.cleaned_data['single_pricing'] = supplier_part.unit_pricing + form.cleaned_data['single_pricing'] = supplier_part.unit_pricing # Hide the single-pricing field (only for creating a new SupplierPart!) form.fields['single_pricing'].widget = HiddenInput()