Anonymous | Login
Project:
RSS
  
News | My View | View Issues | Roadmap | Summary

View Issue DetailsJump to Notes ] Issue History ] Print ]
ID
0018150
TypeCategorySeverityReproducibilityDate SubmittedLast Update
defect[Openbravo ERP] A. Platformminorhave not tried2011-08-01 16:172011-08-08 09:13
ReportermtaalView Statuspublic 
Assigned Tomtaal 
PrioritynormalResolutionfixedFixed in Version
StatusclosedFix in branchFixed in SCM revision19135e59b532
ProjectionnoneETAnoneTarget Version3.0MP2
OSAnyDatabaseAnyJava version
OS VersionDatabase versionAnt version
Product Version3.0MP1SCM revision 
Review Assigned To
Web browser
ModulesCore
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary

0018150: Destroy context menu when a listgrid is destroyed

DescriptionAccording 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.
TagsNo 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
Powered by Mantis Bugtracker