search

doadin / Baggins

zlib License
6 stars 7 forks source link

[Bug] Cloth categorized as other #35

Closed doadin closed 3 years ago

doadin commented 4 years ago

There seems to be a error with the cloth category at least with classic, cloth such as silk cloth show as "other". Changing category > Cloth > Item Subtype to Trade Goods instead of blank seems work fine.

@thohell tagging you in case of any other instances like this I might have missed.

toneomgomg commented 4 years ago

I'll take a look at this next week. Full time daddy this week :D

doadin commented 4 years ago

Possibly related https://github.com/doadin/Baggins/blob/master/Baggins-Options.lua#L1173 also https://github.com/doadin/Baggins/pull/37/files#diff-e624a9d636de7c6f62af15683a9cba19R34 There may also be more points of interset idk, as you know the code could use quite a bit of cleaning.

doadin commented 3 years ago

cloth isn't a subtype on classic. removed that default category since it doesn't make sense along with leather.