KeinNiemand / Factorio-GhostOnWater

Factorio mod to make it possible to paste blueprints on water in one go
https://mods.factorio.com/mod/GhostOnWater
MIT License
1 stars 8 forks source link

Update all blueprints in books #23

Closed spenczar closed 1 year ago

spenczar commented 1 year ago

This changes behavior for blueprint books. With this change, if a blueprint book is held in the cursor, all blueprints in the book are updated. This is done recursively, updating any books within the book, too.

This was requested in https://mods.factorio.com/mod/GhostOnWater/discussion/636127bf321a67b094934dfb, and I agree with the request. I have a large blueprint book of train blueprints, and I'd like to update all of them in one pass. I have another book of hundreds of belt balancers - and likewise I want that to be updated in one go. Writing this PR was honestly quicker than doing each update one-by-one!

I haven't updated the changelog or any documentation; let me know whether you'd like me to do that or you prefer to do it yourself.

KeinNiemand commented 1 year ago

While I don't have anything against updating all BP in BP books I'm not sure if this is something everyone would always want. so maybe add a mod setting to turn recursively update BP books on or off (probably on by default). Not sure if that setting is nearly nectary tough. I'd be happy if you updated the changelog and maybe the readme (which is basically the same thing as the mod page), as this adds a feature I'd be version 0.11.0 , I'll change the date in the changelog when I release it. I won't say to to having to to less work myself, If you don't want to do it I will probably do it myself eventually but that would talke longer as I'm currently not activly working on this mod.

spenczar commented 1 year ago

Sure @KeinNiemand. I have put this behavior behind a setting which defaults to true and updated the readme and changelog.