Project:
| View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
| ID | ||||||||
| 0021352 | ||||||||
| Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
| defect | [Openbravo ERP] A. Platform | major | always | 2012-08-13 10:36 | 2012-09-28 14:38 | |||
| Reporter | jonalegriaesarte | View Status | public | |||||
| Assigned To | mtaal | |||||||
| Priority | immediate | Resolution | fixed | Fixed in Version | 3.0MP15 | |||
| Status | closed | Fix in branch | pi | Fixed in SCM revision | d500176d7771 | |||
| Projection | none | ETA | none | Target Version | 3.0MP15 | |||
| OS | Any | Database | Any | Java version | ||||
| OS Version | Database version | Ant version | ||||||
| Product Version | 3.0MP13 | SCM revision | ||||||
| Review Assigned To | marvintm | |||||||
| Web browser | ||||||||
| Modules | Core | |||||||
| Regression level | Production - Confirmed Stable | |||||||
| Regression date | 2012-05-13 | |||||||
| Regression introduced in release | ||||||||
| Regression introduced by commit | https://code.openbravo.com/erp/devel/pi/rev/8f6f2842e1f0 [^] | |||||||
| Triggers an Emergency Pack | No | |||||||
| Summary | 0021352: Enter key duplicates the line number in sales order line | |||||||
| Description | Enter key duplicates the line number in sales order line | |||||||
| Steps To Reproduce | Case 1: the product selector does not show products - Create a new sales order - Create a new line in grid view - Lines: show the "net list price" column in grid view - Select a product. The list is empty Case 2: the line number is duplicated - Create a new sales order - Create a new line in grid view - Select a product. - Lines: show the "net list price" column in grid view - Use tab key to go to "Net list price" field - Enter key - The sequence number in the new line is duplicated | |||||||
| Tags | No tags attached. | |||||||
| Attached Files | ||||||||
 Relationships [ Relation Graph ]
[ Dependency Graph ]
|
|||||||||||||||
|
|||||||||||||||
 Relationships |
