Shiloh/brotherton-erpnext
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

fix bug: cast to flt

Browse Source
This commit is contained in:
tunde 2017-09-25 23:24:32 +01:00
parent 541d3f9567
commit 31ddbe36c0
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
erpnext/accounts/doctype/payment_terms_template/payment_terms_template.py
View File

@ -6,7 +6,7 @@ from __future__ import unicode_literals
import frappe import frappe
from frappe.model.document import Document from frappe.model.document import Document
from frappe.utils import flt from frappe.utils import flt, cint
from frappe import _ from frappe import _
@ -19,14 +19,14 @@ class PaymentTermsTemplate(Document):
def validate_invoice_portion(self): def validate_invoice_portion(self):
total_portion = 0 total_portion = 0
for term in self.terms: for term in self.terms:
total_portion += term.invoice_portion total_portion += flt(term.get('invoice_portion', 0))
if flt(total_portion, 2) != 100.00: if flt(total_portion, 2) != 100.00:
frappe.msgprint(_('Combined invoice portion must equal 100%'), raise_exception=1, indicator='red') frappe.msgprint(_('Combined invoice portion must equal 100%'), raise_exception=1, indicator='red')
def validate_credit_days(self): def validate_credit_days(self):
for term in self.terms: for term in self.terms:
if term.credit_days < 0: if cint(term.credit_days) < 0:
frappe.msgprint(_('Credit Days cannot be a negative number'), raise_exception=1, indicator='red') frappe.msgprint(_('Credit Days cannot be a negative number'), raise_exception=1, indicator='red')
def check_duplicate_terms(self): def check_duplicate_terms(self):