Project:
| View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
| ID | ||||||||
| 0054932 | ||||||||
| Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
| backport | [POS2] Core | major | have not tried | 2024-01-02 13:07 | 2024-03-21 18:28 | |||
| Reporter | jarmendariz | View Status | public | |||||
| Assigned To | cberner | |||||||
| Priority | normal | Resolution | fixed | Fixed in Version | 23Q4.3 | |||
| Status | closed | Fix in branch | Fixed in SCM revision | |||||
| Projection | none | ETA | none | Target Version | 23Q4.3 | |||
| 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 | 0054932: Error page is not shown if the error occurs during registry process | |||||||
| Description | In case the application has a blocking error, an error page defined in ErrorFallback.jsx is shown, but this page shows the company logo using the CompanyLogo component. If this error occurs during the registration process, where the State is not available yet, the Error page won't be shown and the user will encounter a blank screen. | |||||||
| Steps To Reproduce | 1- Force an error during the registration process. This can be achieved by adding the following in checkIsRegistered() function in Registry.js: function checkIsRegistered(registry, name, type) { if (!_.has(registry, name) || name === 'OBC2_Logout') { throw Error(`${type} ${name} is not registered`); } } 2- Open POS2 application in the browser. Instead of rendering the login page or the error page, a blank screen will appear and the following errors in the browser's console: Uncaught Error: could not find react-redux context value; please ensure the component is wrapped in a <Provider> at useReduxContext (useReduxContext.js:24:1) at useSelector (useSelector.js:125:1) at CompanyLogo (CompanyLogo.jsx:31:1) ... | |||||||
| Proposed Solution | Adding a check in CompanyLogo to show no logo if the state information is not available, preventing the error situation. This way the error page will be shown at least even if it has no logo. | |||||||
| Tags | No tags attached. | |||||||
| Attached Files | ||||||||
 Relationships [ Relation Graph ]
[ Dependency Graph ]
|
||||||||
|
||||||||
 Relationships |
|
Notes |
|
|
(0162517) hgbot (developer) 2024-03-21 18:27 |
Merge Request created: https://gitlab.com/openbravo/product/pmods/org.openbravo.core2/-/merge_requests/1428 [^] |
|
(0162518) hgbot (developer) 2024-03-21 18:27 |
Merge request merged: https://gitlab.com/openbravo/product/pmods/org.openbravo.core2/-/merge_requests/1428 [^] |
|
(0162519) hgbot (developer) 2024-03-21 18:28 |
Directly closing issue as related merge request is already approved. Repository: https://gitlab.com/openbravo/product/pmods/org.openbravo.core2 [^] Changeset: 42980e7a87a87771626562197469f141e69e3d12 Author: Cristian Berner <cristian.berner@openbravo.com> Date: 21-03-2024 18:27:23 URL: https://gitlab.com/openbravo/product/pmods/org.openbravo.core2/-/commit/42980e7a87a87771626562197469f141e69e3d12 [^] Fixes ISSUE-54932: Error page is not shown if the error occurs during registry process The main problem was that CompanyLogo component is used in the ErrorFallback component, and it makes use of redux state. When errors happen in the initialization processes(registration of components, and other initializations), the ErrorFallback component was being rendered without any provider. As a fix, even when there are registration errors, CompanyLogo is added conditionally, if there's no state, it will not be added. --- M web-jspack/org.openbravo.core2/src/components/ErrorFallback/ErrorFallback.jsx --- |
|
Notes |
|
Issue History |
|||
| Date Modified | Username | Field | Change |
| 2024-03-13 10:17 | AugustoMauch | Type | defect => backport |
| 2024-03-13 10:17 | AugustoMauch | Target Version | => 23Q4.3 |
| 2024-03-21 18:27 | hgbot | Note Added: 0162517 | |
| 2024-03-21 18:27 | hgbot | Note Added: 0162518 | |
| 2024-03-21 18:27 | hgbot | Resolution | open => fixed |
| 2024-03-21 18:27 | hgbot | Status | scheduled => closed |
| 2024-03-21 18:28 | hgbot | Fixed in Version | => 23Q4.3 |
| 2024-03-21 18:28 | hgbot | Note Added: 0162519 | |
|
Issue History |
| Copyright © 2000 - 2009 MantisBT Group |
 |