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

DisplayEntities remain after unlink #4445

Open
2 tasks done
Nik2810-Original opened this issue Jun 4, 2024 · 1 comment
Open
2 tasks done

DisplayEntities remain after unlink #4445

Nik2810-Original opened this issue Jun 4, 2024 · 1 comment
Labels
Requires Testing This is a new issue which needs to be approved before labeled with "bug"

Comments

@Nik2810-Original
Copy link

Server Implementation

Paper

Server Version

1.20.4

Describe the bug

If you merge your plot and then place your DisplayEntity on your plot where the street was before, then unlink it, the DisplayEntity will remain at the position where the street is again and will not be deleted.

To Reproduce

  1. Create a 2x1 Merge
  2. /summon the DisplayEntity where the street was before on your plot
  3. /p unlink

Expected behaviour

That the DisplayEntity is deleted from the street like with /p delete.

Screenshots / Videos

No response

Error log (if applicable)

No response

Plot Debugpaste

https://athion.net/ISPaster/paste/view/3d911c872801464aa0e4d53e8ad46661

PlotSquared Version

7.3.8-Premium

Checklist

Anything else?

No response

@Nik2810-Original Nik2810-Original added the Requires Testing This is a new issue which needs to be approved before labeled with "bug" label Jun 4, 2024
@OneLiteFeather
Copy link
Member

I tried it before so I can replicate it.

IntellectualSites/FastAsyncWorldEdit#2592 this has its reason that display entities aren't really implemented yet. I guess this is also with upstream (Worldedit), but you could check if it works with Worldedit. Best workaround for now is not building on the space where the road will be or not unlinking a display entity plot (if there are players, you might don't want them to use it then)

@IntellectualSites/core-team after the 1.21 release, is there time to add display entities maybe or should it be implemented in upstream?
Also is this rather a bug or a missing feature?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Requires Testing This is a new issue which needs to be approved before labeled with "bug"
Projects
None yet
Development

No branches or pull requests

2 participants