Reflection_Engine: StackOverflowException in PropertyFullNameValueGroups fixed

[pawelbaran]

Issues addressed by this PR

Closes #3481

Test files

See #3481

Changelog

Additional comments

[adecler]

Approving based on code review and limited testing in Grasshopper.

Worth noting that PropertyFullNameValueDictionary component is returning null while we should expect a dictionary. It doesn't seems to be related to this PR so still approving but raised a separate issue here: #3483

[adecler]

@BHoMBot check compliance
@BHoMBot check required
@BHoMBot check unit-tests

[bhombot-ci]

@adecler to confirm, the following actions are now queued:

• check code-compliance
• check documentation-compliance
• check project-compliance
• check branch-compliance
• check dataset-compliance
• check copyright-compliance
• check code-compliance
• check documentation-compliance
• check project-compliance
• check core
• check null-handling
• check serialisation
• check versioning
• check installer
• check unit-tests

[bhombot-ci]

The check code-compliance has already been run previously and recorded as a successful check. This check has not been run again at this time.

[bhombot-ci]

The check documentation-compliance has already been run previously and recorded as a successful check. This check has not been run again at this time.

[adecler]

@BHoMBot check ready-to-merge

[bhombot-ci]

@adecler to confirm, the following actions are now queued:

• check ready-to-merge