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

Cannot correctly handle undo of specified operations with specified blocks #2591

Closed
5 tasks done
Windmill1055 opened this issue Jul 30, 2024 · 1 comment
Closed
5 tasks done
Labels
status:invalid This does not describe a request or bug with WorldEdit type:bug Incorrect behavior, not working as intended

Comments

@Windmill1055
Copy link

WorldEdit Version

7.3.5

Platform Version

Fabric 0.15.11

Confirmations

  • I am using the most recent Minecraft release.
  • I am using a version of WorldEdit compatible with my Minecraft version.
  • I am using the latest or recommended version of my platform software.
  • I am NOT using a hybrid server, e.g. a server that combines Bukkit and Forge. Examples include Arclight, Mohist, and Cardboard.
  • I am NOT using a fork of WorldEdit, such as FastAsyncWorldEdit (FAWE) or AsyncWorldEdit (AWE)

Bug Description

Operations that involving some specfied blocks can bot be undone correctly. When trying to set an extent to air or cut it to clipboard and undo it, these blocks can not be restored and just disappear in some cases.
Currently the blocks broken I have tested include redstone wire, redstone block, any carpet, extended piston. I have not tested every block yet, but it seems to be that attachable blocks are mostly likely to fail.

Expected Behavior

When undo a operation, the extent should be restored to its previous state.

Reproduction Steps

For the simplest case:

  1. Setup a line of blocks horizontally
  2. Put redstone wires on top of them
  3. Select this two-height structure
  4. Cut it or set it to air
  5. Undo this operation
  6. Some redstone wires can not be restored

For a complex case:
Just put a bunch of blocks with redstone wires randomly, cut or set-with-air, then undo it. The other blocks have chance to fail.

Anything Else?

This seems to be caused by the removal of reorderExtent which i think is used to schedule the attachable blocks to be handled later than the blocks they attached to, and the current implementation of set-block-without-update failed to handle redstone wires or other blocks.

@Windmill1055 Windmill1055 added status:pending Pending acceptance or closure. type:bug Incorrect behavior, not working as intended labels Jul 30, 2024
@wizjany
Copy link
Collaborator

wizjany commented Jul 30, 2024

duplicates #2516

@wizjany wizjany closed this as not planned Won't fix, can't repro, duplicate, stale Jul 30, 2024
@wizjany wizjany added status:invalid This does not describe a request or bug with WorldEdit and removed status:pending Pending acceptance or closure. labels Jul 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status:invalid This does not describe a request or bug with WorldEdit type:bug Incorrect behavior, not working as intended
Projects
None yet
Development

No branches or pull requests

2 participants