Terrestria

Terrestria

2M Downloads

Migrate legacy Andisol ids

FewerFlaws opened this issue · 7 comments

commented

Andisol dirt, grass blocks, and podzol use "basalt_dirt" "basalt_grass_block" and "basalt_podzol" within the namespace. This is inconsistent because the grass path and farmland blocks use "andisol_grass_path" and "andisol_farmland."

I imagine you already know about this and decided not to correct it. I'm just pointing it out in case it's an oversight. I noticed it while working on compatibility between Presence Footsteps and Terrestria =]

commented

New development on this. REI displays "andisol_grass_path" in the interface, but F3 debug view reveals that a placed block uses "basalt_grass_path"
2020-12-28_22 56 55
2020-12-28_22 57 03

commented

Yeah, this cannot be reasonably changed without breaking worlds. Perhaps we can fix it in the transition to 1.17 by using a datafixer, but unfortunately I don't believe we can get datafixers to run easily between mod version changes.

Luckily, ultimately it doesn't really matter what the ids are.

commented

Yeah, the item was added later after the rename.

commented

Surely the tooltip issue can be fixed, right? The name shown in the extended F3+H tooltip should be the actual namespace name so that mod authors can see the block and reference it properly. Fixing that shouldn't break anything.

commented

Oh, I didn't describe the issue with the grass path properly. Check out that first screenshot to see the issue. The Andisol grass path's extended tooltip is displaying the wrong namespace block ID.

commented

No it's not. Item IDs are different from block IDs.

commented