Duplicate Path Bug Fix

[payneBrandon]

Moving JsonProperty declaration to getters to avoid duplicate fetch in serialization/deserialization scenarios

PR Details

Description

This PR moves the "@JsonProperty" declaration in the NodeSetXY and Nodes classes to their respective getters rather than the private property declaration. For some reason, having those on the private property exposed that private property when serializing/deserializing and as a result both the private property and the public getter method were called - causing paths to come out as duplicated.

Related Issue

Motivation and Context

Solves problem described in description

How Has This Been Tested?

Tested locally, and in WYDOT test environment

Types of changes

• Defect fix (non-breaking change that fixes an issue)
• New feature (non-breaking change that adds functionality)
• Breaking change (fix or feature that cause existing functionality to change)

Checklist:

• I have added any new packages to the sonar-scanner.properties file
• My change requires a change to the documentation.
• I have updated the documentation accordingly.
• I have read the CONTRIBUTING document.
  ODE Contributing Guide
• I have added tests to cover my changes.
• All new and existing tests passed.

[dmccoystephenson]

This looks good to me!

This was tested in my local environment and in WYDOT's environment. Without the fix, we were getting duplicate NodeXY nodes and with the fix, we are no longer getting these duplicates.

With and without the fix, the same number of NodeLL nodes get generated, suggesting that this duplication problem was not affecting TIMs with offsets.