Project:
| View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
| ID | ||||||||
| 0052765 | ||||||||
| Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
| backport | [POS2] Core | minor | have not tried | 2023-06-16 08:43 | 2023-06-16 09:02 | |||
| Reporter | AugustoMauch | View Status | public | |||||
| Assigned To | AugustoMauch | |||||||
| Priority | normal | Resolution | fixed | Fixed in Version | ||||
| Status | closed | Fix in branch | Fixed in SCM revision | |||||
| Projection | none | ETA | none | Target Version | 22Q4.2 | |||
| 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 | 0052765: Popup should not be displayed when the application is refreshed programatically | |||||||
| Description | When the user tries to refresh the browser manually, a popup is displayed to ensure the state is persisted before refreshing the browser. But that popup is not needed when the application is refreshed programatically, we could automatically force the persistence of the state as part of that programatical refresh. | |||||||
| Steps To Reproduce | - Open POS2 - Log in - In the back office, do a change in a terminal property (i.e. change the default business partner and address) - In POS2, refresh the master data. A refresh will be tried because there were changes in the terminal, and the intrusive popup will be displayed that will allow the user to skip the refresh resulting in inconsistencies | |||||||
| Tags | No tags attached. | |||||||
| Attached Files | ||||||||
 Relationships [ Relation Graph ]
[ Dependency Graph ]
|
||||||||
|
||||||||
 Relationships |
|
Notes |
|
|
(0151259) hgbot (developer) 2023-06-16 09:01 |
Merge Request created: https://gitlab.com/openbravo/product/pmods/org.openbravo.core2/-/merge_requests/1138 [^] |
|
(0151260) hgbot (developer) 2023-06-16 09:02 |
Merge Request created: https://gitlab.com/openbravo/product/pmods/org.openbravo.mobile.core/-/merge_requests/554 [^] |
|
(0151261) hgbot (developer) 2023-06-16 09:02 |
Directly closing issue as related merge request is already approved. Repository: https://gitlab.com/openbravo/product/pmods/org.openbravo.core2 [^] Changeset: 5c722e7cb45328279484ab137d0713f8741f010a Author: Augusto Mauch <augusto.mauch@openbravo.com> Date: 16-06-2023 09:00:29 URL: https://gitlab.com/openbravo/product/pmods/org.openbravo.core2/-/commit/5c722e7cb45328279484ab137d0713f8741f010a [^] Fixes ISSUE-52765: Uses OB.App.reload instead of window.location.reload The OB.App.reload function will ensure that the state is persisted before reload and also that the popup that is open when the user refreshes manually to have time to persist the state is not rendered --- M web-jspack/org.openbravo.core2/src/components/AppBar/ProfileSelector/ProfileSelector.jsx M web-jspack/org.openbravo.core2/src/components/AppBar/ProfileSelector/__test__/ProfileSelector.test.jsx M web-jspack/org.openbravo.core2/src/components/ErrorFallback/ErrorFallback.jsx M web-jspack/org.openbravo.core2/src/core/authentication/LoginProcess.js M web-jspack/org.openbravo.core2/src/core/authentication/__test__/LoginProcess.test.js M web-jspack/org.openbravo.core2/src/core/masterdata/MasterdataRefresh.js M web-jspack/org.openbravo.core2/src/model/user-interface/user-actions/RefreshMasterdata.js M web-jspack/org.openbravo.core2/src/registerServiceWorker.js M web-jspack/org.openbravo.core2/src/serviceWorkerRegistration.js --- |
|
(0151262) hgbot (developer) 2023-06-16 09:02 |
Merge request merged: https://gitlab.com/openbravo/product/pmods/org.openbravo.core2/-/merge_requests/1138 [^] |
|
(0151263) hgbot (developer) 2023-06-16 09:02 |
Merge request merged: https://gitlab.com/openbravo/product/pmods/org.openbravo.mobile.core/-/merge_requests/554 [^] |
|
(0151264) hgbot (developer) 2023-06-16 09:02 |
Directly closing issue as related merge request is already approved. Repository: https://gitlab.com/openbravo/product/pmods/org.openbravo.mobile.core [^] Changeset: 9fe57a106ab24959b9c2db7a99395327cde4f7a0 Author: Augusto Mauch <augusto.mauch@openbravo.com> Date: 16-06-2023 09:01:29 URL: https://gitlab.com/openbravo/product/pmods/org.openbravo.mobile.core/-/commit/9fe57a106ab24959b9c2db7a99395327cde4f7a0 [^] Fixes ISSUE-52765: Adds a function to reload the properly reload the application programatically The new function OB.App.reload will ensure the state is flushed before the reload, and will also unregister the onbeforeunload listener to prevent the reload popup from appearing --- M web/org.openbravo.mobile.core/app/model/application-state/StatePersistence.js --- |
|
Notes |
|
Issue History |
|||
| Date Modified | Username | Field | Change |
| 2023-06-16 08:43 | AugustoMauch | New Issue | |
| 2023-06-16 08:43 | AugustoMauch | Assigned To | => AugustoMauch |
| 2023-06-16 08:43 | AugustoMauch | Triggers an Emergency Pack | => No |
| 2023-06-16 08:43 | AugustoMauch | Issue generated from | 0052764 |
| 2023-06-16 09:01 | hgbot | Note Added: 0151259 | |
| 2023-06-16 09:02 | hgbot | Note Added: 0151260 | |
| 2023-06-16 09:02 | hgbot | Resolution | open => fixed |
| 2023-06-16 09:02 | hgbot | Status | new => closed |
| 2023-06-16 09:02 | hgbot | Fixed in Version | => 22Q4.3 |
| 2023-06-16 09:02 | hgbot | Note Added: 0151261 | |
| 2023-06-16 09:02 | hgbot | Note Added: 0151262 | |
| 2023-06-16 09:02 | hgbot | Note Added: 0151263 | |
| 2023-06-16 09:02 | hgbot | Fixed in Version | 22Q4.3 => RR22Q4.3 |
| 2023-06-16 09:02 | hgbot | Note Added: 0151264 | |
| 2023-06-21 09:26 | AugustoMauch | Relationship added | blocks 0052314 |
|
Issue History |
| Copyright © 2000 - 2009 MantisBT Group |
 |