Page 1 of 1
Auto-barrel doesn't take into account icons with size > 32
Posted: Wed May 22, 2019 10:08 pm
by bobingabout
Firstly, the auto-barrel (data-updates.lua creates a filled barrel item, fill and empty recipes, and assigns icons to them all) icon generation for the fill and empty recipes doesn't take into account icons with a size other than 32.
https://www.dropbox.com/s/qss8qzvvwef7h ... 4.png?dl=0
in this example screenshot, pure water is icon_size 64.
Also, if a fluid doesn't have an icon= tag (EG, it has an icons= tag), the function refuses to run on it at all.
Re: Auto-barrel doesn't take into account icons with size > 32
Posted: Thu May 23, 2019 11:56 am
by posila
Thanks for fixing it for 0.17.43
Re: Auto-barrel doesn't take into account icons with size > 32
Posted: Thu May 23, 2019 12:55 pm
by eradicator
Btw, barreling also doesn't account for fluids with localised_name like it could:
Code: Select all
localised_name = {"recipe-name.fill-barrel", fluid.localised_name or {"fluid-name." .. fluid.name}},
Re: Auto-barrel doesn't take into account icons with size > 32
Posted: Thu May 23, 2019 8:41 pm
by bobingabout
eradicator wrote: Thu May 23, 2019 12:55 pm
Btw, barreling also doesn't account for fluids with localised_name like it could:
Code: Select all
localised_name = {"recipe-name.fill-barrel", fluid.localised_name or {"fluid-name." .. fluid.name}},
because if no localised_name is specified, that will return nil and use the or condition to use the default.
simple fix.
is it localised or localized? there's a lot of American in the code.
Re: Auto-barrel doesn't take into account icons with size > 32
Posted: Thu May 23, 2019 9:45 pm
by bobingabout
eradicator wrote: Thu May 23, 2019 12:55 pm
Btw, barreling also doesn't account for fluids with localised_name like it could:
Code: Select all
localised_name = {"recipe-name.fill-barrel", fluid.localised_name or {"fluid-name." .. fluid.name}},
you should probably make a new bug report, as this one is in the resolved bin, and further conversation will likely be ignored.
Re: Auto-barrel doesn't take into account icons with size > 32
Posted: Fri May 24, 2019 11:28 am
by eradicator