Add Try-catch for Framing element design properties ensuring Bar pull does not crash

[Martian42]

Try catch is added around the if statement that causes the pulling method to crash in Grasshopper when pulling Robot model.

It is encouraged to review and update FramingElementDesignProperties in the future.

NOTE: Depends on

Issues addressed by this PR

Closes #

Test files

Changelog

Additional comments

See issue in LINK

[IsakNaslundBh]

Would put the try catch around all the contents of this for loop, not just this part, to make sure we do not end up with a similar situation for other framing properties. This is, from line 45 all the way down to line 155.

[IsakNaslundBh]

Or, actually, at least covering all that are trying to extract the properties. This is from current position down to line 148

[Martian42]

Or, actually, at least covering all that are trying to extract the properties. This is from current position down to line 148

@IsakNaslundBh try and catch has been updated as suggested.

[IsakNaslundBh]

LGTM as an initial fix to make sure things does not crash.

For #486 to be resolved properly in upcoming PRs.

[IsakNaslundBh]

@BHoMBot check compliance
@BHoMBot check required

[bhombot-ci]

@IsakNaslundBh to confirm, the following checks are now queued:

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

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

[IsakNaslundBh]

@BHoMBot check ready-to-merge

[bhombot-ci]

@IsakNaslundBh to confirm, the following checks are now queued:

• ready-to-merge

There are 11 requests in the queue ahead of you.