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: billed_qty to show a sum of all invoiced qty from the purchase order item. #37539

Merged
merged 1 commit into from
Oct 18, 2023
Merged

fix: billed_qty to show a sum of all invoiced qty from the purchase order item. #37539

merged 1 commit into from
Oct 18, 2023

Conversation

HarryPaulo
Copy link
Contributor

when i create a purchase order with one item and 2 qty, and a make 2 invoices for this order, invoicing 1 qty each, the column billed_qty must be agragated with sum.

Currently the system presents an incorrect value, showing only the first quantity invoiced.

With my change, the REAL amount will be displayed in the sum.

@github-actions github-actions bot added the needs-tests This PR needs automated unit-tests. label Oct 16, 2023
@s-aga-r s-aga-r merged commit fbd8660 into frappe:develop Oct 18, 2023
10 of 12 checks passed
@s-aga-r s-aga-r added backport version-14-hotfix backport to version 14 and removed needs-tests This PR needs automated unit-tests. labels Oct 18, 2023
s-aga-r pushed a commit that referenced this pull request Oct 18, 2023
…rder item. (backport #37539) (#37558)

fix: billed_qty to show a sum of all invoiced qty from the purchase order item.

(cherry picked from commit 8a72f4f)

Co-authored-by: HarryPaulo <[email protected]>
frappe-pr-bot pushed a commit that referenced this pull request Oct 19, 2023
## [14.44.1](v14.44.0...v14.44.1) (2023-10-19)

### Bug Fixes

* billed_qty to show a sum of all invoiced qty from the purchase order item. (backport [#37539](#37539)) ([#37558](#37558)) ([ac7d6d6](ac7d6d6))
* consider received qty while creating SO -> MR (backport [#37414](#37414)) ([#37514](#37514)) ([1b94510](1b94510))
* don't set finance books if gross_purchase_amount is not set (backport [#37480](#37480)) ([#37482](#37482)) ([0590f21](0590f21))
* e-commerce permissions for address ([#37554](#37554)) ([022f85d](022f85d))
* german tranlations of "Is Return" ([f9b2355](f9b2355))
* GL Entries not getting created for PR Return (backport [#37513](#37513)) ([#37516](#37516)) ([c32258e](c32258e))
* **gp:** wrong `allocated_amount` on multi sales person invoice ([d266423](d266423))
* Incorrect vat amount in KSA VAT report ([44f7de0](44f7de0))
* inflated total amt in TDS report using back calculation ([78e22af](78e22af))
* Issues related to RFQ and Supplier Quotation on Portal (backport [#37565](#37565)) ([#37577](#37577)) ([e1504ef](e1504ef))
* keep customer/supplier website role by default ([76ef61c](76ef61c))
* keyerror on gl and pl comparision report ([6f143d3](6f143d3))
* payment entry count on supplier dashboard (backport [#37571](#37571)) ([#37575](#37575)) ([95abd79](95abd79))
* same Serial No get mapped while creating SO -> DN ([#37527](#37527)) ([5025850](5025850))
* serial and batch no get removed on save of return DN ([#37476](#37476)) ([f1814a1](f1814a1))
* Stock Reconciliation Insufficient Stock Error ([#37494](#37494)) ([9406ddb](9406ddb))
* **test:** project test case (backport [#37541](#37541)) ([#37543](#37543)) ([e23710b](e23710b))
* use `flt` to ignore TypeError ([#37481](#37481)) ([d2b22db](d2b22db))

### Performance Improvements

* index `dn_detail` in `Delivery Note Item` (backport [#37528](#37528)) ([#37530](#37530)) ([001c230](001c230))
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jan 21, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
backport version-14-hotfix backport to version 14
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants