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

View Issue DetailsJump to Notes ] Issue History ] Print ]
ID
0046491
TypeCategorySeverityReproducibilityDate SubmittedLast Update
backport[Retail Modules] Web POScriticalhave not tried2021-05-04 10:442021-05-05 08:02
ReportercaristuView Statuspublic 
Assigned Toranjith_qualiantech_com 
PriorityimmediateResolutionfixedFixed in VersionRR20Q4.4
StatusclosedFix in branchFixed in SCM revision
ProjectionnoneETAnoneTarget VersionRR20Q4.4
OSAnyDatabaseAnyJava version
OS VersionDatabase versionAnt version
Product VersionSCM revision 
Review Assigned To
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary

0046491: action preparation order is modified when the action fails

DescriptionWhen an action fails, the action preparation rollback mechanism is executed[1].

This mechanism executes all the rollback functions of each action preparation in reverse order.

In order to retrieve the reversed order of rollback functions we are using the reverse function[2]. Note that reverse function mutates the original array[3].

In practice, this means that if an action fails, the next time it is executed the action preparations are executed in reversed order.

[1] http://wiki.openbravo.com/wiki/StateController#Action_Preparations [^]
[2] https://gitlab.com/openbravo/product/pmods/org.openbravo.mobile.core/-/blob/master/web/org.openbravo.mobile.core/app/model/application-state/State.js#L60 [^]
[3] https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array/reverse [^]
Steps To Reproduce1) Put mobile.core in this branch:
2) Execute the State.test.js which contains a reproducer for the situation described in Description
3) Note that the test fails
TagsNo tags attached.
Attached Files

- Relationships Relation Graph ] Dependency Graph ]
blocks defect 0046479 closedcaristu action preparation order is modified when the action fails 

-  Notes
(0127796)
hgbot (developer)
2021-05-05 07:43

Merge Request created: https://gitlab.com/openbravo/product/pmods/org.openbravo.mobile.core/-/merge_requests/222 [^]
(0127807)
hgbot (developer)
2021-05-05 08:02

Directly closing issue as related merge request is already approved.

Repository: https://gitlab.com/openbravo/product/pmods/org.openbravo.mobile.core [^]
Changeset: 24b99546ca54f7b45b5d4aef75c75b5805402b1f
Author: Ranjith S R <ranjith@qualiantech.com>
Date: 2021-05-04T22:27:28+05:30
URL: https://gitlab.com/openbravo/product/pmods/org.openbravo.mobile.core/-/commit/24b99546ca54f7b45b5d4aef75c75b5805402b1f [^]

Fixed ISSUE-46491: Action preparation order is modified when the action fails

---
M web-test/model/application-state/State.test.js
M web/org.openbravo.mobile.core/app/model/application-state/State.js
---
(0127808)
hgbot (developer)
2021-05-05 08:02

Merge request merged: https://gitlab.com/openbravo/product/pmods/org.openbravo.mobile.core/-/merge_requests/222 [^]

- Issue History
Date Modified Username Field Change
2021-05-04 17:06 marvintm Type defect => backport
2021-05-04 17:06 marvintm Target Version => RR20Q4.4
2021-05-04 18:36 ranjith_qualiantech_com Assigned To Retail => ranjith_qualiantech_com
2021-05-05 07:43 hgbot Note Added: 0127796
2021-05-05 08:02 hgbot Resolution open => fixed
2021-05-05 08:02 hgbot Status scheduled => closed
2021-05-05 08:02 hgbot Fixed in Version => RR20Q4.4
2021-05-05 08:02 hgbot Note Added: 0127807
2021-05-05 08:02 hgbot Note Added: 0127808


Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker