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

Database-purger removes valid entries of a partial plot area when minimum plot ID X or Z is set to a negative value #4326

Open
2 tasks done
V1Matt opened this issue Feb 8, 2024 · 0 comments
Labels
Requires Testing This is a new issue which needs to be approved before labeled with "bug"

Comments

@V1Matt
Copy link

V1Matt commented Feb 8, 2024

Server Implementation

Paper

Server Version

1.20.4

Describe the bug

I have a survival world set up with a partial plot area. The minimum plot ID is -1;-4 and the maximum plot ID is 3;0. When a player claims a plot, it is stored in the MySQL database. However, the entries are removed when the server closes.
Entries are not removed if database-purger is disabled.
Entries are not removed if a partial plot area is set up with a minimum plot ID that only has X and Z values of 0 or higher.

I tested without other plugins (only FastAsyncWorldEdit and PlotSquared).
I tested with different databases, servers and configurations, each time getting the same results.

To Reproduce

  1. Enable database-purger
  2. Run /p setup to create a partial plot area with a minimum plot ID that has negative values (for example: -1;-1)
  3. Restart the server.
  4. Claim one of the new plots. It should show in the database.
  5. Close the server. The database entry is now removed.

Expected behaviour

Correct entries not being removed by database purger.

Screenshots / Videos

No response

Error log (if applicable)

No response

Plot Debugpaste

https://athion.net/ISPaster/paste/view/2afa1aa88c59427cbf65c49b0a00f4b5

PlotSquared Version

PlotSquared-7.3.3-Premium

Checklist

Anything else?

English is not my native language, I hope you understand my bug report :) thank you for reading

@V1Matt V1Matt added the Requires Testing This is a new issue which needs to be approved before labeled with "bug" label Feb 8, 2024
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

1 participant