Rounding in totals calculation
This commit is contained in:
parent
4215b3afc3
commit
ccd9fd3e94
@ -210,14 +210,15 @@ erpnext.buying.BuyingController = erpnext.TransactionController.extend({
|
|||||||
calculate_totals: function() {
|
calculate_totals: function() {
|
||||||
var tax_count = this.frm.tax_doclist.length;
|
var tax_count = this.frm.tax_doclist.length;
|
||||||
this.frm.doc.grand_total = flt(tax_count ?
|
this.frm.doc.grand_total = flt(tax_count ?
|
||||||
this.frm.tax_doclist[tax_count - 1].total : this.frm.doc.net_total,
|
this.frm.tax_doclist[tax_count - 1].total : this.frm.doc.net_total);
|
||||||
precision("grand_total"));
|
this.frm.doc.grand_total_import = flt(this.frm.doc.grand_total / this.frm.doc.conversion_rate);
|
||||||
this.frm.doc.grand_total_import = flt(this.frm.doc.grand_total /
|
|
||||||
this.frm.doc.conversion_rate, precision("grand_total_import"));
|
|
||||||
|
|
||||||
this.frm.doc.total_tax = flt(this.frm.doc.grand_total - this.frm.doc.net_total,
|
this.frm.doc.total_tax = flt(this.frm.doc.grand_total - this.frm.doc.net_total,
|
||||||
precision("total_tax"));
|
precision("total_tax"));
|
||||||
|
|
||||||
|
this.frm.doc.grand_total = flt(this.frm.doc.grand_total, precision("grand_total"));
|
||||||
|
this.frm.doc.grand_total_import = flt(this.frm.doc.grand_total_import, precision("grand_total_import"));
|
||||||
|
|
||||||
// rounded totals
|
// rounded totals
|
||||||
if(frappe.meta.get_docfield(this.frm.doc.doctype, "rounded_total", this.frm.doc.name)) {
|
if(frappe.meta.get_docfield(this.frm.doc.doctype, "rounded_total", this.frm.doc.name)) {
|
||||||
this.frm.doc.rounded_total = Math.round(this.frm.doc.grand_total);
|
this.frm.doc.rounded_total = Math.round(this.frm.doc.grand_total);
|
||||||
|
|||||||
@ -110,13 +110,13 @@ class BuyingController(StockController):
|
|||||||
self.round_floats_in(self, ["net_total", "net_total_import"])
|
self.round_floats_in(self, ["net_total", "net_total_import"])
|
||||||
|
|
||||||
def calculate_totals(self):
|
def calculate_totals(self):
|
||||||
self.grand_total = flt(self.tax_doclist[-1].total if self.tax_doclist
|
self.grand_total = flt(self.tax_doclist[-1].total if self.tax_doclist else self.net_total)
|
||||||
else self.net_total, self.precision("grand_total"))
|
self.grand_total_import = flt(self.grand_total / self.conversion_rate)
|
||||||
self.grand_total_import = flt(self.grand_total / self.conversion_rate,
|
|
||||||
self.precision("grand_total_import"))
|
|
||||||
|
|
||||||
self.total_tax = flt(self.grand_total - self.net_total,
|
self.total_tax = flt(self.grand_total - self.net_total, self.precision("total_tax"))
|
||||||
self.precision("total_tax"))
|
|
||||||
|
self.grand_total = flt(self.grand_total, self.precision("grand_total"))
|
||||||
|
self.grand_total_import = flt(self.grand_total_import, self.precision("grand_total_import"))
|
||||||
|
|
||||||
if self.meta.get_field("rounded_total"):
|
if self.meta.get_field("rounded_total"):
|
||||||
self.rounded_total = rounded(self.grand_total)
|
self.rounded_total = rounded(self.grand_total)
|
||||||
|
|||||||
@ -216,17 +216,17 @@ class SellingController(StockController):
|
|||||||
self.round_floats_in(self, ["net_total", "net_total_export"])
|
self.round_floats_in(self, ["net_total", "net_total_export"])
|
||||||
|
|
||||||
def calculate_totals(self):
|
def calculate_totals(self):
|
||||||
self.grand_total = flt(self.tax_doclist and \
|
self.grand_total = flt(self.tax_doclist[-1].total if self.tax_doclist else self.net_total)
|
||||||
self.tax_doclist[-1].total or self.net_total, self.precision("grand_total"))
|
|
||||||
self.grand_total_export = flt(self.grand_total / self.conversion_rate,
|
|
||||||
self.precision("grand_total_export"))
|
|
||||||
|
|
||||||
self.other_charges_total = flt(self.grand_total - self.net_total,
|
self.grand_total_export = flt(self.grand_total / self.conversion_rate)
|
||||||
self.precision("other_charges_total"))
|
|
||||||
|
|
||||||
self.other_charges_total_export = flt(self.grand_total_export -
|
self.other_charges_total = flt(self.grand_total - self.net_total, self.precision("other_charges_total"))
|
||||||
self.net_total_export + flt(self.discount_amount),
|
|
||||||
self.precision("other_charges_total_export"))
|
self.other_charges_total_export = flt(self.grand_total_export - self.net_total_export +
|
||||||
|
flt(self.discount_amount), self.precision("other_charges_total_export"))
|
||||||
|
|
||||||
|
self.grand_total = flt(self.grand_total, self.precision("grand_total"))
|
||||||
|
self.grand_total_export = flt(self.grand_total_export, self.precision("grand_total_export"))
|
||||||
|
|
||||||
self.rounded_total = rounded(self.grand_total)
|
self.rounded_total = rounded(self.grand_total)
|
||||||
self.rounded_total_export = rounded(self.grand_total_export)
|
self.rounded_total_export = rounded(self.grand_total_export)
|
||||||
|
|||||||
@ -341,11 +341,8 @@ erpnext.selling.SellingController = erpnext.TransactionController.extend({
|
|||||||
var me = this;
|
var me = this;
|
||||||
var tax_count = this.frm.tax_doclist.length;
|
var tax_count = this.frm.tax_doclist.length;
|
||||||
|
|
||||||
this.frm.doc.grand_total = flt(
|
this.frm.doc.grand_total = flt(tax_count ? this.frm.tax_doclist[tax_count - 1].total : this.frm.doc.net_total);
|
||||||
tax_count ? this.frm.tax_doclist[tax_count - 1].total : this.frm.doc.net_total,
|
this.frm.doc.grand_total_export = flt(this.frm.doc.grand_total / this.frm.doc.conversion_rate);
|
||||||
precision("grand_total"));
|
|
||||||
this.frm.doc.grand_total_export = flt(this.frm.doc.grand_total / this.frm.doc.conversion_rate,
|
|
||||||
precision("grand_total_export"));
|
|
||||||
|
|
||||||
this.frm.doc.other_charges_total = flt(this.frm.doc.grand_total - this.frm.doc.net_total,
|
this.frm.doc.other_charges_total = flt(this.frm.doc.grand_total - this.frm.doc.net_total,
|
||||||
precision("other_charges_total"));
|
precision("other_charges_total"));
|
||||||
@ -353,6 +350,9 @@ erpnext.selling.SellingController = erpnext.TransactionController.extend({
|
|||||||
this.frm.doc.net_total_export + flt(this.frm.doc.discount_amount),
|
this.frm.doc.net_total_export + flt(this.frm.doc.discount_amount),
|
||||||
precision("other_charges_total_export"));
|
precision("other_charges_total_export"));
|
||||||
|
|
||||||
|
this.frm.doc.grand_total = flt(this.frm.doc.grand_total, precision("grand_total"));
|
||||||
|
this.frm.doc.grand_total_export = flt(this.frm.doc.grand_total_export, precision("grand_total_export"));
|
||||||
|
|
||||||
this.frm.doc.rounded_total = Math.round(this.frm.doc.grand_total);
|
this.frm.doc.rounded_total = Math.round(this.frm.doc.grand_total);
|
||||||
this.frm.doc.rounded_total_export = Math.round(this.frm.doc.grand_total_export);
|
this.frm.doc.rounded_total_export = Math.round(this.frm.doc.grand_total_export);
|
||||||
},
|
},
|
||||||
|
|||||||
Loading…
x
Reference in New Issue
Block a user