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

Bug 486498

Summary:	Make binding of IHandlePartFactory and IFeedbackPartFactory optional.
Product:	[Tools] GEF	Reporter:	Matthias Wienand <matthias.wienand>
Component:	GEF MVC	Assignee:	gef-inbox <gef-inbox>
Status:	RESOLVED FIXED	QA Contact:
Severity:	normal
Priority:	P3
Version:	0.2.0
Target Milestone:	4.0.0 / 3.11.0 (Neon) M5
Hardware:	All
OS:	All
Whiteboard:

Description Matthias Wienand

2016-01-25 12:15:26 EST

If the bindings for IHandlePartFactory or IFeedbackPartFactory are missing, no handles or feedback should be generated, but the application should run nonetheless.

Comment 1 Matthias Wienand

2016-01-26 11:05:31 EST

I removed the injections of IFeedbackPartFactory and IHandlePartFactory from AbstractBehavior and moved the injections into the behaviors that need to access them:

- HoverBehavior: inject IFeedbackPartFactory.
- SelectionBehavior: inject IFeedbackPartFactory and IHandlePartFactory.
- FXHoverBehavior: inject IHandlePartFactory

Therefore, binding of both factories is only necessary if the HoverBehavior or SelectionBehavior is used. The code is published on the master branch, therefore, I resolve this ticket as fixed for 3.11.0M5.