Project:
View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
ID | ||||||||
0046786 | ||||||||
Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
defect | [POS2] POS | minor | have not tried | 2021-05-21 13:19 | 2021-07-29 16:30 | |||
Reporter | caristu | View Status | public | |||||
Assigned To | hernan-dp | |||||||
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 | 0046786: OB.App.State.Configuration.setCurrentWindow should not be directly called | |||||||
Description | We have different places[1],[2] where the OB.App.State.Configuration.setCurrentWindow state action is being directly called. By doing this all the onExit/onEnter hook execution is skipped because they are fired from the user action layer. It would be nice to fix all the places doing this call and replace the correct user action to switch the window. [1] https://gitlab.com/openbravo/product/pmods/org.openbravo.pos2/-/blob/master/web-jspack/org.openbravo.pos2/src/model/user-interface/user-actions/SelectCurrentSchema.js#L29 [^] [2] https://gitlab.com/openbravo/product/pmods/org.openbravo.pos2/-/blob/master/web-jspack/org.openbravo.pos2/src/model/user-interface/user-actions/initialCount/OpenInitialCount.js#L35 [^] | |||||||
Steps To Reproduce | . | |||||||
Proposed Solution | Use the corresponding user action to navigate to the corresponding window/flow instead of calling OB.App.State.Configuration.setCurrentWindow. See[1] [1] https://docs.google.com/document/d/1tuGxwiNnThguqkrSfUmv9gktmdjdlWmcIkbTPUhVV3g/edit#heading=h.ap1d4lcsowsb [^] | |||||||
Tags | No tags attached. | |||||||
Attached Files | ||||||||
![]() |
|
![]() |
|||
Date Modified | Username | Field | Change |
2021-05-21 13:19 | caristu | New Issue | |
2021-05-21 13:19 | caristu | Assigned To | => Retail |
2021-05-21 13:19 | caristu | OBNetwork customer | => No |
2021-05-21 13:19 | caristu | Resolution time | => 1625176800 |
2021-05-21 13:19 | caristu | Triggers an Emergency Pack | => No |
2021-05-21 18:22 | guilleaer | Assigned To | Retail => platform |
2021-05-21 18:22 | guilleaer | Status | new => acknowledged |
2021-07-14 13:26 | dmiguelez | Resolution time | 1625176800 => 1633039200 |
2021-07-26 17:48 | hgbot | Merge Request Status | => open |
2021-07-26 17:48 | hgbot | Note Added: 0130765 | |
2021-07-26 17:49 | hgbot | Note Added: 0130766 | |
2021-07-27 12:47 | hgbot | Merge Request Status | open => approved |
2021-07-27 12:47 | hgbot | Note Added: 0130791 | |
2021-07-27 12:48 | hgbot | Note Added: 0130792 | |
2021-07-27 12:48 | dmiguelez | Status | acknowledged => scheduled |
2021-07-27 12:48 | dmiguelez | Assigned To | platform => hernan-dp |
2021-07-27 12:48 | dmiguelez | Status | scheduled => resolved |
2021-07-27 12:48 | dmiguelez | Resolution | open => fixed |
2021-07-27 12:48 | dmiguelez | Status | resolved => closed |
2021-07-29 16:28 | hgbot | Note Added: 0130876 | |
2021-07-29 16:30 | hgbot | Note Added: 0130878 | |
2021-07-29 16:30 | hgbot | Note Added: 0130879 |
Copyright © 2000 - 2009 MantisBT Group |