|
Notes |
|
|
(0051255) AugustoMauch (manager) 2012-08-13 13:04 |
I have not been able to reproduce the issue neigher in pi, nor in main nor in demo. |
|
(0051276) jonalegriaesarte (reporter) 2012-08-14 10:16 |
Steps to reproduce updated. Notice that the problem appears if the user shows a new column in the grid, after create a new line in the grid. |
|
(0051474) AugustoMauch (manager) 2012-08-22 17:36 |
This issue is only reproducible when the user adds or removes a column while editing a new record. If the record has been saved and it is being edited again, the error will not happen. |
|
(0051505) hgbot (developer) 2012-08-23 14:28 |
Repository: erp/devel/pi Changeset: b71a7f9c8a25584de7f67f7300de135ad9a954eb Author: Augusto Mauch <augusto.mauch <at> openbravo.com> Date: Thu Aug 23 14:24:46 2012 +0200 URL: http://code.openbravo.com/erp/devel/pi/rev/b71a7f9c8a25584de7f67f7300de135ad9a954eb [^] Fixes issue 21352: Changing the shown fields while editing the grid is possible Due to a change in smartclient, it was not possible to hide or show a field in the grid while editing a row. This happened because when a field was shown or hided, the edit values of the record were being deleted. This edit values were being used to retrieve all the record values in order to send them the the FIC. This deleted values were not being restored ever. Now, to fix this, the edit values of the grid are saved when hiding or showing fields. If there is call to doChangeFICCall caused by this, the saved edit values will be used: record = isc.addProperties({}, this.viewGrid.getRecord(rowNum), this.viewGrid.getEditValues(rowNum), this.viewGrid._savedEditValues); This was only reproducible while editing new rows because saved rows had all the properties in the own record (this.viewGrid.getRecord()), so it did not matter that they were being deleted from the edit values. --- M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/grid/ob-view-grid.js M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/main/ob-standard-view.js --- |
|
(0051547) jonalegriaesarte (reporter) 2012-08-24 17:54 |
New case to reproduce the issue: Case 3: the line number is duplicated - Create a new sales order - Create a new line in grid view - Lines: show the "net list price" column in grid view - Select a product. - Use tab key to go to "Net list price" field - Change it - Enter key - The sequence number in the new line is duplicated Notice that the issue always appears when the user changes the value in the column showed dynamically. |
|
(0051549) AugustoMauch (manager) 2012-08-24 19:50 |
This happens when a line is created and the enter key is pressed after modifying a field that launches a callout, while the focus is still in that field. The line number of the new line is not initialized properly because the call to the FIC in new mode is done before saving the current record. |
|
(0051550) AugustoMauch (manager) 2012-08-24 19:56 |
If instead of pressing the Enter key the Insert Row toolbar button is pressed, the line number is properly initialized. This is the order in which the events occur in this case: - The FIC is called in CHANGE mode - The current line is saved - The FIC is called in NEW mode. |
|
(0051620) mtaal (manager) 2012-08-29 11:52 |
The solution is that opening the new line (and calling the FIC for that line) should be done after the save of the current line returns. Smartclient has a property for this: waitForSave on the listgrid but this will make grid editing slower in other cases as the next row can't be edited before the save of the previous one has been done. See the smartclient ListGrid._saveAndStartEditing method for more information. |
|
(0051628) hgbot (developer) 2012-08-29 13:05 |
Repository: erp/devel/pi Changeset: d500176d7771e75ea21f5fbe34c90e296110b8ab Author: Martin Taal <martin.taal <at> openbravo.com> Date: Wed Aug 29 13:05:11 2012 +0200 URL: http://code.openbravo.com/erp/devel/pi/rev/d500176d7771e75ea21f5fbe34c90e296110b8ab [^] Fixes issue 21352: Enter key duplicates the line number in sales order line Enable waitForSave so that save is done before initial FIC of next edit line. Make sure that the validation of number items sets the correct value. --- M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/form/formitem/ob-formitem-number.js M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/grid/ob-view-grid.js --- |
|
(0051638) marvintm (developer) 2012-08-29 16:33 |
I don't think it is working correctly yet. I do the following: - I create a sales order line - I select a product - I add a column to the grid (net list price) - I tab to the Net List Price field, and change its value. - I press enter In this case, the record is saved, and a new record is created. However, when I go to the saved record, I see that the value for the Net List Price is null, so it seems this value is not saved for some reason. |
|
(0051734) hudsonbot (developer) 2012-09-01 01:02 |
A changeset related to this issue has been promoted main and to the Central Repository, after passing a series of tests. Promotion changeset: https://code.openbravo.com/erp/devel/main/rev/05b3ea484705 [^] Maturity status: Test |
|
(0051765) mtaal (manager) 2012-09-03 13:15 |
scheduled... |
|
(0051766) mtaal (manager) 2012-09-03 13:16 |
Resolved |
|
(0051767) marvintm (developer) 2012-09-03 13:19 |
There is a separate issue, related to how the non-initially displayed fields in the grid, but the initial problem reported in this issue has been fixed: https://issues.openbravo.com/view.php?id=21533 [^] https://issues.openbravo.com/view.php?id=21530 [^] |
|
(0052279) hgbot (developer) 2012-09-21 12:36 |
Repository: erp/devel/pi Changeset: 4457b10d26e6686d742c609407d3cb2c3740760e Author: Augusto Mauch <augusto.mauch <at> openbravo.com> Date: Fri Sep 21 12:33:54 2012 +0200 URL: http://code.openbravo.com/erp/devel/pi/rev/4457b10d26e6686d742c609407d3cb2c3740760e [^] Fixes issue 21709: Tab title is updated when a record is saved in the grid view The fix of the issue 21352 caused the updateTabTitle function to be executed when the grid was still in edit mode, so the '*' in the tab title remained. Removing the code that caused this issue was not possible, because then the original issue was reproducible. Instead, an extra call to updateTabTitle is done when the records exists the edit mode. --- M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/grid/ob-view-grid.js --- |
|
(0052664) hgbot (developer) 2012-09-26 17:16 |
Repository: erp/devel/pi Changeset: a975007ee08356357dc2737b24e59c1a2e544ce0 Author: Augusto Mauch <augusto.mauch <at> openbravo.com> Date: Fri Sep 21 12:33:54 2012 +0200 URL: http://code.openbravo.com/erp/devel/pi/rev/a975007ee08356357dc2737b24e59c1a2e544ce0 [^] Fixes issue 21709: Tab title is updated when a record is saved in the grid view The fix of the issue 21352 caused the updateTabTitle function to be executed when the grid was still in edit mode, so the '*' in the tab title remained. Removing the code that caused this issue was not possible, because then the original issue was reproducible. Instead, an extra call to updateTabTitle is done when the records exists the edit mode. --- M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/grid/ob-view-grid.js --- |
|
(0052878) hudsonbot (developer) 2012-09-28 14:38 |
A changeset related to this issue has been promoted main and to the Central Repository, after passing a series of tests. Promotion changeset: https://code.openbravo.com/erp/devel/main/rev/6d2f8cffd4b1 [^] Maturity status: Test |
|
Notes |
|
Issue History |
|||
| Date Modified | Username | Field | Change |
| 2012-08-13 10:36 | jonalegriaesarte | New Issue | |
| 2012-08-13 10:36 | jonalegriaesarte | Assigned To | => alostale |
| 2012-08-13 10:36 | jonalegriaesarte | Modules | => Core |
| 2012-08-13 10:36 | jonalegriaesarte | Regression level | => Production - QA Approved |
| 2012-08-13 13:04 | AugustoMauch | Note Added: 0051255 | |
| 2012-08-13 13:04 | AugustoMauch | Status | new => feedback |
| 2012-08-14 10:16 | jonalegriaesarte | Note Added: 0051276 | |
| 2012-08-14 10:16 | jonalegriaesarte | Status | feedback => new |
| 2012-08-14 10:16 | jonalegriaesarte | Summary | Enter key duplicates the sequence number in sales order line => Enter key duplicates the line number in sales order line |
| 2012-08-14 10:16 | jonalegriaesarte | Description Updated | View Revisions |
| 2012-08-14 10:16 | jonalegriaesarte | Steps to Reproduce Updated | View Revisions |
| 2012-08-14 10:53 | AugustoMauch | Regression level | Production - QA Approved => Production - Confirmed Stable |
| 2012-08-14 10:53 | AugustoMauch | Regression date | => 2012-02-09 |
| 2012-08-14 10:53 | AugustoMauch | Regression introduced by commit | => https://issues.openbravo.com/view.php?id=19165 [^] |
| 2012-08-14 11:51 | AugustoMauch | Regression introduced by commit | https://issues.openbravo.com/view.php?id=19165 [^] => https://code.openbravo.com/erp/devel/pi/rev/634e843d6f22049c10eb8a7fc62661668e3c7669 [^] |
| 2012-08-14 13:11 | AugustoMauch | Relationship added | related to 0020722 |
| 2012-08-14 13:13 | AugustoMauch | Relationship deleted | related to 0020722 |
| 2012-08-14 14:45 | AugustoMauch | Status | new => scheduled |
| 2012-08-14 14:45 | AugustoMauch | fix_in_branch | => pi |
| 2012-08-14 14:45 | AugustoMauch | Assigned To | alostale => AugustoMauch |
| 2012-08-21 09:38 | heccam | Issue Monitored: heccam | |
| 2012-08-22 17:36 | AugustoMauch | Note Added: 0051474 | |
| 2012-08-23 14:27 | AugustoMauch | Closed by | => marvintm |
| 2012-08-23 14:27 | AugustoMauch | fix_in_branch | pi => |
| 2012-08-23 14:28 | hgbot | Checkin | |
| 2012-08-23 14:28 | hgbot | Note Added: 0051505 | |
| 2012-08-23 14:28 | hgbot | Status | scheduled => resolved |
| 2012-08-23 14:28 | hgbot | Resolution | open => fixed |
| 2012-08-23 14:28 | hgbot | Fixed in SCM revision | => http://code.openbravo.com/erp/devel/pi/rev/b71a7f9c8a25584de7f67f7300de135ad9a954eb [^] |
| 2012-08-23 14:30 | AugustoMauch | Status | resolved => new |
| 2012-08-23 14:30 | AugustoMauch | Resolution | fixed => open |
| 2012-08-23 14:33 | AugustoMauch | Regression date | 2012-02-09 => 2012-04-16 |
| 2012-08-23 14:33 | AugustoMauch | Regression introduced by commit | https://code.openbravo.com/erp/devel/pi/rev/634e843d6f22049c10eb8a7fc62661668e3c7669 [^] => https://code.openbravo.com/erp/devel/pi/rev/54d4694d51fc [^] |
| 2012-08-23 14:33 | AugustoMauch | Status | new => scheduled |
| 2012-08-23 14:33 | AugustoMauch | fix_in_branch | => pi |
| 2012-08-23 14:33 | AugustoMauch | Status | scheduled => resolved |
| 2012-08-23 14:33 | AugustoMauch | Resolution | open => fixed |
| 2012-08-24 08:09 | marvintm | Status | resolved => closed |
| 2012-08-24 08:09 | marvintm | Fixed in Version | => 3.0MP15 |
| 2012-08-24 17:54 | jonalegriaesarte | Note Added: 0051547 | |
| 2012-08-24 17:54 | jonalegriaesarte | Status | closed => new |
| 2012-08-24 17:54 | jonalegriaesarte | Resolution | fixed => open |
| 2012-08-24 17:54 | jonalegriaesarte | Fixed in Version | 3.0MP15 => |
| 2012-08-24 19:50 | AugustoMauch | Note Added: 0051549 | |
| 2012-08-24 19:56 | AugustoMauch | Note Added: 0051550 | |
| 2012-08-26 20:13 | AugustoMauch | Assigned To | AugustoMauch => marvintm |
| 2012-08-26 20:14 | AugustoMauch | Regression date | 2012-04-16 => 2012-05-13 |
| 2012-08-26 20:14 | AugustoMauch | Regression introduced by commit | https://code.openbravo.com/erp/devel/pi/rev/54d4694d51fc [^] => https://code.openbravo.com/erp/devel/pi/rev/8f6f2842e1f0 [^] |
| 2012-08-26 20:14 | AugustoMauch | fix_in_branch | pi => |
| 2012-08-29 11:16 | marvintm | Assigned To | marvintm => mtaal |
| 2012-08-29 11:52 | mtaal | Note Added: 0051620 | |
| 2012-08-29 13:05 | hgbot | Checkin | |
| 2012-08-29 13:05 | hgbot | Note Added: 0051628 | |
| 2012-08-29 13:05 | hgbot | Status | new => resolved |
| 2012-08-29 13:05 | hgbot | Resolution | open => fixed |
| 2012-08-29 13:05 | hgbot | Fixed in SCM revision | http://code.openbravo.com/erp/devel/pi/rev/b71a7f9c8a25584de7f67f7300de135ad9a954eb [^] => http://code.openbravo.com/erp/devel/pi/rev/d500176d7771e75ea21f5fbe34c90e296110b8ab [^] |
| 2012-08-29 16:33 | marvintm | Note Added: 0051638 | |
| 2012-08-29 16:33 | marvintm | Status | resolved => new |
| 2012-08-29 16:33 | marvintm | Resolution | fixed => open |
| 2012-09-01 01:02 | hudsonbot | Checkin | |
| 2012-09-01 01:02 | hudsonbot | Note Added: 0051734 | |
| 2012-09-03 13:10 | mtaal | Relationship added | related to 0021530 |
| 2012-09-03 13:15 | mtaal | Status | new => scheduled |
| 2012-09-03 13:15 | mtaal | Note Added: 0051765 | |
| 2012-09-03 13:15 | mtaal | fix_in_branch | => pi |
| 2012-09-03 13:16 | mtaal | Note Added: 0051766 | |
| 2012-09-03 13:16 | mtaal | Status | scheduled => resolved |
| 2012-09-03 13:16 | mtaal | Fixed in Version | => 3.0MP15 |
| 2012-09-03 13:16 | mtaal | Resolution | open => fixed |
| 2012-09-03 13:19 | marvintm | Note Added: 0051767 | |
| 2012-09-03 13:19 | marvintm | Status | resolved => closed |
| 2012-09-21 12:29 | AugustoMauch | Relationship added | related to 0021709 |
| 2012-09-21 12:36 | hgbot | Checkin | |
| 2012-09-21 12:36 | hgbot | Note Added: 0052279 | |
| 2012-09-26 17:16 | hgbot | Checkin | |
| 2012-09-26 17:16 | hgbot | Note Added: 0052664 | |
| 2012-09-28 14:38 | hudsonbot | Checkin | |
| 2012-09-28 14:38 | hudsonbot | Note Added: 0052878 | |
|
Issue History |
| Copyright © 2000 - 2009 MantisBT Group |
 |