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.

Sign up for GitHub

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

Jump to bottom

[Bug Report] Recalculate price for RMA #4255

Open
yamelsenih opened this issue Dec 1, 2023 · 1 comment
Open

[Bug Report] Recalculate price for RMA #4255

yamelsenih opened this issue Dec 1, 2023 · 1 comment
Assignees
@yamelsenih
Labels
02 bug 02 bugfix -> 3.9.4.002 (SO) Sales Management

Comments

@yamelsenih
Copy link
Member

Bug Report

When a customer or supplier return is made from a sales order, prices are not maintained. This is not correct since returns are always valued at the price of the original document.

Step by Step

  1. Create a Sales Order with at least one product and complete
  2. Change the price of the product that is on the sales order (from the price list window)
  3. Place a return order
@yamelsenih yamelsenih self-assigned this Dec 1, 2023
This was referenced Dec 1, 2023
Open
Merged
yamelsenih added a commit to erpcya/adempiere that referenced this issue Dec 3, 2023
@yamelsenih
Merge branch 'hotfix/3.9.4.002' into hotfix/3.9.4.002-local/adempiere…
712f826 
…#4255-Recalculate_price_for_RMA
@SusanneCalderon
Copy link
Collaborator

Hi @yamelsenih
I could not reproduce the error.
Steps:
1.
PriceList Old, Product Azalee, $30
00_PL_Old

New Order, Product Azalee, Paymentterm Immediate, Tax exempt, Price $30, Linenetamt $30
01_Orderline

  1. Change PriceList product azalee to $35
    02_PL_New

  2. Cache Reset

  3. RMA, Product Azalee, Price $30 as in the Orderline
    04_rmaline

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@yamelsenih yamelsenih

Labels
02 bug 02 bugfix -> 3.9.4.002 (SO) Sales Management
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@yamelsenih @SusanneCalderon