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

Bug 498287

Summary:	Ensure label parts are only determined once within TranslateSelectedAndRelocateLabelsOnDragPolicy
Product:	[Tools] GEF	Reporter:	Matthias Wienand <matthias.wienand>
Component:	GEF Zest	Assignee:	gef-inbox <gef-inbox>
Status:	RESOLVED FIXED	QA Contact:
Severity:	normal
Priority:	P3
Version:	1.0.0
Target Milestone:	4.1.0 (Neon.1) RC1
Hardware:	All
OS:	All
Whiteboard:

Description Matthias Wienand

2016-07-21 10:15:29 EDT

Currently, the TranslateSelectedAndRelocateLabelsOnDragPolicy determines the label parts that are relocated in every drag and dragAborted. This leads to NullPointerException in some cases.

Comment 1 Matthias Wienand

2016-08-09 09:05:26 EDT

I split the code, so that the label parts are determined only once during interaction. The getLabelParts() method returns the previously determined parts if available, otherwise, the parts are computed and stored. The new method computeLabelParts() is used to determine the parts. The code is published on the master branch and was ported back to R4_0_maintenance. Therefore, I resolve this ticket as fixed for 4.1.0 RC1 / 5.0.0 M1.