Openbravo Issue Tracking System - POS2
View Issue Details
0057671POS2POSpublic2025-01-17 06:332025-02-06 14:44
axelmercado 
prakashmurugesan88 
highmajoralways
closedfixed 
5
 
25Q2 
OBPS
110411
No
0057671: Problem with “Maximum discount per role” functionality, does not work correctly.
We apply the changes but there is an error in the way decimals and rounding are handled, resulting in inconsistencies when validating the maximum discounts allowed. This allows the system to accept final prices that are below the calculated limits for both the Unit Price and the List Price.
Navigate to role and configure the detailed actions and the maximum discount as below.
In the terminal, add a product and to the line, add a manual discount as below.
You can notice following Rounding issue.


Initial Context:

Product with List Price: 5.90€.
Product with Unit Price (price already reduced): 4.90€.
Observed Behavior:

The system warns that the maximum discount allowed is 16%, based on the Unit Price.

Calculated discount: 4.90€ × 0.16 = 0.784€ ≈ 0.78€ (rounding).
Expected minimum allowed price: €4.90 - €0.78 = €4.12.
However, the POS allows to apply a price of €4.11, which is below the expected limit of €4.12.

Extended Case:

If the original List Price of €5.90 is considered, a maximum discount of 30% would imply:

Minimum price allowed: €5.90 × 0.70 = €4.13.
But the system allows applying a discount that results in a final price of 4.11€, equivalent to a discount of 30.33%, exceeding the established limit.
No tags attached.
related to defect 0056718 closed prakashmurugesan88 Problem with “Maximum discount per role” functionality, does not work correctly. 
depends on backport 005767225Q1 closed prakashmurugesan88 Problem with “Maximum discount per role” functionality, does not work correctly. 
Issue History
2025-01-17 06:33prakashmurugesan88New Issue
2025-01-17 06:33prakashmurugesan88Assigned To => prakashmurugesan88
2025-01-17 06:33prakashmurugesan88OBNetwork customer => No
2025-01-17 06:33prakashmurugesan88Triggers an Emergency Pack => No
2025-01-17 06:34prakashmurugesan88Relationship addedrelated to 0056718
2025-01-17 06:36prakashmurugesan88Issue cloned0057672
2025-01-17 06:40prakashmurugesan88Relationship addeddepends on 0057672
2025-01-17 06:40prakashmurugesan88Statusnew => scheduled
2025-01-17 06:58prakashmurugesan88Note Added: 0174253
2025-01-17 07:55prakashmurugesan88Note Edited: 0174253bug_revision_view_page.php?bugnote_id=0174253#r29064
2025-01-17 21:52axelmercadoOBNetwork customerNo => OBPS
2025-01-17 21:52axelmercadoSupport ticket => 110411
2025-01-17 21:52axelmercadoReporterprakashmurugesan88 => axelmercado
2025-02-06 14:44hgbotResolutionopen => fixed
2025-02-06 14:44hgbotStatusscheduled => resolved
2025-02-06 14:44hgbotFixed in Version => 25Q2
2025-02-06 14:44hgbotNote Added: 0175225
2025-02-06 14:44hgbotNote Added: 0175226
2025-02-06 14:44hgbotNote Added: 0175227
2025-02-06 14:44hgbotNote Added: 0175228
2025-02-06 14:44marvintmStatusresolved => closed

Notes
(0174253)
prakashmurugesan88   
2025-01-17 06:58   
(edited on: 2025-01-17 07:55)
Created following Merge Request
https://gitlab.com/openbravo/product/pmods/org.openbravo.pos2/-/merge_requests/3359 [^]

(0175225)
hgbot   
2025-02-06 14:44   
Repository: https://gitlab.com/orisha-group/bu-commerce/openbravo/product/pmods/org.openbravo.pos2 [^]
Changeset: 2adcef8831a65f041ff5410e057731a55ce181cd
Author: Prakash Murugesan <p.murugesan@external.orisha.com>
Date: 06-02-2025 13:44:07
URL: https://gitlab.com/orisha-group/bu-commerce/openbravo/product/pmods/org.openbravo.pos2/-/commit/2adcef8831a65f041ff5410e057731a55ce181cd [^]

Fixes ISSUE-57671: Fixed percentage calculation rounding issue

---
M web-jspack/org.openbravo.pos2/src/model/ticket/user-actions/discounts/ApplyHigherDiscountThanAllowedForRole.js
---
(0175226)
hgbot   
2025-02-06 14:44   
Repository: https://gitlab.com/orisha-group/bu-commerce/openbravo/product/pmods/org.openbravo.pos2 [^]
Changeset: 3c08f173766d6c7ed8cd15d450db318affb39fb0
Author: Prakash Murugesan <p.murugesan@external.orisha.com>
Date: 06-02-2025 13:44:07
URL: https://gitlab.com/orisha-group/bu-commerce/openbravo/product/pmods/org.openbravo.pos2/-/commit/3c08f173766d6c7ed8cd15d450db318affb39fb0 [^]

Related to ISSUE-57671: Fixed error message

---
M src-db/database/sourcedata/AD_MESSAGE.xml
M web-jspack/org.openbravo.pos2/src/model/ticket/user-actions/discounts/ApplyHigherDiscountThanAllowedForRole.js
---
(0175227)
hgbot   
2025-02-06 14:44   
Repository: https://gitlab.com/orisha-group/bu-commerce/openbravo/product/pmods/org.openbravo.pos2 [^]
Changeset: debf15301b82acccfdfed6ede6535b859a7419bc
Author: Prakash Murugesan <p.murugesan@external.orisha.com>
Date: 06-02-2025 13:44:07
URL: https://gitlab.com/orisha-group/bu-commerce/openbravo/product/pmods/org.openbravo.pos2/-/commit/debf15301b82acccfdfed6ede6535b859a7419bc [^]

Related to ISSUE-57671: Fixed copy right

---
M web-jspack/org.openbravo.pos2/src/model/ticket/user-actions/discounts/ApplyHigherDiscountThanAllowedForRole.js
---
(0175228)
hgbot   
2025-02-06 14:44   
Repository: https://gitlab.com/orisha-group/bu-commerce/openbravo/product/pmods/org.openbravo.pos2 [^]
Changeset: bbda20a8a103727e95c2f73b47b611a2d8f10a96
Author: Prakash Murugesan <p.murugesan@external.orisha.com>
Date: 06-02-2025 13:44:07
URL: https://gitlab.com/orisha-group/bu-commerce/openbravo/product/pmods/org.openbravo.pos2/-/commit/bbda20a8a103727e95c2f73b47b611a2d8f10a96 [^]

Related to ISSUE-57671: Fixed jest test

---
M web-jspack/org.openbravo.pos2/src/model/ticket/user-actions/discounts/__test__/areDiscountsExceedingMaximumAllowedByRole.test.js
---