Project:
View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
ID | ||||||||
0028667 | ||||||||
Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
defect | [Openbravo ERP] 09. Financial management | major | always | 2015-01-18 23:13 | 2015-04-29 20:34 | |||
Reporter | rjapoova | View Status | public | |||||
Assigned To | Triage Omni OMS | |||||||
Priority | urgent | Resolution | fixed | Fixed in Version | 3.0PR15Q1.4 | |||
Status | closed | Fix in branch | Fixed in SCM revision | PR15Q1.4 | ||||
Projection | none | ETA | none | Target Version | ||||
OS | Any | Database | Any | Java version | ||||
OS Version | Database version | Ant version | ||||||
Product Version | 3.0PR14Q4 | SCM revision | ||||||
Review Assigned To | eduardo_Argal | |||||||
Web browser | Google Chrome | |||||||
Modules | Core | |||||||
Regression level | ||||||||
Regression date | ||||||||
Regression introduced in release | ||||||||
Regression introduced by commit | ||||||||
Triggers an Emergency Pack | No | |||||||
Summary | 0028667: Cancelling a Match Statement flow does not fully return system to previous state | |||||||
Description | The Match Statement process presents two buttons: Done and Cancel. Done completes the process, Cancel should stop it and take the user back to the previous state (as if the window had never been open). Using the Advanced Matching Algorithm, however, the process creates transactions in the background. These transactions must be deleted if Cancel is pressed. Please notice that this is a functional regression: in the previous version of this window (manual window, pre PR14Q4), cancelling a match was deleting the automated transactions. Please also notice that the current behavior is very confusing for end users and a source of many mistakes. | |||||||
Steps To Reproduce | See video: https://drive.google.com/file/d/0B2uCUQlrOPvINlU3VUxNNnJpYVU/view?usp=sharing [^] | |||||||
Tags | No tags attached. | |||||||
Attached Files | ||||||||
Relationships [ Relation Graph ] [ Dependency Graph ] | ||||||||
|
Notes | |
(0076765) dmitry_mezentsev (developer) 2015-04-24 20:21 |
>Please notice that this is a functional regression: in the previous version of >this window (manual window, pre PR14Q4), cancelling a match was deleting the >automated transactions. It is done by intention as a reaction to the feedback we received from multi-user concurrent environments. When people were working in parallel previous behavior of not persisting transactions were creating different inconsistent states if two users were doing it at the same time and one was saving and another one was cancelling. Based on this the decision was made to persist all transactions after algorithm execution. This makes this process also consistent with the rest of the application: massive processes like creating invoices from orders, etc., that persist data and do not have massive cancelling operations. Together with this I agree that it is a behavior change and I propose to keep this issue to think on the way to make it clear and not confusing for the user. |
(0076897) dmitry_mezentsev (developer) 2015-04-29 20:34 |
Fixes issue 28668: Pressing Cancel in Match Statement or in Process Reconciliation does not trigger a refresh. Cancel button has been removed as it was considered confusing. Now there are two buttons, one for closing the window called 'OK', as it was before, and another one called 'Reconcile'. Both triggers refresh. Messages have been added to improve user understanding on how the window works. New message to inform the user any change will be persisted. This message can be disabled once clear and will not popup again. Message to run algorithm as well improved. |
Issue History | |||
Date Modified | Username | Field | Change |
2015-01-18 23:13 | rjapoova | New Issue | |
2015-01-18 23:13 | rjapoova | Assigned To | => dmiguelez |
2015-01-18 23:13 | rjapoova | Web browser | => Google Chrome |
2015-01-18 23:13 | rjapoova | Modules | => Core |
2015-01-18 23:13 | rjapoova | Triggers an Emergency Pack | => No |
2015-04-24 20:21 | dmitry_mezentsev | Note Added: 0076765 | |
2015-04-24 20:22 | dmitry_mezentsev | Web browser | Google Chrome => Google Chrome |
2015-04-24 20:22 | dmitry_mezentsev | Assigned To | dmiguelez => Triage Finance |
2015-04-24 20:22 | dmitry_mezentsev | Priority | normal => urgent |
2015-04-24 20:22 | dmitry_mezentsev | Severity | critical => major |
2015-04-24 20:22 | dmitry_mezentsev | Status | new => acknowledged |
2015-04-29 20:32 | dmitry_mezentsev | Relationship added | related to 0026668 |
2015-04-29 20:32 | dmitry_mezentsev | Relationship deleted | related to 0026668 |
2015-04-29 20:32 | dmitry_mezentsev | Relationship added | related to 0028668 |
2015-04-29 20:33 | dmitry_mezentsev | Status | acknowledged => scheduled |
2015-04-29 20:34 | dmitry_mezentsev | Note Added: 0076897 | |
2015-04-29 20:34 | dmitry_mezentsev | Status | scheduled => resolved |
2015-04-29 20:34 | dmitry_mezentsev | Fixed in Version | => 3.0PR15Q1.4 |
2015-04-29 20:34 | dmitry_mezentsev | Fixed in SCM revision | => PR15Q1.4 |
2015-04-29 20:34 | dmitry_mezentsev | Resolution | open => fixed |
2015-04-29 20:34 | dmitry_mezentsev | Review Assigned To | => eduardo_Argal |
2015-04-29 20:34 | dmitry_mezentsev | Status | resolved => closed |
Copyright © 2000 - 2009 MantisBT Group |