Fix bug where SectionProperty ids were not sequential for multiple pushes

[peterjamesnugent]

Issues addressed by this PR

Closes #359

Test files

https://burohappold.sharepoint.com/:u:/r/sites/BHoM/02_Current/12_Scripts/01_Issue/BHoM/MidasCivil_Toolkit/%23359-SectionPropertyIdBug.gh?csf=1&web=1&e=iWCRtf

Changelog

• Fixed bug where SectionProperty objects did not have sequential ids when using multiple push components, leading to errors when trying to read them in to a Dictionary.

Additional comments

[KalleEdstroem]

Tested with testscript (Grasshopper - BuroHappold_BHoM_v.6.3.beta_Create read loads.gh)
Works as expected. Everything pushed and no errors

[peterjamesnugent]

@BHoMBot check required
@BHoMBot check copyright-compliance

[bhombot-ci]

@peterjamesnugent 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
• check copyright-compliance

There are 5 requests in the queue ahead of you.

[peterjamesnugent]

@BHoMBot check installer
@BHoMBot check versioning

[bhombot-ci]

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

• check installer
• check versioning

There are 2 requests in the queue ahead of you.

[peterjamesnugent]

@BHoMBot check ready-to-merge

[bhombot-ci]

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

• check ready-to-merge