]> gerrit.simantics Code Review - simantics/platform.git/commit
Fix NPE from flagTransform 98/4398/1
authorTuukka Lehtonen <tuukka.lehtonen@semantum.fi>
Fri, 21 Aug 2020 12:29:39 +0000 (15:29 +0300)
committerTuukka Lehtonen <tuukka.lehtonen@semantum.fi>
Fri, 21 Aug 2020 13:00:09 +0000 (16:00 +0300)
commit2526602db10c29f0c30c4a3cffaf33012d9c4a0e
treea0859fac165e100a27673f3996d13cd8460839ae
parent15a025f0981f33f15d39a3d2de3b0f9bf7a18c79
Fix NPE from flagTransform

The bug affected loading of route graph connections about to be split in
half when attached to diagram flags at either or both ends.

The NPE caused the flag terminal position to receive identity transform
as value which is plain wrong. This in turn resulted in the splitting
position being completely wrong compared to what the user requested and
the result looking very strange.

This fixes the immediate problem but still doesn't work properly if the
flag is actually attached to a diagram template flag table. In this case
the terminal's position will be incorrectly calculated since
runtimeDiagram is null. This needs another issue.

gitlab #586

Change-Id: If09524c71c701f2c1190d0f915aafbb972c51da4
bundles/org.simantics.diagram/src/org/simantics/diagram/function/All.java