Anonymous | Login
Project:
RSS
  
News | My View | View Issues | Roadmap | Summary

View Issue DetailsJump to Notes ] Issue History ] Print ]
ID
0045618
TypeCategorySeverityReproducibilityDate SubmittedLast Update
backport[Retail Modules] Web POScriticalalways2020-12-15 18:372020-12-18 06:47
ReporterguillermogilView Statuspublic 
Assigned Toprakashmurugesan88 
PriorityimmediateResolutionfixedFixed in VersionRR20Q3.3
StatusclosedFix in branchFixed in SCM revision
ProjectionnoneETAnoneTarget VersionRR20Q3.3
OSAnyDatabaseAnyJava version
OS VersionDatabase versionAnt version
Product VersionRR19Q3.3SCM revision 
Review Assigned To
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary

0045618: Not possible to click several buttons after an addProduct

DescriptionNot possible to click several buttons after an addProduct.
There are several exceptions in the code that avoid to finish the addProdct functionality but the ProcessController is never closed.

e.g.-
in _addProduct:

                  if (OB.MobileApp.model.get('inPaymentTab')) {
                    if (args.options && args.options.blockAddProduct) {
                      OB.error(
                        'An add product is executed. At this point, this action is not allowed. Skipping product ' +
                          p.get('_identifier')
                      );
                      if (args && args.attrs && args.attrs.obposEpccode) {
                        OB.UTIL.RfidController.removeEpc(
                          args.attrs.obposEpccode
                        );
                      }
                      return;
                    }
                  }
That returns breaks the flow as it is not executing the finalCallback.
There are several other cases across the code.

The code is still the same on PI although it is on the legacy function. Releases <20Q4 will have this issue in the case exposed.
Steps To ReproduceDo an addProduct in the payment tab
ProcessController starts
Log is raised: An add product is executed. At this point, this action is not allowed. Skipping product
Process controller is never finished

After that you get the 120s timeout from the ProcessController
Buttons are deactivated forever
Proposed SolutionReturn the callback on the _addProduct with the success as false
TagsNo tags attached.
Attached Files

- Relationships Relation Graph ] Dependency Graph ]
blocks defect 0045589 closedprakashmurugesan88 Not possible to click several buttons after an addProduct 

-  Notes
(0124897)
hgbot (developer)
2020-12-17 10:50

Merge Request created: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal/-/merge_requests/223 [^]
(0124911)
hgbot (developer)
2020-12-18 06:47

Merge request merged: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal/-/merge_requests/223 [^]
(0124912)
hgbot (developer)
2020-12-18 06:47

Directly closing issue as related merge request is already approved.

Repository: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal [^]
Changeset: 19fecce3d8774d6b234a404e83b901d27f08ad60
Author: Prakash M <prakash@qualiantech.com>
Date: 2020-12-17T15:16:35+05:30
URL: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal/-/commit/19fecce3d8774d6b234a404e83b901d27f08ad60 [^]

Fixed BUG-45618: Fixes process timeout when error raised in addProduct
* Added callback with success as false to handle process completion

---
M web/org.openbravo.retail.posterminal/js/model/order.js
---

- Issue History
Date Modified Username Field Change
2020-12-17 10:38 prakashmurugesan88 Type defect => backport
2020-12-17 10:38 prakashmurugesan88 Target Version => RR20Q3.3
2020-12-17 10:50 hgbot Note Added: 0124897
2020-12-18 06:47 hgbot Resolution open => fixed
2020-12-18 06:47 hgbot Status scheduled => closed
2020-12-18 06:47 hgbot Note Added: 0124911
2020-12-18 06:47 hgbot Fixed in Version => RR20Q3.3
2020-12-18 06:47 hgbot Note Added: 0124912


Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker