Refactor of StationBankAccountComponent.Accounts

[alexalexmax]

About the PR

I replaced the Accounts and RevenueDistribution fields of StationBankAccountComponent with a single field that uses a CargoAccountData record. I updated all uses of Accounts and RevenueDistribution to use the new field.

Why / Balance

The way it was handled made it really annoying to add new fields for cargo accounts. Each field had its own dictionary mapping accounts to a value. The keys in the dictionaries were all coupled and values were hardcoded into the component.

In the future, new fields can be created by adding a new datafield to CargoAccountData and adjusting the constructor.

Technical details

Replaced the Accounts and RevenueDistribution fields of StationBankAccountComponent with a single field that uses a CargoAccountData record.
Updated all uses of Accounts and RevenueDistribution to use the new field.
Created CargoAccountData record with Balance and RevenueDistribution fields.
Added TryGetAccountBalance. It does what TryGetAccount used to do, but is named better.
TryGetAccount now gets the CargoAccountData and not the balance.

Unhardcoded default balance and distribution values and moved them into base.yml.

Media

Requirements

• I have read and am following the Pull Request and Changelog Guidelines.
• I have added media to this PR or it does not require an in-game showcase.

Breaking changes

StationBankAccountComponent.Accounts is now a dictionary mapping accounts to CargoAccountData. The values that were previously held in StationBankAccountComponent.Accounts and StationBankAccountComponent.RevenueDistribution are now stored in the CargoAccountData in StationBankAccountComponent.Accounts.

TryGetAccount now gets the CargoAccountData and not the balance. Use TryGetAccountBalance to achieve the same effect as the old TryGetAccount.

Changelog

not player facing

[alexalexmax]

requested changes made;

• CargoAccountData implements IRobustCloneable and is a class now
• CreateAccountDistribution has an ignorePrimaryAccount parameter which defaults to false
• The timed addition of funds now correctly ignores the primary account
• Cleanup unused dependencies and redundant code