satisfactory-dev / Satisfactory-Production-Calculator-UI

Issue Tracker for SignpostMarv's Satisfactory Production Planner UI
https://u8.satisfactory.dev/planner/
1 stars 0 forks source link

deleting output destinations can trigger error #56

Closed SignpostMarv closed 2 months ago

SignpostMarv commented 2 months ago

production flinging is intended to operate on a principle of using non-empty arrays, such that empty arrays would trigger production flinging to be ignored for a given pool.

currently, one can set up production flinging in such a way that deletion of destinations has the empty destination array being used when the Distributor instance should have been disposed of.

SignpostMarv commented 2 months ago

difficult to replicate this today :/

SignpostMarv commented 2 months ago

replication

steps seem to the thus:

  1. open new session
  2. add 1 new pool
  3. add 1 steel ingot to first pool
  4. open production flinging
  5. fling 1 steel ingot to second pool
  6. remove fling entry
  7. add 1 steel beam to second pool

Expected

No errors

Actual

Error in Distributor.destinations