Project:
| View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
| ID | ||||||||
| 0035665 | ||||||||
| Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
| defect | [Openbravo ERP] A. Platform | minor | sometimes | 2017-03-31 11:40 | 2017-10-03 17:40 | |||
| Reporter | lorenzofidalgo | View Status | public | |||||
| Assigned To | caristu | |||||||
| Priority | urgent | Resolution | no change required | 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 | pi | SCM revision | ||||||
| Review Assigned To | caristu | |||||||
| Web browser | ||||||||
| Modules | Core | |||||||
| Regression level | ||||||||
| Regression date | ||||||||
| Regression introduced in release | ||||||||
| Regression introduced by commit | ||||||||
| Triggers an Emergency Pack | No | |||||||
| Summary | 0035665: Wrong behaviour filling some fields. In Financial Acc. || Transaction while filling Transaction Date, Accounting Date is deleted | |||||||
| Description | There is a strange behaviour in Financial Acc. || Transaction that involves, at least, "Transaction Date", "Accounting Date" fields. Filling those input fields is not being properly performed (Check attached video). Until now, it has been impossible to get it manually. But it is an stable error in CI and I have been able to record that regression locally. One of the failures that is being reproduced in CI is: https://ci.openbravo.com/view/qa/job/qa-pi-reconciliation-pgsql/695/ [^] It seems it is the callout what induces that strange refresh in the field where the focus is placed. The "WrongBehaviourEvenWithSleepAdded" attached videos are different automation executions with different static sleeps added to test if it is affected by performance. | |||||||
| Steps To Reproduce | Check attached video to get complete trace. Attached also an smoke test case (FRM0060_SetDateAfterFICCall.java) where the problem can be reproduced in a stable way. | |||||||
| Proposed Solution | It seems that the root of the problem is in the test infrastructure itself: for Date items, the text box is cleared before entering the value. This method is causing problems when trying to set a date value right after the FIC has been invoked. This use case happens when after editing a field that invokes a callout the focus is immediately placed on a date field. Under this circumstances, the clear method is not working fine because at the same time the blur event of the date is fired. This event is fired by smartclient when disabling the form during the FIC call to calculate the results of the callout. It must be reviewed if that clear() invokation can be safely removed. | |||||||
| Tags | No tags attached. | |||||||
| Attached Files |  TransactionWronglyFilled [^] (612,982 bytes) 2017-03-31 11:40 WrongBehaviourEvenWithSleepAdded.avi [^] (624,988 bytes) 2017-04-04 11:01 WrongBehaviourEvenWithSleepAdded2 [^] (609,926 bytes) 2017-04-04 11:01 WrongBehaviourEvenWithSleepAdded3 [^] (677,644 bytes) 2017-04-04 11:01 WrongBehaviourEvenWithSleepAdded4 [^] (688,120 bytes) 2017-04-04 11:02 WrongBehaviourEvenWithSleepAdded5 [^] (770,232 bytes) 2017-04-04 11:02 FRM0060_SetDateAfterFICCall.java [^] (4,875 bytes) 2017-09-21 19:42 | |||||||
 Relationships [ Relation Graph ]
[ Dependency Graph ]
|
||||||||
|
||||||||
 Relationships |
