chore: resolve conflicts

This commit is contained in:
ljain112
2026-06-09 18:12:53 +05:30
parent eef075a2ba
commit 5629056ec2
3 changed files with 7 additions and 24 deletions

View File

@@ -26,14 +26,9 @@ from erpnext.utilities.regional import temporary_flag
class calculate_taxes_and_totals:
def __init__(self, doc: Document):
self.doc = doc
<<<<<<< HEAD
frappe.flags.round_off_applicable_accounts = []
frappe.flags.round_row_wise_tax = frappe.db.get_single_value(
"Accounts Settings", "round_row_wise_tax"
=======
frappe.flags.round_off_applicable_accounts = (
get_round_off_applicable_accounts(self.doc.company, [], self.doc) or []
>>>>>>> 08129ff71c (fix: update round off account functions to accept document context for regional overrides (#55758))
)
if doc.get("round_off_applicable_accounts_for_tax_withholding"):
@@ -43,7 +38,9 @@ class calculate_taxes_and_totals:
self._items = self.filter_rows() if self.doc.doctype == "Quotation" else self.doc.get("items")
get_round_off_applicable_accounts(self.doc.company, frappe.flags.round_off_applicable_accounts)
get_round_off_applicable_accounts(
self.doc.company, frappe.flags.round_off_applicable_accounts, self.doc
)
self.calculate()
def filter_rows(self):
@@ -1133,13 +1130,9 @@ def get_itemised_tax_breakup_html(doc):
@frappe.whitelist()
<<<<<<< HEAD
def get_round_off_applicable_accounts(company, account_list):
=======
def get_round_off_applicable_accounts(
company: str, account_list: list | str, doc: str | dict | Document | None = None
):
>>>>>>> 08129ff71c (fix: update round off account functions to accept document context for regional overrides (#55758))
# required to set correct region
with temporary_flag("company", company):
return get_regional_round_off_accounts(company, account_list, doc)

View File

@@ -1,3 +1,5 @@
from unittest.mock import patch
import frappe
from frappe.tests.utils import FrappeTestCase
@@ -5,10 +7,7 @@ from erpnext.controllers.taxes_and_totals import calculate_taxes_and_totals
from erpnext.selling.doctype.sales_order.test_sales_order import make_sales_order
<<<<<<< HEAD
class TestTaxesAndTotals(FrappeTestCase):
=======
class TestTaxesAndTotals(ERPNextTestSuite):
def test_regional_round_off_accounts(self):
"""
Regional overrides cannot extend the list in-place — the return
@@ -31,7 +30,6 @@ class TestTaxesAndTotals(ERPNextTestSuite):
self.assertIn(test_account, frappe.flags.round_off_applicable_accounts)
>>>>>>> 08129ff71c (fix: update round off account functions to accept document context for regional overrides (#55758))
def test_disabling_rounded_total_resets_base_fields(self):
"""Disabling rounded total should also clear base rounded values."""
so = make_sales_order(do_not_save=True)

View File

@@ -204,19 +204,11 @@ erpnext.taxes_and_totals = class TaxesAndTotals extends erpnext.payments {
if (me.frm.doc.company) {
frappe.call({
<<<<<<< HEAD
"method": "erpnext.controllers.taxes_and_totals.get_round_off_applicable_accounts",
"args": {
"company": me.frm.doc.company,
"account_list": frappe.flags.round_off_applicable_accounts
=======
method: "erpnext.controllers.taxes_and_totals.get_round_off_applicable_accounts",
args: {
company: me.frm.doc.company,
account_list: frappe.flags.round_off_applicable_accounts,
// pass the doc so regional overrides can inspect it
doc: me.frm.doc,
>>>>>>> 08129ff71c (fix: update round off account functions to accept document context for regional overrides (#55758))
"account_list": frappe.flags.round_off_applicable_accounts,
"doc": me.frm.doc,
},
callback(r) {
if (r.message) {