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: alternate item not getting updated in work order #45553

Open
wants to merge 1 commit into
base: version-15-hotfix
Choose a base branch
from

Conversation

govindsmenokee
Copy link
Contributor

Incorrect field referred in the update qty in snot present in work order item

Please provide enough information so that others can review your pull request:

Explain the details for making this change. What existing problem does the pull request solve?

Screenshots/GIFs

Incorrect field referred in the update qty in snot present in work order item
@@ -568,7 +568,7 @@ erpnext.utils.select_alternate_items = function (opts) {
qty = row.qty;
}
row[item_field] = d.alternate_item;
frappe.model.set_value(row.doctype, row.name, "qty", qty);
frappe.model.set_value(row.doctype, row.name, "required_qty", qty);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The fieldname is "required_qty" only for "Work Order Item", not for "BOM Item" and "Stock Entry Item". Change accordingly.

Also, always send the pull request to develop branch, will backport it to v15-hotfix.

@nabinhait nabinhait self-assigned this Jan 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants