Added override to Delete method in toolkit.

[andrbak]

… of elements by id.

Issues addressed by this PR

Closes #274

Added override to Delete method in toolkit. Thereby allowing deletion of elements by id.

Test files

Changelog

Additional comments

[IsakNaslundBh]

sorry, @andrbak , had missed this as I do not personally have access to SAP2000. For the same reason, I am unable to test this.

@mchaf @enarhi would you be able to assist?

[enarhi]

@BHoMBot check compliance

[bhombot-ci]

@enarhi 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

There are 1 requests in the queue ahead of you.

[IsakNaslundBh]

@enarhi @mchaf From a code perspective, I think the changes in here are fine.

They align with what is being done in https://github.com/BHoM/BHoM_Adapter/blob/583860a32c114e00e5b1f717058bbd634ed7b1ab/BHoM_Adapter/CRUD/IRead.cs#L75-L94

Long terms we need a filter with an explicit list of ObejctIds, rather than reying on the equalities, but that is out of scope for this PR.

If the functionality of the PR works, according to your testing, I am happy for this to be merged in.

[enarhi]

Tested using this Test File, delete worked as expected, removing elements with matching type and objectId from the model in SAP2000.

[FraserGreenroyd]

@BHoMBot check required

[bhombot-ci]

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

• check code-compliance
• check documentation-compliance
• check project-compliance
• check core
• check null-handling
• check serialisation
• check versioning
• check installer

[FraserGreenroyd]

@BHoMBot check ready-to-merge

[bhombot-ci]

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

• check ready-to-merge