Skip to content

Commit

Permalink
fix: Item rate reset on changing posting date (#30990)
Browse files Browse the repository at this point in the history
* fix: Item rate reset on changing posting date

* chore: Remove debugger
  • Loading branch information
deepeshgarg007 authored May 13, 2022
1 parent 9a2b11f commit 54d6cf1
Showing 1 changed file with 8 additions and 6 deletions.
14 changes: 8 additions & 6 deletions erpnext/public/js/controllers/transaction.js
Original file line number Diff line number Diff line change
Expand Up @@ -923,12 +923,12 @@ erpnext.TransactionController = class TransactionController extends erpnext.taxe
}

currency() {
/* manqala 19/09/2016: let the translation date be whichever of the transaction_date or posting_date is available */
var transaction_date = this.frm.doc.transaction_date || this.frm.doc.posting_date;
/* end manqala */
var me = this;
// The transaction date be either transaction_date (from orders) or posting_date (from invoices)
let transaction_date = this.frm.doc.transaction_date || this.frm.doc.posting_date;

let me = this;
this.set_dynamic_labels();
var company_currency = this.get_company_currency();
let company_currency = this.get_company_currency();
// Added `ignore_price_list` to determine if document is loading after mapping from another doc
if(this.frm.doc.currency && this.frm.doc.currency !== company_currency
&& !(this.frm.doc.__onload && this.frm.doc.__onload.ignore_price_list)) {
Expand All @@ -942,7 +942,9 @@ erpnext.TransactionController = class TransactionController extends erpnext.taxe
}
});
} else {
this.conversion_rate();
// company currency and doc currency is same
// this will prevent unnecessary conversion rate triggers
this.frm.set_value("conversion_rate", 1.0);
}
}

Expand Down

0 comments on commit 54d6cf1

Please sign in to comment.