Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
0031070Openbravo ERPA. Platformpublic2015-10-06 17:012015-11-23 21:17
caristu 
caristu 
urgentmajoralways
closedfixed 
5
 
3.0PR16Q13.0PR16Q1 
alostale
No
Core
No
0031070: Deleting pop-up is never closed under some circumnstances
Deleting pop-up is never closed if we click on edit record and then we click on the delete button of the toolbar.
1) Go to the Sales Invoice window
2) Create a new record in the header
3) Go to the Lines tab, and create a new record
4) Still on the Lines tab, in grid view, click on the edit icon
5) Without saving the record, click directly on the delete button of the toolbar

Notice that the record is deleted but the deleting pop-up is not closed
No tags attached.
png popUpNotHidden.png (46,669) 2015-10-06 17:02
https://issues.openbravo.com/file_download.php?file_id=8529&type=bug
png
Issue History
2015-10-06 17:01caristuNew Issue
2015-10-06 17:01caristuAssigned To => platform
2015-10-06 17:01caristuOBNetwork customer => No
2015-10-06 17:01caristuModules => Core
2015-10-06 17:01caristuTriggers an Emergency Pack => No
2015-10-06 17:02caristuFile Added: popUpNotHidden.png
2015-10-06 17:02caristuIssue Monitored: networkb
2015-10-07 08:31alostaleStatusnew => acknowledged
2015-10-07 08:31alostaleTarget Version => 3.0PR16Q1
2015-11-11 11:33alostalePriorityhigh => urgent
2015-11-11 12:26caristuAssigned Toplatform => caristu
2015-11-11 12:26caristuStatusacknowledged => scheduled
2015-11-11 16:43caristuReview Assigned To => alostale
2015-11-11 16:43caristuIssue Monitored: alostale
2015-11-11 16:44hgbotCheckin
2015-11-11 16:44hgbotNote Added: 0081719
2015-11-11 16:44hgbotStatusscheduled => resolved
2015-11-11 16:44hgbotResolutionopen => fixed
2015-11-11 16:44hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/b59a2f4e4a114754328a34722d4f7c64c56ade2e [^]
2015-11-16 12:02alostaleNote Added: 0081867
2015-11-16 12:02alostaleStatusresolved => closed
2015-11-16 12:02alostaleFixed in Version => 3.0PR16Q1
2015-11-23 21:17hudsonbotCheckin
2015-11-23 21:17hudsonbotNote Added: 0082198

Notes
(0081719)
hgbot   
2015-11-11 16:44   
Repository: erp/devel/pi
Changeset: b59a2f4e4a114754328a34722d4f7c64c56ade2e
Author: Carlos Aristu <carlos.aristu <at> openbravo.com>
Date: Wed Nov 11 16:42:20 2015 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/b59a2f4e4a114754328a34722d4f7c64c56ade2e [^]

fixes issue 31070: Deleting pop-up is never closed under some circumnstances

The problem was that when removing a record that is being editing, the updateTabTitle function was crashing when trying to access to this.viewGrid.getEditForm().isNew because this.viewGrid.getEditForm() is undefined at that point.

Apart from this, the isEditingGrid is not restored to false when removing a record which is being edited. This caused new errors after removing, because the view is not marked as in non-editing state. For this reason, now the isEditingGrid value is set to false for this case also.

---
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
---
(0081867)
alostale   
2015-11-16 12:02   
code reviewed

tested:
- Deletion in grid while editing without saving
- Deletion in grid not editing
- Deletion in grid multiple record
- Deletion in form
(0082198)
hudsonbot   
2015-11-23 21:17   
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/7b56bebaaa88 [^]
Maturity status: Test