Openbravo Issue Tracking System - POS2
View Issue Details
0046646POS2POSpublic2021-05-12 15:342021-06-29 17:14
jorge-garcia 
javierRodriguez 
normalmajorhave not tried
closedfixed 
5
 
TAPTAP 
guilleaer
No
0046646: Allow multiple instances per line flag for manual promotionsis not working properly
Allow multiple instances per line allows to define the same manual promotion (m_offer_id) to be applied to the same line.

If the flag is unchecked, in old POS if a manual promotion is added to a line but this promotion was already in the line, the new added promotion overrides the previous applied promotion

If the flag is checked,every time this manual promotion is added, it will generate a new instance of the promotion (discountinstance property from promotion object)
1) Configure Allow multiple instances per line allows flag in any way
2) The behavior is always the same
For new POS, this is the expected behavior with "Allow multiple instances per line" flag:
  - Manual Promotion with flag CHECKED
      The promotion is added always to the ticket and the promotion object should generate a discountinstance value using an UUID
  - Manual Promotion with flag UNCHECKED
    - Ticket has not applied the promotion -> Add the promotion
    - Ticket has applied the promotion before -> A error message is shown indicating it cannot be added again

To know if a promotion is the same, we should always check the id and the discountinstance of the promotion
      
No tags attached.
blocks defect 0046645 closed javierRodriguez Allow multiple instances per line flag for manual promotionsis not working properly 
Issue History
2021-05-12 15:39guilleaerTypedefect => backport
2021-05-12 15:39guilleaerTarget Version => TAP
2021-05-12 15:39guilleaerStatusscheduled => acknowledged
2021-05-19 17:45javierRodriguezAssigned ToRetail => javierRodriguez
2021-05-19 17:45javierRodriguezStatusacknowledged => scheduled
2021-05-19 17:53javierRodriguezNote Added: 0128418
2021-05-19 17:53javierRodriguezNote Edited: 0128418bug_revision_view_page.php?bugnote_id=0128418#r22562
2021-05-19 17:53javierRodriguezNote Added: 0128419
2021-05-19 17:53javierRodriguezStatusscheduled => resolved
2021-05-19 17:53javierRodriguezResolutionopen => fixed
2021-06-29 17:14guilleaerReview Assigned To => guilleaer
2021-06-29 17:14guilleaerStatusresolved => closed
2021-06-29 17:14guilleaerFixed in Version => TAP

Notes
(0128418)
javierRodriguez   
2021-05-19 17:53   
fixed in commit: https://gitlab.com/obcustomers/AGAP/org.openbravo.pos2/-/commit/50454f6649c40d31ca315daecfa9348af8a9a47b [^]

MR: https://gitlab.com/openbravo/product/pmods/org.openbravo.pos2/-/merge_requests/469 [^]

(0128419)
javierRodriguez   
2021-05-19 17:53   
resolved