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

Bug 489457

Summary:	[Composite Diagram] NPE in BehaviorPortEditPolicy when port deleted
Product:	[Modeling] Papyrus	Reporter:	Christian Damus <give.a.damus>
Component:	Diagram	Assignee:	Christian Damus <give.a.damus>
Status:	RESOLVED FIXED	QA Contact:
Severity:	major
Priority:	P2	CC:	rschnekenburger
Version:	1.1.4
Target Milestone:	SR2
Hardware:	PC
OS:	Mac OS X
See Also:	https://git.eclipse.org/r/68238 https://git.eclipse.org/c/papyrus/org.eclipse.papyrus.git/commit/?id=ec4bb1c7aa4b8be21521b33ff9d6fef874ee1dbe https://git.eclipse.org/r/68243 https://git.eclipse.org/c/papyrus/org.eclipse.papyrus.git/commit/?id=7f74f3419f36897b0a6c5f708dd7b2436eb55622
Whiteboard:
Bug Depends on:
Bug Blocks:	483637

Description Christian Damus

2016-03-11 14:19:15 EST

Found in analysis of Papyrus-RT bug 483637 on the latest Mars maintenance release.

The BehaviorPortEditPolicy listens to ports in a Composite Diagram to update their visual presentation when their behaviour nature changes.  However, if the port in question is also being removed from the diagram (which happens in an in-development enhancement to synchronization in Capsule Diagrams), then an NPE results in the getPossibleCompartment(EditPart) method when the notifyChange(Notification) method triggers an update via the udaptePortBehavior [sic] method.

It is difficult to provide steps to reproduce, as they depend on code changes that are pending in the Papyrus-RT project that can't actually be delivered until this blocking NPE is fixed.

Comment 1 Eclipse Genie

2016-03-11 14:34:38 EST

New Gerrit change created: https://git.eclipse.org/r/68238

Comment 2 Eclipse Genie

2016-03-11 15:09:45 EST

Gerrit change https://git.eclipse.org/r/68238 was merged to [streams/1.1-maintenance].
Commit: http://git.eclipse.org/c/papyrus/org.eclipse.papyrus.git/commit/?id=ec4bb1c7aa4b8be21521b33ff9d6fef874ee1dbe

Comment 3 Eclipse Genie

2016-03-11 15:10:17 EST

New Gerrit change created: https://git.eclipse.org/r/68243

Comment 4 Eclipse Genie

2016-03-11 16:25:16 EST

Gerrit change https://git.eclipse.org/r/68243 was merged to [master].
Commit: http://git.eclipse.org/c/papyrus/org.eclipse.papyrus.git/commit/?id=7f74f3419f36897b0a6c5f708dd7b2436eb55622

Comment 5 Christian Damus

2016-03-11 16:25:46 EST

Fixed on Neon (master) and Mars branches.