When material request is to be made for purchase qty should be converted
to purchase UOM
(cherry picked from commit 48f3d53556)
Co-authored-by: Ankush Menat <ankush@frappe.io>
* fix: reservation for production incorrect
The query uses case to decide what fields to compute reservation on,
this case is outermost case hence the very first Work order's "Skip
transfer" is considered for ALL work orders.
Solution: move the case inside Sum.
Steps to reproduce:
1. Make work order for more than 1 qty (with | without skip transfer)
2. Create manufacture and transfer entries.
3. Keep checking reserved quantities during this process.
* test: use default warehouse for testing reservation
(cherry picked from commit 80f1a8c645)
* fix(UX): hide RM table(Job Card) if material transfer is against work order (#28746)
(cherry picked from commit cc8c0595e2)
# Conflicts:
# erpnext/manufacturing/doctype/job_card/job_card.js
* chore: resolve conflicts
Co-authored-by: Noah Jacob <noahjacobkurian@gmail.com>
Co-authored-by: Ankush Menat <ankush@frappe.io>
* fix: fixes in work order doctype
* fix: sider issues and disabled set only once property
* fix: set default qty to manufacture
* fix: dont manually collapse sections
* fix: remove unnecessary messages
* fix: make dependent fields read only
Co-authored-by: Mohammed Yusuf Shaikh <49878143+mohammedyusufshaikh@users.noreply.github.com>
Co-authored-by: Ankush Menat <ankush@frappe.io>
* fix: Closed status error in Work Order Summary (#28460)
* fix: Closed status error in Work Order Summary
* chore: use get_meta to get status options
* refactor: simplify code
Co-authored-by: Ankush Menat <ankush@frappe.io>
(cherry picked from commit aa689874e3)
# Conflicts:
# erpnext/manufacturing/report/work_order_summary/work_order_summary.py
* fix: resolve conflict
Co-authored-by: Sagar Sharma <63660334+s-aga-r@users.noreply.github.com>
Co-authored-by: Ankush Menat <ankush@frappe.io>
* fix: Closed status error in Work Order Summary (#28460)
* fix: Closed status error in Work Order Summary
* chore: use get_meta to get status options
* refactor: simplify code
Co-authored-by: Ankush Menat <ankush@frappe.io>
(cherry picked from commit aa689874e3)
# Conflicts:
# erpnext/manufacturing/report/work_order_summary/work_order_summary.py
* fix: resolve conflict
Co-authored-by: Sagar Sharma <63660334+s-aga-r@users.noreply.github.com>
Co-authored-by: Ankush Menat <ankush@frappe.io>
* fix: added job_card_item links in material request
* fix: add no copy to row references
(cherry picked from commit cb064b06ad)
Co-authored-by: Noah Jacob <noahjacobkurian@gmail.com>
* fix: added job_card_item links in material request
* fix: add no copy to row references
(cherry picked from commit cb064b06ad)
Co-authored-by: Noah Jacob <noahjacobkurian@gmail.com>
- make `tests_that_skip_setup` a tuple (added comma)
- remove manual teardown in `test_job_card_material_transfer_correctness` to avoid premature committing
- transfer_material_against = "Job Card" while making BOM with mulitple operations
(cherry picked from commit bb561ba7a8)
- `get_cached_value` throws a DoesNotExistError if non-existent value, used `get_value` instead
- accomodate production plan items that dont have WO/PO against them as well (blank values)
- added some None value handling to avoid AttributeError
(cherry picked from commit 1eb3ca2b86)
- Fixed transferred qty not back updating on JC if partial transfer
- Partial transfer not mapping pending qty from JC correctly in SE
- tests for above cases
- minor code cleanup
(cherry picked from commit e8d0c25dff)
