Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Some crafted tools don't have a level and node counter when same tools crafter previously do #389

Closed
int-ua opened this issue Dec 27, 2019 · 3 comments
Labels
bug 🐛 Something isn't working Fixed Issue has been fixed/resolved but is waiting for something or needs testing

Comments

@int-ua
Copy link

int-ua commented Dec 27, 2019

Mentioned by Yad in #main, verified by me on mithril pickaxe. New one I just crafter didn't show node counter even after using it but I have two 6th level mithril pickaxes in storage.

@OgelGames OgelGames added the bug 🐛 Something isn't working label Dec 28, 2019
@OgelGames
Copy link
Contributor

OgelGames commented Dec 28, 2019

So I found the issue; silver and mithril tools (from moreores) are not overridden with toolranks descriptions. Still can't find why it was working before though...

BTW, the description is part of the tool's metadata, that's why your other pick still shows its level 😉 (also I think if you used that old pick, the node count would not go up)

EDIT: This is when it stopped working: pandorabox-io/pandorabox-mods@f29d022#diff-8903239df476d7401cf9e76af0252622
The original toolranks supported moreores (https://github.com/lisacvuk/minetest-toolranks/blob/master/init.lua#L208), but TenPlus1's does not (https://notabug.org/TenPlus1/toolranks/src/master/init.lua#L115).

EDIT 2: Seems to be a dependency issue: lisacvuk/minetest-toolranks#5 (moreores dependency was removed in TenPlus1's toolranks)
The correct solution would be for moreores to override its own tools, just like other mods like lavastuff and multitools do.

@OgelGames
Copy link
Contributor

OgelGames commented Dec 30, 2019

I've opened two PR's to fix this issue: minetest-mods/moreores#37 and lisacvuk/minetest-toolranks#7

Also, of course, my fork can be used to fix it on the server in the meantime, as it is also compatible with TenPlus1's fork 😉

@OgelGames OgelGames added the Fixed Issue has been fixed/resolved but is waiting for something or needs testing label Dec 30, 2019
thomasrudin pushed a commit to pandorabox-io/pandorabox-mods that referenced this issue Dec 30, 2019
thomasrudin pushed a commit to pandorabox-io/pandorabox-mods that referenced this issue Dec 30, 2019
@thomasrudin
Copy link

Thanks @OgelGames, i switched to your forks and subscribed to the PR's 😄

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🐛 Something isn't working Fixed Issue has been fixed/resolved but is waiting for something or needs testing
Projects
None yet
Development

No branches or pull requests

3 participants