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

View Issue Details[ Jump to Notes ] [ Issue History ] [ Print ]
ID
0052765
TypeCategorySeverityReproducibilityDate SubmittedLast Update
backport[POS2] Coreminorhave not tried2023-06-16 08:432023-06-16 09:02
ReporterAugustoMauchView Statuspublic 
Assigned ToAugustoMauch 
PrioritynormalResolutionfixedFixed in Version
StatusclosedFix in branchFixed in SCM revision
ProjectionnoneETAnoneTarget Version22Q4.2
OSAnyDatabaseAnyJava version
OS VersionDatabase versionAnt version
Product VersionSCM revision 
Merge Request Statusapproved
Review Assigned To
OBNetwork customerNo
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary

0052765: Popup should not be displayed when the application is refreshed programatically

DescriptionWhen 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
TagsNo tags attached.
Attached Files

- Relationships [ Relation Graph ] [ Dependency Graph ]
blocks defect 0052314 closedAugustoMauch Popup should not be displayed when the application is refreshed programatically 

-  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
---

- 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 OBNetwork customer => No
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 Merge Request Status => open
2023-06-16 09:01 hgbot Note Added: 0151259
2023-06-16 09:02 hgbot Note Added: 0151260
2023-06-16 09:02 hgbot Merge Request Status open => approved
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

Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker