Error when removing assembler

kyranf / robotarmyfactorio

A mod to add robot troop units and perhaps associated support buildings and items to produce and control them.

MIT License

35 stars 12 forks source link

Error when removing assembler #115

Closed 0ptera closed 7 years ago

0ptera commented 7 years ago

Removing an assembler crashes at /robolib/eventhandlers.lua:342: attempt to index local 'assembler'

Seems to only happen with GLOBAL_TARGETING_TYPE = 3

kyranf commented 7 years ago

thanks, i'll look into it. Is repeatable? Just setting global type to 3 and then removing an assembler some time during play?

0ptera commented 7 years ago

For me it was very consistent with type 3 and never happened at type 1. I haven't tested type 2. Edit: writing type this often makes me realize how wrong it sounds. Shouldn't it be targeting mode?

kyranf commented 7 years ago

yes, should be mode. blame Dauphin. Anyway it turns out I already found and fixed this bug in the Unstable branch, you could grab the files from that and just replace your current mod files and you'll also get the other fixes in there. One of the fixes makes squads move away from the spawning area better without getting dragged back by new units spawning while the squad is leaving but doesn't leave fast enough..

kyranf commented 7 years ago

fixed in unstable