petrify( entity, mode ) about factorio-stdlib HOT 6 CLOSED

Sure, I bet there are a lot of similar low-hanging fruit for entities.

from factorio-stdlib.

@aubergine10 thoughts on whether petrify should include minable and destructible as fields to set too?

from factorio-stdlib.

Hmm.. I feel they are two distinct use cases:

• "Don't let player change entity", and/or
• "Don't let player remove entity"

There is, however, obvious cases where there will be overlap. So I'm not really sure what best approach is here.

from factorio-stdlib.

So maybe a set_indestructible as well? Or is that just going to be extra code for little purpose, given it is only two fields to set.

from factorio-stdlib.

It'll make code more semantic, not to mention the error checking that's bundled with stdlib functions, so yes IMO it's worth adding.

from factorio-stdlib.

Alright, I've implemented Entity.set_industructible as well.

from factorio-stdlib.