Issue 5736: Preventing Negative Personnel Income

[psikomonkie]

Fixes #5736

An alternative to #5741

• Added a check to ensure end-of-contract Shares payouts are not made if they would be negative. This would result in the personnel paying the campaign.
• Maintained the TotalEarnings field, and modified the PayPersonnel method to only allow positive values. If someone wants to add a feature that charges Personnel later, I'd love to see it added through a new method to prevent accidents.
• Shuffled around when and how we pay out salaries so we shouldn't have to iterate through all our personnel as many times when paying them

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 10.20%. Comparing base (fd739ba) to head (5e9e2e7).
Report is 46 commits behind head on master.

Additional details and impacted files

@@             Coverage Diff              @@
##             master    #5752      +/-   ##
============================================
- Coverage     10.21%   10.20%   -0.02%     
+ Complexity     6076     6069       -7     
============================================
  Files          1010     1010              
  Lines        139310   139327      +17     
  Branches      20448    20447       -1     
============================================
- Hits          14231    14217      -14     
- Misses       123703   123733      +30     
- Partials       1376     1377       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.