BHoM_Engine: Replaced C:\ProgramData with SpecialFolder construct

[albinber]

…clude the new method instead

NOTE: Depends on

Issues addressed by this PR

Closes #3219

Replaced C:\ProgramData with methods using SpecialFolder construct.

Test files

Changelog

Additional comments

[FraserGreenroyd]

Can you add the /***************************************************/ between each method to break them up 😄

[FraserGreenroyd]

Looking good @albinber , this will help with our migration from a fixed C: to a location anyone can use 😄

Can you add descriptions to all the methods for documentation purposes. For example, for the Upgrades method, it would be useful to state something like Returns the folder path for the top level upgrades directory, which houses sub-directories for upgraders for different BHoM versions to make it clear to users and developers that it only links to the top-level and nothing else (so they'll need to Path.Combine to get into sub-directories/folders) 😄

[FraserGreenroyd]

I've run through in Grasshopper checking the output is correct for my machine and it all looks good, good work @albinber 😄

[FraserGreenroyd]

@BHoMBot check compliance
@BHoMBot check required

[bhombot-ci]

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

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

[FraserGreenroyd]

@BHoMBot this is a DevOps instruction. I am requesting neutral checks on: unit-tests

[bhombot-ci]

@FraserGreenroyd I have provided neutral checks to the checks requested. These checks will need to be run properly to obtain full results.

[FraserGreenroyd]

@BHoMBot check ready-to-merge

[bhombot-ci]

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

• check ready-to-merge