new-frontiers-14 / frontier-station-14

A multiplayer game about paranoia and chaos on a space station. Remake of the cult-classic Space Station 13.
https://spacestation14.io
GNU Affero General Public License v3.0
79 stars 463 forks source link

Computer YAML cleanup #1723

Closed whatston3 closed 1 month ago

whatston3 commented 1 month ago

About the PR

Fixes a lot of strange parenting and redundancy in computers. All computers should now inherit from BaseComputer (a computer) instead of BaseStructureComputer (the frame), and many fewer things need to be specified.

ComputerCargoOrders was restored to its upstream component set before the june merge.

Capitalized "bank ATM" and changed wording to "withdraw-only bank ATM". Resolved ATM inheritance order.

Why / Balance

Less object-specific behaviour to maintain, better parenting, cleaner YAML.

I noticed that many of the consoles inheriting from weren't lit up correctly, e.g. the contraband pallet computer, and this stemmed from the inheritance from BaseStructureComputer and the redefinition of fields.

How to test

  1. Find (or spawn) a wallmount ATM (in a wall).
  2. Interact with it.
  3. Remove cash.
  4. Deposit cash.
  5. Go to the station admin console in the SR's office.
  6. Remove cash.
  7. Deposit cash (ensure you have a valid reason and the amount matches what's in the cash slot)
  8. Warp to the Cargo Depot.
  9. Spawn a stack of plasma.
  10. Sell it, it should sell for $954.
  11. Warp to the Trade Outpost.
  12. Check a cargo request computer.
  13. It should be lit up.
  14. Purchase something, it should spawn as normal.
  15. Warp to the NFSD Outpost.
  16. Go to the contraband console.
  17. Spawn a pair of syndie pyjamas on the contraband pallets.
  18. Sell it for 1 FUC.

Media

This shows the lit up state of various consoles. These are all inherited from BaseComputer (apart from the colour of the PointLight on the cargo consoles). image

Breaking changes

Changelog

Mostly internal, no CL.