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

Fluid would not be erase after Plot being Un-merged, unlink, clear. #3251

Open
2 tasks done
Rainformaloh1020 opened this issue Sep 19, 2021 · 5 comments
Open
2 tasks done
Labels
Approved A project maintainer or developer has reproduced this issue Bug Something isn't working

Comments

@Rainformaloh1020
Copy link

Server Implementation

Paper

Server Version

1.17.1

Describe the bug

Plot that merged, If there's water (lava tested and can not produced the same issue)
presented within the merged / connected area between 2 signal plot,
Water may not be clean up when the plot was being ' erase ' directly.
2 command can be used to reproduce the issue. in my testing,
command /plot unlink & /plot clear were used.

Picture evidence will be attached in the section below.

To Reproduce

  1. claim 2 plot that is next to each other.
  2. stand on 1 side of plot, facing toward to the plot that are going to merge.
  3. /plot merge
  4. filling water in btw the merging area manually or by worldedit. (FAWE was used in this test)
  5. the merging area by definition, was the 'extra' region that player can used to build and destroy blocks; originally shown as the ' border ' / the road that speared each plot.
  6. /plot unlink or /plot clear without cleaning any water that present within the merged area.

Expected behaviour

Water and any other blocks are expected to be delete/ remove during plot un-merged.

Screenshots / Videos

Pic. 1
2 plot merged with water in btw.
Picture3

Pic. 2
Expected to see with 2 plot that unlink/ unmerged after placing water.
Picture4

Pic 3.
Bird's-eye view of the issue area
Picture5b

Error log (if applicable)

No response

Plot Debugpaste

https://athion.net/ISPaster/paste/view/1c669f7ba94d4850b4ad837bf52da7e6

PlotSquared Version

Plotsquared-6.1.2-Premium

Checklist

Anything else?

No response

@Rainformaloh1020 Rainformaloh1020 added the Requires Testing This is a new issue which needs to be approved before labeled with "bug" label Sep 19, 2021
@Rainformaloh1020
Copy link
Author

Rainformaloh1020 commented Oct 22, 2021

Update [01]
It has been officially 1 months after posting with this issue, and still haven't heard anythings or follow-up from the developer nor develop team member assign with this issue.

Recorded and last check: 22.Oct.2021

Information update:
Plot Debugpaste : https://athion.net/ISPaster/paste/view/1fe383a6585d4408b9e00e812d3d28f5
PlotSquared version : Plotquared-6.1.3-Premium

@NotMyFault
Copy link
Member

There's no need to bump issues, someone may take a look when and if they have the time to.

@Rainformaloh1020
Copy link
Author

Thanks for your reply, I hope and will love to see some progression if not all but some.
This is definitely better than getting ignored and hear nothing back.
First reporting, Not quite sure if the issue has already being noticed / or even being received.

@PierreSchwang
Copy link
Member

Can't seem to reproduce that.
Before (Merged + previous road filled)
grafik
After (Unlink)
grafik

@NotMyFault NotMyFault added Approved A project maintainer or developer has reproduced this issue Bug Something isn't working and removed Requires Testing This is a new issue which needs to be approved before labeled with "bug" labels Oct 23, 2021
@NotMyFault
Copy link
Member

NotMyFault commented Oct 23, 2021

I can replicate that: https://streamable.com/jpzkcr

Streamable
Watch "2021-10-23 15-33-51" on Streamable.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Approved A project maintainer or developer has reproduced this issue Bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants