search

BHoM / MidasCivil_Toolkit

GNU Lesser General Public License v3.0
7 stars 4 forks source link

Fixed bug where SaveAs command caused an error #366

Closed KalleEdstroem closed 1 year ago

KalleEdstroem commented 1 year ago

The copying of the .mcb and .mct file when using the SaveAs command did not work. It used the wrong input for the File.Copy(string, string). The second string pointed to a directory rather than a destination file.

This made it so the .mcb and .mct file was not copied. The issue also resulted in the issue stated in #289 which this closes.

Issues addressed by this PR

Closes #289

Test files

Please use the test script below and run the execute workflow. Before testing https://burohappold.sharepoint.com/sites/BHoM/02_Current/Forms/AllItems.aspx?csf=1&web=1&e=bdWLDi&cid=c70586d3%2Df382%2D473f%2Dab89%2Db962fae9a4aa&RootFolder=%2Fsites%2FBHoM%2F02%5FCurrent%2F12%5FScripts%2F01%5FIssue%2FBHoM%2FMidasCivil%5FToolkit%2F%23289%2DSaveAsNotWorking&FolderCTID=0x0120008122C8891F89054B8ACED0196C70DFC4

peterjamesnugent commented 1 year ago

@BHoMBot check required

bhombot-ci[bot] commented 1 year ago
@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` There are 3 requests in the queue ahead of you.
peterjamesnugent commented 1 year ago

@BHoMBot check copyright-compliance

bhombot-ci[bot] commented 1 year ago
@peterjamesnugent to confirm, the following actions are now queued: - check `copyright-compliance` There are 7 requests in the queue ahead of you.
FraserGreenroyd commented 1 year ago

@BHoMBot check ready-to-merge

bhombot-ci[bot] commented 1 year ago
@FraserGreenroyd to confirm, the following actions are now queued: - check `ready-to-merge`