Project:
View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
ID | ||||||||
0018150 | ||||||||
Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
defect | [Openbravo ERP] A. Platform | minor | have not tried | 2011-08-01 16:17 | 2011-08-08 09:13 | |||
Reporter | mtaal | View Status | public | |||||
Assigned To | mtaal | |||||||
Priority | normal | Resolution | fixed | Fixed in Version | ||||
Status | closed | Fix in branch | Fixed in SCM revision | 19135e59b532 | ||||
Projection | none | ETA | none | Target Version | 3.0MP2 | |||
OS | Any | Database | Any | Java version | ||||
OS Version | Database version | Ant version | ||||||
Product Version | 3.0MP1 | SCM revision | ||||||
Review Assigned To | ||||||||
Web browser | ||||||||
Modules | Core | |||||||
Regression level | ||||||||
Regression date | ||||||||
Regression introduced in release | ||||||||
Regression introduced by commit | ||||||||
Triggers an Emergency Pack | No | |||||||
Summary | 0018150: Destroy context menu when a listgrid is destroyed | |||||||
Description | According to the SC documentation the contextMenu property of a canvas is not destroyed when a canvas is destroyed. This because context menus can be shared. We don't share context menus, so code should be added to explicitly remove context menus to prevent memory leaks. | |||||||
Steps To Reproduce | . | |||||||
Tags | No tags attached. | |||||||
Attached Files | ||||||||
Relationships [ Relation Graph ] [ Dependency Graph ] | |
Notes | |
(0039747) hgbot (developer) 2011-08-01 16:58 |
Repository: erp/devel/pi Changeset: 19135e59b53249b9a0b351052664ec910889a828 Author: Martin Taal <martin.taal <at> openbravo.com> Date: Mon Aug 01 16:57:41 2011 +0200 URL: http://code.openbravo.com/erp/devel/pi/rev/19135e59b53249b9a0b351052664ec910889a828 [^] Fixes issue 18150: Destroy context menu when a listgrid is destroyed Added explicit destroy of context menu when a grid is destroyed --- M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/grid/ob-view-grid.js --- |
(0039925) hudsonbot (developer) 2011-08-06 05:32 |
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/5744448753e7 [^] Maturity status: Test |
(0039980) iperdomo (reporter) 2011-08-08 09:13 |
Tested on pi @ rev cd93116382e9 ---- Using Chrome Dev Tools, checking the value of the contextMenu in a Grid isc_OBStandardView_0.viewGrid _2 isc_OBStandardView_0.viewGrid.ID "isc_OBViewGrid_8" isc_OBStandardView_0.viewGrid.contextMenu _2 isc_OBStandardView_0.viewGrid.contextMenu.ID "isc_Menu_4" isc_Menu_4 _2 isc_Menu_4 === null false isc_Menu_4 === null true |
Issue History | |||
Date Modified | Username | Field | Change |
2011-08-01 16:17 | mtaal | New Issue | |
2011-08-01 16:17 | mtaal | Assigned To | => mtaal |
2011-08-01 16:17 | mtaal | Modules | => Core |
2011-08-01 16:58 | hgbot | Checkin | |
2011-08-01 16:58 | hgbot | Note Added: 0039747 | |
2011-08-01 16:58 | hgbot | Status | new => resolved |
2011-08-01 16:58 | hgbot | Resolution | open => fixed |
2011-08-01 16:58 | hgbot | Fixed in SCM revision | => http://code.openbravo.com/erp/devel/pi/rev/19135e59b53249b9a0b351052664ec910889a828 [^] |
2011-08-06 05:32 | hudsonbot | Checkin | |
2011-08-06 05:32 | hudsonbot | Note Added: 0039925 | |
2011-08-08 09:13 | iperdomo | Note Added: 0039980 | |
2011-08-08 09:13 | iperdomo | Status | resolved => closed |
Copyright © 2000 - 2009 MantisBT Group |