Skip to content

Conversation

@BenjaminAmos
Copy link
Contributor

@BenjaminAmos BenjaminAmos commented Feb 12, 2022

Description

This pull request fixes some of the issues with code modules currently by exposing the new ECS classes as API classes, as well as the GameOptions class, which was omitted previously. It also fixes DestinationSol/warp#3 by wrapping the SaveManager write methods with AccessController::doPrivileged. The save read methods have not been wrapped, since having modules re-load the current world might be considered too destructive.

Testing

  • The API classes should now be accessible from modules. This can't be tested directly with the current line-up but after this change warp might start using the GameOptions class
  • Use the ability of the Warp Endeavour ship and ensure that the crash in Warp Endeavour ability causes game crash DestinationSol/warp#3 does not occur.

Notes

ECS System classes have not been included as API classes, since I believe that the intention was for communication between systems to occur via components and events, rather than directly. I am open to changing this though.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Warp Endeavour ability causes game crash

1 participant