|
Notes |
|
|
(0095893) inigosanchez (developer) 2017-04-07 11:06 edited on: 2017-04-07 11:07 |
It seems that this part of code is related with the reported problem (strange refresh in the field where the focus is placed). It should be checked (ob-view-form.js): .......... setDisabledWhenStillInFIC: function () { if (this.inFicCall) { this.disableForm(true); } }, .......... |
|
(0099727) caristu (developer) 2017-10-03 16:24 |
Waiting for the QA team to confirm that this issue is caused by the behavior of the date items implemented in the test infrastructure itself. See [1]. [1] https://code.openbravo.com/tools/automation/pi-smoke/rev/4812b976e01a [^] |
|
(0099733) lorenzofidalgo (developer) 2017-10-03 17:22 |
I confirm that changeset [1] solves the problem. As we discussed, that malfunctioning was caused by a conflict between that "clear()" method and the callout that was being executed. [1] https://code.openbravo.com/tools/automation/pi-smoke/rev/4812b976e01a [^] |
|
(0099739) caristu (developer) 2017-10-03 17:40 edited on: 2017-10-03 17:40 |
Issue not reproducible in a real environment. This is an "artificial" issue solved with the following fix[1] in the testing infrastructure. [1] https://code.openbravo.com/tools/automation/pi-smoke/rev/4812b976e01a [^] |
|
Notes |
|
Issue History |
|||
| Date Modified | Username | Field | Change |
| 2017-03-31 11:40 | lorenzofidalgo | New Issue | |
| 2017-03-31 11:40 | lorenzofidalgo | Assigned To | => platform |
| 2017-03-31 11:40 | lorenzofidalgo | File Added: TransactionWronglyFilled | |
| 2017-03-31 11:40 | lorenzofidalgo | Modules | => Core |
| 2017-03-31 11:40 | lorenzofidalgo | Regression level | => Automated tests |
| 2017-03-31 11:40 | lorenzofidalgo | Regression introduced in release | => pi |
| 2017-03-31 11:40 | lorenzofidalgo | Triggers an Emergency Pack | => No |
| 2017-04-04 10:59 | lorenzofidalgo | Description Updated | View Revisions |
| 2017-04-04 11:01 | lorenzofidalgo | File Added: WrongBehaviourEvenWithSleepAdded.avi | |
| 2017-04-04 11:01 | lorenzofidalgo | File Added: WrongBehaviourEvenWithSleepAdded2 | |
| 2017-04-04 11:01 | lorenzofidalgo | File Added: WrongBehaviourEvenWithSleepAdded3 | |
| 2017-04-04 11:02 | lorenzofidalgo | File Added: WrongBehaviourEvenWithSleepAdded4 | |
| 2017-04-04 11:02 | lorenzofidalgo | File Added: WrongBehaviourEvenWithSleepAdded5 | |
| 2017-04-04 11:04 | lorenzofidalgo | Description Updated | View Revisions |
| 2017-04-07 09:06 | alostale | Regression level | Automated tests => |
| 2017-04-07 09:06 | alostale | Regression introduced in release | pi => |
| 2017-04-07 09:06 | alostale | Priority | immediate => high |
| 2017-04-07 09:06 | alostale | Status | new => acknowledged |
| 2017-04-07 09:06 | alostale | Relationship added | related to 0035658 |
| 2017-04-07 11:02 | inigosanchez | Steps to Reproduce Updated | View Revisions |
| 2017-04-07 11:06 | inigosanchez | Note Added: 0095893 | |
| 2017-04-07 11:07 | inigosanchez | Note Edited: 0095893 | View Revisions |
| 2017-06-06 08:46 | alostale | Priority | high => urgent |
| 2017-09-19 18:55 | caristu | Assigned To | platform => caristu |
| 2017-09-19 18:55 | caristu | Status | acknowledged => scheduled |
| 2017-09-21 19:40 | caristu | Steps to Reproduce Updated | View Revisions |
| 2017-09-21 19:40 | caristu | Proposed Solution updated | |
| 2017-09-21 19:42 | caristu | File Added: FRM0060_SetDateAfterFICCall.java | |
| 2017-09-21 19:42 | caristu | Proposed Solution updated | |
| 2017-09-21 19:42 | caristu | Steps to Reproduce Updated | View Revisions |
| 2017-10-03 16:24 | caristu | Note Added: 0099727 | |
| 2017-10-03 16:24 | caristu | Assigned To | caristu => lorenzofidalgo |
| 2017-10-03 16:24 | caristu | Status | scheduled => feedback |
| 2017-10-03 17:22 | lorenzofidalgo | Note Added: 0099733 | |
| 2017-10-03 17:23 | lorenzofidalgo | Assigned To | lorenzofidalgo => caristu |
| 2017-10-03 17:23 | lorenzofidalgo | Status | feedback => scheduled |
| 2017-10-03 17:40 | caristu | Review Assigned To | => caristu |
| 2017-10-03 17:40 | caristu | Note Added: 0099739 | |
| 2017-10-03 17:40 | caristu | Status | scheduled => closed |
| 2017-10-03 17:40 | caristu | Resolution | open => no change required |
| 2017-10-03 17:40 | caristu | Note Edited: 0099739 | View Revisions |
|
Issue History |
| Copyright © 2000 - 2009 MantisBT Group |
 |