Project:
View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
ID | ||||||||
0047744 | ||||||||
Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
defect | [POS2] POS | major | always | 2021-09-23 09:10 | 2021-10-07 18:11 | |||
Reporter | jmelin | View Status | public | |||||
Assigned To | caristu | |||||||
Priority | urgent | Resolution | fixed | Fixed in Version | ||||
Status | closed | Fix in branch | Fixed in SCM revision | |||||
Projection | none | ETA | none | Target Version | ||||
OS | Any | Database | Any | Java version | ||||
OS Version | Database version | Ant version | ||||||
Product Version | SCM revision | |||||||
Review Assigned To | ||||||||
Regression level | ||||||||
Regression date | ||||||||
Regression introduced in release | ||||||||
Regression introduced by commit | ||||||||
Triggers an Emergency Pack | No | |||||||
Summary | 0047744: JIRA 2312 - Quantity on article go under 0 when u press "-" to fast on the board | |||||||
Description | The issue was only reproduced a physical till (not a pc or example) When clicking on the "-" button to decrease the quantity of a product, it can create product in the sales order with negative quantity. Sometime u can get a pop up with the error "Cannot destructure property 'qty' of 'o.a.App.State.getState(...).Ticket.lines.find(...)' as it is undefined" | |||||||
Steps To Reproduce | - Add a product several times to the ticket so that its quantity is > 1 - Click on the name of the product on the ticket area so that the product popover is open. Now click as fast as possible the minus "-" sign to decrease the quantity of the product - If you click fast enough, the number of article will drop under 0 and create product in the sales order with a negative quantity. This is easier to reproduce if you set the Performance -> CPU property to 6xSlowdown in the Chrome Debugger | |||||||
Tags | No tags attached. | |||||||
Attached Files | ||||||||
Relationships [ Relation Graph ] [ Dependency Graph ] | ||||||||
|
Notes | |
(0132090) prakashmurugesan88 (developer) 2021-10-01 08:19 edited on: 2021-10-01 08:19 |
Decided to move this issue to the Platform team to be fixed in general for all user-actions in pos2 |
(0132168) hgbot (developer) 2021-10-05 14:00 |
Merge Request created: https://gitlab.com/openbravo/product/pmods/org.openbravo.core2/-/merge_requests/642 [^] |
(0132259) hgbot (developer) 2021-10-07 18:10 |
Directly closing issue as related merge request is already approved. Repository: https://gitlab.com/openbravo/product/pmods/org.openbravo.core2 [^] Changeset: 39f0e631b74fc62ee452b1a081bfc3ae14964767 Author: Carlos Aristu <carlos.aristu@openbravo.com> Date: 2021-10-07T18:09:49+02:00 URL: https://gitlab.com/openbravo/product/pmods/org.openbravo.core2/-/commit/39f0e631b74fc62ee452b1a081bfc3ae14964767 [^] fixes ISSUE-47744: display BasePopovers behind the user action scrim The UserActionScrim is a scrim that is rendered meanwhile a user action is being executed in order to avoid accidental clicks which may accidentaly launch other user actions while having a user action in progress. This scrim "protects" from this accidental clicks by being placing in front of the whole application. The problem was that default z-index of the BasePopover components makes them to be rendered ahead of the scrim. With this fix, we are making the BasePopover components to be rendered behind the scrim by setting a z-index value small enough to achieve that. --- A web-jspack/org.openbravo.core2/src/components/BasePopover/stories/MockBasePopover.jsx M web-jspack/org.openbravo.core2/src/components/BasePopover/BasePopover.jsx M web-jspack/org.openbravo.core2/src/components/BasePopover/BasePopover.scss M web-jspack/org.openbravo.core2/src/components/BasePopover/__test__/BasePopover.test.jsx M web-jspack/org.openbravo.core2/src/components/BasePopover/stories/BasePopover.stories.jsx --- |
(0132260) hgbot (developer) 2021-10-07 18:10 |
Merge request merged: https://gitlab.com/openbravo/product/pmods/org.openbravo.core2/-/merge_requests/642 [^] |
Issue History | |||
Date Modified | Username | Field | Change |
2021-09-23 09:10 | jmelin | New Issue | |
2021-09-23 09:10 | jmelin | Assigned To | => Retail |
2021-09-23 09:10 | jmelin | Triggers an Emergency Pack | => No |
2021-09-23 10:39 | guilleaer | Status | new => scheduled |
2021-09-23 10:39 | guilleaer | Status | scheduled => acknowledged |
2021-09-29 08:59 | prakashmurugesan88 | Assigned To | Retail => prakashmurugesan88 |
2021-09-29 09:00 | prakashmurugesan88 | Status | acknowledged => scheduled |
2021-10-01 08:19 | prakashmurugesan88 | Assigned To | prakashmurugesan88 => Retail |
2021-10-01 08:19 | prakashmurugesan88 | Status | scheduled => acknowledged |
2021-10-01 08:19 | prakashmurugesan88 | Note Added: 0132090 | |
2021-10-01 08:19 | prakashmurugesan88 | Note Edited: 0132090 | View Revisions |
2021-10-04 09:55 | caristu | Assigned To | Retail => caristu |
2021-10-05 14:00 | hgbot | Note Added: 0132168 | |
2021-10-05 17:48 | guilleaer | Status | acknowledged => scheduled |
2021-10-05 17:48 | guilleaer | Assigned To | caristu => AugustoMauch |
2021-10-05 18:16 | AugustoMauch | Steps to Reproduce Updated | View Revisions |
2021-10-05 18:38 | AugustoMauch | Assigned To | AugustoMauch => ablasco |
2021-10-05 18:49 | caristu | Assigned To | ablasco => caristu |
2021-10-07 18:10 | hgbot | Resolution | open => fixed |
2021-10-07 18:10 | hgbot | Status | scheduled => closed |
2021-10-07 18:10 | hgbot | Note Added: 0132259 | |
2021-10-07 18:10 | hgbot | Note Added: 0132260 | |
2021-10-07 18:11 | AugustoMauch | Status | closed => resolved |
2021-10-07 18:11 | AugustoMauch | Status | resolved => closed |
Copyright © 2000 - 2009 MantisBT Group |