Investigation needed on apparently unnecessary if condition

[So-Fras]

• Do you want to request a feature or report a bug?
  bug / investigation needed

• What is the current behavior?
  In SLD, in the PositionFromExtension.java > gatherLayoutExtensionInformation () function, a test on the direction value is added before setting the bus cell direction:

if (direction != Direction.UNDEFINED) {
    bc.setDirection(direction);
}

If this condition is removed, some unit tests fail (there are errors in the drawings of intern cells).
However, this condition should not be necessary as the UNDEFINED value is the default value for direction.

• If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem
  In order to reproduce the bug, remove the if condition

• What is the expected behavior?
  The expected behavior is that the "if condition" is not necessary. Some investigations are needed here to understand if there is an underlying bug.

• What is the motivation / use case for changing the behavior?
  Cleaner code

• Please tell us about your environment:

  • PowSyBl Version: 3.0.0