Versioning_Engine: Fix Issue with all dictionary entries throwing and catching exceptions

[IsakNaslundBh]

Issues addressed by this PR

Closes #3517

Fixes problem with exceptions being thrown and caught for each entry in a non-string based dictionary going through the upgrading loop.

Could have fixed by checking if the document contains "_t" in the UpgradeObejct method, but this almost felt clearer in terms of data flow, as the UpgradeObejctProperties is the only method in UpgradeObejct that will be called for a non-existing _t (all other parts relies on the "oldType" property.

This PR should not really change anything it terms of functionality, just make it run quicker, and hopefully make the code flow cleaner

Test files

Changelog

Additional comments

[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]

Happy with the code changes and with all checks are passing

[adecler]

@BHoMBot check ready-to-merge

[bhombot-ci]

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

• check ready-to-merge