Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: update item_tax_rate in backend (backport #44884) #45008

Merged

Conversation

mergify[bot]
Copy link
Contributor

@mergify mergify bot commented Dec 31, 2024

Issue: item_tax_rate is only set from the front end.
If the Item tax template is updated from the backend incorrect taxes are calculated.

Steps on how to replicate this issue

  • Create one item with only one item tax template
  • Create a transaction for this item. This should auto-fetch item tax template.
  • Unset item tax template from the item. Save

With unset, it doesn't unset item tax rate, causing incorrect final taxes.

Closes: #41081
Frappe Support Issue: https://support.frappe.io/app/hd-ticket/27836

backport-version-15-hotfix
backport-varsion-14-hotfix


This is an automatic backport of pull request #44884 done by [Mergify](https://mergify.com).

(cherry picked from commit de54c0b)

# Conflicts:
#	erpnext/controllers/taxes_and_totals.py
Copy link
Contributor Author

mergify bot commented Dec 31, 2024

Cherry-pick of de54c0b has failed:

On branch mergify/bp/version-15-hotfix/pr-44884
Your branch is up to date with 'origin/version-15-hotfix'.

You are currently cherry-picking commit de54c0b41f.
  (fix conflicts and run "git cherry-pick --continue")
  (use "git cherry-pick --skip" to skip this patch)
  (use "git cherry-pick --abort" to cancel the cherry-pick operation)

Changes to be committed:
	modified:   erpnext/accounts/doctype/sales_invoice/test_sales_invoice.py

Unmerged paths:
  (use "git add <file>..." to mark resolution)
	both modified:   erpnext/controllers/taxes_and_totals.py

To fix up this pull request, you can check it out locally. See documentation: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/checking-out-pull-requests-locally

@ruthra-kumar ruthra-kumar force-pushed the mergify/bp/version-15-hotfix/pr-44884 branch from 0ee7b20 to e09fb87 Compare December 31, 2024 11:42
@ruthra-kumar ruthra-kumar merged commit 1a1476a into version-15-hotfix Jan 1, 2025
11 checks passed
@ruthra-kumar ruthra-kumar deleted the mergify/bp/version-15-hotfix/pr-44884 branch January 1, 2025 08:18
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jan 16, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants