sirenfal-factorio / Free_Real_Estate

A faster alternative to Factorissimo with more features, from scratch.
2 stars 4 forks source link

Mining factories with players in them #1

Closed Adilf closed 7 years ago

Adilf commented 7 years ago

Mining a factory containing online player results in

__Free_Real_Estate__/factory/factory.lua:477: attempt to index global 'p' (a nil value)
3943.397 Error ServerMultiplayerManager.cpp:93: MultiplayerManager failed: "Error while running event on_preplayer_mined_item (ID 11)
__Free_Real_Estate__/factory/factory.lua:477: attempt to index global 'p' (a nil value)"

Mining a factory with offline player traps him inside. Placement of a factory containing a player that has just reconnected results in single desync.

avosirenfal commented 7 years ago

This should be fixed in 1.0.1. I didn't test it with recursion because I'm tired, but it seems to work fine without recursion enabled.

Thanks for the report, let me know if you find anything else.

Download: https://github.com/sirenfal-factorio/Free_Real_Estate/releases/download/1.0.1/Free_Real_Estate_1.0.1.zip