search

redruin1 / factorio-draftsman

A complete, well-tested, and up-to-date module to manipulate Factorio blueprint strings. Compatible with mods.
MIT License
94 stars 17 forks source link

Fix compatibility with some mods #69

Closed louga31 closed 1 year ago

louga31 commented 1 year ago

Fix compatibility with mods that uses both list and dict as recipe ingredients (For example, Space Exploration)

redruin1 commented 1 year ago

This might actually be an issue with the way that I'm creating the raw datatable in draftsman-update, since I have to convert lua tables to lists/sets/dicts, and I could be accidentally something that should be a dict to a list.

I'll check it myself over this week, but if you were to dump out data.raw from Factorio itself would it show the recipe in list format there? If so I'll merge this, if not I'll take a closer look at update.

redruin1 commented 1 year ago

Alright, after reviewing the prototypes this fix makes sense and is also more terse. Going to also leave a reminder to figure out how to distinguish between normal and expensive recipes (if they're available).