Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 346842 - [Table] CopyHandler and DeleteHandler register multipleListener on the EMF Facet table
Summary: [Table] CopyHandler and DeleteHandler register multipleListener on the EMF Fa...
Status: NEW
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: EMF-Facet (show other bugs)
Version: unspecified   Edit
Hardware: All All
: P5 trivial (vote)
Target Milestone: ---   Edit
Assignee: Project Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-05-23 05:47 EDT by Nicolas Guyomar CLA
Modified: 2020-05-01 11:27 EDT (History)
1 user (show)

See Also:
gdupe: juno+

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Nicolas Guyomar CLA 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 CLA 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