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