(@Violine1101) This template does not appear to sync properly. Today I added a note to Wandering Trader (Special:Diff/3234030), but I had to save a null edit at Trading to update the table. --MinecraftExp123(talk|contribs) 03:09, 26 October 2025 (UTC)
- That's completely normal and affects all templates using bucket or similar. These changes need some time to take effect throughout the wiki (which can be sped up by doing a null edit). | violine1101 (talk) 12:47, 26 October 2025 (UTC)
- Does that mean there is a cache, and null-editing just purges it? I guess that makes sense. --MinecraftExp123(talk|contribs) 12:48, 26 October 2025 (UTC)
- Kind of, yeah. Though the "cache" should get purged automatically as well, but that might take a bit of time. | violine1101 (talk) 12:50, 26 October 2025 (UTC)
- Okay. --MinecraftExp123(talk|contribs) 12:51, 26 October 2025 (UTC)
(@Violine1101) The price multiplier still doesn't transclude properly, and also sometimes the probability displayed is incorrect, e.g. compare Trading#Shepherd and Shepherd#Trading. This likely is because it doesn't read the {{{weight}}} parameter from the TradeLine properly, or something like that. --MinecraftExp123(talk|contribs) 01:55, 5 November 2025 (UTC)
- Actually the probability right now just reads the Bedrock Edition probability, and ignores the Java Edition one, meaning all Java Edition probabilities on Trading are currently incorrect. --MinecraftExp123(talk|contribs) 01:57, 5 November 2025 (UTC)
- Will fix the probability ASAP, but will take care of implementing the multiplier a bit later, that's a bit more effort required | violine1101 (talk) 08:44, 5 November 2025 (UTC)
- I think the probability was fixed. It was just caused by some edits attempt to fix something else on Module:TradeTable, where those changes are reverted now. --MinecraftExp123(talk|contribs) 08:46, 5 November 2025 (UTC)
- Yup, just saw as well - that was unrelated | violine1101 (talk) 08:49, 5 November 2025 (UTC)
- @Violine1101 Can you get the multiplier to work (as it's been a month)? It's kinda confusing that it's not mentioned at Trading. ‑‑MinecraftExp123(talk|contribs) 08:40, 12 December 2025 (UTC)
- @Violine1101 I think I 👁 Image
Done it, by copying the overall syntax of the hideXP, and there does not appear to be issues at Trading as the multiplier seems to display correctly. ‑‑MinecraftExp123(talk|contribs) 02:28, 13 December 2025 (UTC)
- Awesome, thank you! Typical case of me having perfect be the enemy of good. (For context, the trade info modules do need code cleanup, we currently have two that do the same thing) | violine1101 (talk) 10:29, 13 December 2025 (UTC)
- Okay. I also really hope I didn't break anything, as I just copied the basic syntax of displaying the XP. I'm not sure the new hideMulti parameter is even needed, but added it just because there are the other hide parameters.
- By the way, I also cleaned up the multiplier at Module:TradeTable, as there's some redundant legacy code that is an artifact of when it displayed "High" and "Low" instead of just the value of the multipler. ‑‑MinecraftExp123(talk|contribs) 11:08, 13 December 2025 (UTC)
- Also it's partially my fault that the other module has some weird parts, like the thing I fixed was actually caused by me: Special:Diff/3164213 and Special:Diff/3316487 ‑‑MinecraftExp123(talk|contribs) 11:10, 13 December 2025 (UTC)
- No worries, I don't think you broke anything :) The "no_xp" (and now "no_multiplier") argument is basically only there for the wandering trader offers at Trading#Wandering Trader offers. I'd like to replace them with a smarter system as well but that takes some code rewriting. | violine1101 (talk) 12:06, 13 December 2025 (UTC)
- Great, thanks. I also mentioned them in the documentation. ‑‑MinecraftExp123(talk|contribs) 12:10, 13 December 2025 (UTC)
