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

addition request: refuse to accept items with metadata #87

Closed
SwissalpS opened this issue Feb 3, 2024 · 1 comment · Fixed by #88
Closed

addition request: refuse to accept items with metadata #87

SwissalpS opened this issue Feb 3, 2024 · 1 comment · Fixed by #88

Comments

@SwissalpS
Copy link
Contributor

SwissalpS commented Feb 3, 2024

The way drawers store the contents doesn't allow for items to have metadata. Mostly these items aren't stackable so the check of max_stack weeds out a lot of incompatible items.

However, there are items that drawers currently accept and then return 'corrupted'.

One example are items created by the wrench:wrench (previously part of technic.plus).
It saves the inventory and other node data in .meta.data and .meta.compressed of an item which normally can be stacked and can be used with drawers.

reference: https://github.com/mt-mods/wrench/tree/cc93ad0b541c3fb98c84016d9167424ff7e0aae1

adding a check for existing .metadata, .meta.compressed and .meta.data could do the trick --> not accept items with any non-empty mentioned fields

@SwissalpS
Copy link
Contributor Author

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant