Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 346842

Summary:	[Table] CopyHandler and DeleteHandler register multipleListener on the EMF Facet table
Product:	z_Archived	Reporter:	Nicolas Guyomar <nicolas.guyomar>
Component:	EMF-Facet	Assignee:	Project Inbox <emft.facet-inbox>
Status:	NEW ---	QA Contact:
Severity:	trivial
Priority:	P5	CC:	gdupe
Version:	unspecified	Flags:	gdupe: juno+
Target Milestone:	---
Hardware:	All
OS:	All
Whiteboard:

Description Nicolas Guyomar

2011-05-23 05:47:36 EDT

Hi,

Each time a right click is performed in the table, a new selection listener is added to the table while only one is necessary.

This bug has to be fixed in CopyHandler and DeleteHandler

Regards,
Nicolas Guyomar

Comment 1 Nicolas Guyomar

2011-05-23 05:49:57 EDT

I would add taht those listeners are called in : 
org.eclipse.emf.facet.widgets.nattable.internal.NatTableWidget.tableSelectionChanged()

The result of the getSelection() method call should be stored in a local variable before notifying every listener for performance purpose.

Regards,
Nicolas Guyomar