Scan Tool doesn't prompt "Point one/two saved" message when used on fence block
omega025 opened this issue ยท 5 comments
Minecolonies version
version 0.8.1917
Expected behavior
- Right-clicking on fence block with Scan Tool should prompt "Point one saved" or "Point two saved" chat message.
Actual behaviour
- Right-clicking on fence block with Scan Tool appears to register (see below), but no chat message is displayed.
- Right-clicking on another block with the Scan Tool after right-clicking on a fence block prompts a "Point two saved, right click again to save the schematic" chat message.
or
- Right-clicking on a regular block while holding the Scan Tool prompts the "Point one saved" chat message.
- Right-clicking on an oak fence block while holding the Scan Tool prompts no chat message, but right-clicking again prompts the message indicating that the "scan [was] successfully saved".
Steps to reproduce the problem
- Build something with an oak fence block in a lower front corner.
- Equip Scan Tool.
- Right-click oak fence block while holding Scan Tool.
- Right-click another block while still holding Scan Tool.
or
- Build something with an oak fence block in an upper rear corner and a regular block in a lower front corner.
- Equip Scan Tool.
- Right-click regular block in lower front corner while holding Scan Tool.
- Right-click fence block in upper rear corner while still holding Scan Tool.
- Right-click again to receive notification of successfully saved scan.
Can confirm - scans were indeed saved to the minecolonies/scans/new folder. This was in single-player.
Its likely that some mod (could but minecolonies, unsure) is cancelling the right click event for fences on the client side (in "remote" worlds). So the following code is never called:
if (worldIn.isRemote)
{
LanguageHandler.sendPlayerMessage(playerIn, "item.scepterSteel.point");
}
We should be able to change this to just if (!worldIn.isRemote)
and be good, since deciding whether to actually send the packet, or add a message to chat is handled by minecraft's code anyway.