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

pick block does not pick shadowed items on server #8

Closed
TyIuNnG opened this issue Oct 21, 2022 · 7 comments
Closed

pick block does not pick shadowed items on server #8

TyIuNnG opened this issue Oct 21, 2022 · 7 comments
Assignees
Labels
bug Something isn't working

Comments

@TyIuNnG
Copy link

TyIuNnG commented Oct 21, 2022

Describe the bug
In single-player, players can middle-click a block to pick the block from inventory if available. This works with shadowed-item as well, but on server this only works if the mod is installed on both server and client side in my past experience with the mod. However in V1.9.6-1.19, pick block doesn't pick anything even if the mod is presented on both sides.

To Reproduce
middle-click any block

Expected behavior
items will be picked from inventory even if it's shadowed item

@TyIuNnG TyIuNnG added the bug Something isn't working label Oct 21, 2022
@mattymatty97
Copy link
Owner

that happens because the setting 'shadowItemTooltip' is active.
on servers the mod uses the lore Nbt Tag to show the tooltip to the clients that effectively makes clients think it is a different item preventing middle-click actions.
the client side mod removes the tag and shows the tooltip with it's own edit to the code instead.

TL:DR:
disable 'shadowItemTooltip' to fix the issue

@TyIuNnG
Copy link
Author

TyIuNnG commented Oct 21, 2022

thanks that fix the issue, but why does the client side not remove the tag and do the thing you said?

@mattymatty97
Copy link
Owner

probably some changes in the client code that i didn't notice when i converted the mod to 1.19 have to investigate more in the future

@TyIuNnG
Copy link
Author

TyIuNnG commented Oct 21, 2022

alright thanks for the help, I guess the client code will take some time to fix

@TyIuNnG TyIuNnG closed this as completed Oct 21, 2022
@TyIuNnG TyIuNnG reopened this Nov 8, 2022
@TyIuNnG
Copy link
Author

TyIuNnG commented Feb 24, 2023

probably some changes in the client code that i didn't notice when i converted the mod to 1.19 have to investigate more in the future

or could it possibly be because of the latest change changes
MutableText sub = MutableText.of(new LiteralTextContent(((ShadowItem) (Object) this).getShadowId()));
to
MutableText sub = MutableText.of(new LiteralTextContent(((ShadowItem) (Object) instance).getShadowId()));
Although I do not think it has much to do with it.

@mattymatty97
Copy link
Owner

Potentially Fixed by latest release, please confirm and close the issue 😄

@TyIuNnG
Copy link
Author

TyIuNnG commented May 21, 2023

Yes, it is fixed now, thank you so much!

@TyIuNnG TyIuNnG closed this as completed May 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants