Project:
View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
ID | ||||||||
0047184 | ||||||||
Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
defect | [POS2] Core | minor | always | 2021-06-18 12:29 | 2021-07-15 10:26 | |||
Reporter | gorka_gil | View Status | public | |||||
Assigned To | alostale | |||||||
Priority | normal | 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 | |||||||
Merge Request Status | approved | |||||||
Review Assigned To | ||||||||
OBNetwork customer | No | |||||||
Support ticket | ||||||||
Regression level | ||||||||
Regression date | ||||||||
Regression introduced in release | ||||||||
Regression introduced by commit | ||||||||
Triggers an Emergency Pack | No | |||||||
Summary | 0047184: Confirmation dialog not available on reload, on masterdata load error | |||||||
Description | When there is a masterdata load error, a popup is show with the models that have fail, and only option is reload. If reloading also happen an error in masterdata, then, when trying to show the popup (confirmation dialog) it is still not defined and gives an error. | |||||||
Steps To Reproduce | 1. Place a breakpoint in any masterdata load query ( for example in Product.java, line 62) 2. In a pos2 terminal, clean cache and reload, make login 3. It will stop in product load and when it gives timeout will show the error popup 4. Click on the reload in the popup 5. It will start again the load of masterdata and stop again in the product model 6. PROBLEM: After the timeout instead of show the popup will appear a javascript error saying that confirmation dialog is not defined | |||||||
Proposed Solution | The initialization of the confirmation dialog looks different the first login and the reload, in the reload should be initialized at same moment. | |||||||
Tags | No tags attached. | |||||||
Attached Files | ![]() ![]() | |||||||
![]() |
||||||||
|
![]() |
|
(0130464) hgbot (developer) 2021-07-14 12:16 |
Merge Request created: https://gitlab.com/openbravo/product/pmods/org.openbravo.core2/-/merge_requests/594 [^] |
(0130495) hgbot (developer) 2021-07-15 10:26 |
Merge request merged: https://gitlab.com/openbravo/product/pmods/org.openbravo.core2/-/merge_requests/594 [^] |
(0130496) hgbot (developer) 2021-07-15 10:26 |
Directly closing issue as related merge request is already approved. Repository: https://gitlab.com/openbravo/product/pmods/org.openbravo.core2 [^] Changeset: 896d7ef1875c61af51f93797bcf8b6391b33f601 Author: Asier Lostalé <asier.lostale@openbravo.com> Date: 2021-07-15T08:25:48+00:00 URL: https://gitlab.com/openbravo/product/pmods/org.openbravo.core2/-/commit/896d7ef1875c61af51f93797bcf8b6391b33f601 [^] fixes BUG-47184: 2nd failed attempt to reload masterdata shows tech error When masterdata fails to load on login, the first time a confirmation dialog was displayed to the user to try to reload the page, if while reloading it failed again an error was thrown because confiramation dialog wasn't available. ConfirmationDialog was rendered in LoginScreen and in App as a gloabal component. It worked on first attempt as it was rendered within LoginScreen but not in the 2nd one as LoginScreen is not necessary but at this stage App has not been rendered yet. Fixed by moving ConfirmationDialog to index.jsx as part of the whole layout. Note this makes unecessary to render it again as a global component. --- M web-jspack/org.openbravo.core2/src/components/AuthProvider/LoginScreen.jsx M web-jspack/org.openbravo.core2/src/index.jsx M web-jspack/org.openbravo.core2/src/ob-init.js --- |
![]() |
|||
Date Modified | Username | Field | Change |
2021-06-18 12:29 | gorka_gil | New Issue | |
2021-06-18 12:29 | gorka_gil | Assigned To | => Retail |
2021-06-18 12:29 | gorka_gil | OBNetwork customer | => No |
2021-06-18 12:29 | gorka_gil | Resolution time | => 1627596000 |
2021-06-18 12:29 | gorka_gil | Triggers an Emergency Pack | => No |
2021-06-18 12:31 | gorka_gil | File Added: 47184-popup.png | |
2021-06-18 12:32 | gorka_gil | File Added: 47184-error.png | |
2021-06-18 12:32 | gorka_gil | Status | new => acknowledged |
2021-06-18 12:36 | marvintm | Assigned To | Retail => platform |
2021-06-21 10:06 | gorka_gil | Description Updated | View Revisions |
2021-07-14 12:03 | alostale | Status | acknowledged => scheduled |
2021-07-14 12:03 | alostale | Assigned To | platform => alostale |
2021-07-14 12:16 | hgbot | Merge Request Status | => open |
2021-07-14 12:16 | hgbot | Note Added: 0130464 | |
2021-07-15 10:26 | hgbot | Merge Request Status | open => approved |
2021-07-15 10:26 | hgbot | Resolution | open => fixed |
2021-07-15 10:26 | hgbot | Status | scheduled => closed |
2021-07-15 10:26 | hgbot | Note Added: 0130495 | |
2021-07-15 10:26 | hgbot | Note Added: 0130496 | |
2022-12-14 08:45 | meriem_azaf | Relationship added | related to 0051148 |
Copyright © 2000 - 2009 MantisBT Group |