Skip to content

Conversation

@SwissalpS
Copy link

when clicking on the 'wrong' part of a multinode-node,
direction calculation is nerfed. This patch avoids a
server crash and emits a sound and message.
User can simply click on another part of the multinode-
node and the procedure should work.

I didn't bother adjusting the pointed_thing to make this work, as I
don't consider the trouble worthwhile for this
kind of node that breaks the voxel world.

The bug was detected by player frogTheSecond and reported
by Huhhila on pandorabox. Thanks guys.

SwissalpS and others added 30 commits September 30, 2021 14:04
moved the strings to replacer_blabla.lua
Use new Technic Plus tool API
respect param2 when replacing in field mode
not doing so in crust mode as that would break usefulness

also check for vacuum when placing in crust mode
and some combined with double to single quotation marks
also makes infrastructure for post on_place actions
besides adding support for direct placing of certain
nodes that have special nodes depending on rotation.
This allows servers to restrict usage under dynamic conditions
SwissalpS and others added 30 commits February 10, 2022 20:28
fixes: #39
Reduce Technic registration warnings
The drop list from engine is in reverse order, so e.g. when
trying to set to leaves without leaves in main inventory,
replacer would be set to sapling.
This commit checks all drops first and if finds same drop
as node, it is used. Otherwise the first in list is used.
Thanks Niklp09 for reporting.
fixes: #47
when clicking on the 'wrong' part of a multinode-node,
direction calculation is nerfed. This patch avoids a
server crash and emits a sound and message.
User can simply click on another part of the multinode-
node and the procedure should work.

I didn't bother adjusting the pointed_thing to make this work, as I
don't consider the trouble worthwhile for this
kind of node that breaks the voxel world.

The bug was detected by player frogTheSecond and reported
by Huhhila on pandorabox. Thanks guys.
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 this pull request may close these issues.

3 participants