Corporea string item loading is done client-side#4249
Corporea string item loading is done client-side#4249gamma-delta wants to merge 7 commits intoVazkiiMods:1.19.xfrom
Conversation
|
whoops I very cleverly put backticks in the last commit name so it tried to interpret them as commands. it should say "make matching |
|
Say I have an iron ingot in the system called (I'll run spotless at the end) |
|
One problem I thought of: it won't work nicely with items that have a different display name per stack irregardless of the display name, like hexcasting scrolls... |
Yes, i'd say this is a goal. Currently not feeling very well but if i get better in the next few days i can test whether it works currently
Can you rephrase? I don't get what you mean
I think you already got the answer to this but, for posterity: No, it shouldn't return the renamed item. |
Consider an item that changes its display name by its NBT, such as a fluid tank that may be named "Empty Tank" or "Tank of {X liquid}" |
|
Can anything from this PR be salvaged? |
|
Even today, in all non-English language servers, Corporea index still can't recognize localized item names. It seems like this problem will take a long time to fix, or maybe it has been permanently put on hold XD |
|
It's not a particularly simple change, and it will very likely not happen in 1.20.1 anymore. |
This closes #4248 and also has some other nice effects:
stack ofand such are still Englishy)thisnow uses aCorporeaItemStackMatcherinstead of searching for the name of the itemI've done some tests and the following works:
64 stonewith nothing in the system64 kiwenin Toki Pona gives you the stone after switching back to EnglishTodo: