Openbravo Issue Tracking System - Openbravo ERP |
| View Issue Details |
|
| ID | Project | Category | View Status | Date Submitted | Last Update |
| 0052983 | Openbravo ERP | A. Platform | public | 2023-07-17 16:35 | 2023-07-27 08:56 |
|
| Reporter | shuehner | |
| Assigned To | shuehner | |
| Priority | normal | Severity | minor | Reproducibility | have not tried |
| Status | closed | Resolution | fixed | |
| Platform | | OS | 5 | OS Version | |
| Product Version | | |
| Target Version | | Fixed in Version | PR23Q4 | |
| Merge Request Status | approved |
| Review Assigned To | |
| OBNetwork customer | No |
| Web browser | |
| Modules | Core |
| Support ticket | |
| Regression level | |
| Regression date | |
| Regression introduced in release | |
| Regression introduced by commit | |
| Triggers an Emergency Pack | No |
|
| Summary | 0052983: Remove unused/obsolete 2.50 style alert code from VerticalMenu |
| Description | VerticalMenu.html still contains chunk of code related to old 2.50 style alerting.
In 2.50 a period 50s timer was setup in that file to call the backend to check if new Alerts are present.
That is long obsolete and the server-side code was removed in 2017 here
https://gitlab.com/openbravo/product/openbravo/-/commit/e59ab256267daccf41a96e21cbda151f88dd2ce2 [^]
The frontend code is still present and would be used if hideMenu=false param was passed.
It can't work anymore with both the removed backend code + the domElement alertCount gone.
|
| Steps To Reproduce | - |
| Proposed Solution | Delete the obsolete client side code. |
| Additional Information | |
| Tags | No tags attached. |
| Relationships | |
| Attached Files | |
|
| Issue History |
| Date Modified | Username | Field | Change |
| 2023-07-17 16:35 | shuehner | New Issue | |
| 2023-07-17 16:35 | shuehner | Assigned To | => shuehner |
| 2023-07-17 16:35 | shuehner | OBNetwork customer | => No |
| 2023-07-17 16:35 | shuehner | Modules | => Core |
| 2023-07-17 16:35 | shuehner | Triggers an Emergency Pack | => No |
| 2023-07-17 16:42 | hgbot | Merge Request Status | => open |
| 2023-07-17 16:42 | hgbot | Note Added: 0152566 | |
| 2023-07-27 08:56 | hgbot | Merge Request Status | open => approved |
| 2023-07-27 08:56 | hgbot | Resolution | open => fixed |
| 2023-07-27 08:56 | hgbot | Status | new => closed |
| 2023-07-27 08:56 | hgbot | Note Added: 0152903 | |
| 2023-07-27 08:56 | hgbot | Fixed in Version | => PR23Q4 |
| 2023-07-27 08:56 | hgbot | Note Added: 0152904 | |