IntellectualSites / IntellectualSites/PlotSquared
Auto Merge does not check merge limits
Nobody has claimed this yet.
- Dominant language
- Java
- Stars
- 653
- Forks
- 975
- Avg merge
- 2m
- Merged PRs (30d)
- 6
Description
Server Implementation
Paper
Server Version
1.19
Describe the bug
Gave players the plots.merge.9 permission node, but with auto-merge enabled they can still create a plot of ANY size.
To Reproduce
- Create new plot world, default settings
- Enable auto-merge in world settings
- Set merge limit with permission node
- Start claiming plots
Expected behaviour
Auto-merge does not merge after 9 plots because of permission node.
Screenshots / Videos
No response
Error log (if applicable)
No response
Plot Debugpaste
https://athion.net/ISPaster/paste/view/329fc18b8e164dddbf9e9e014b0b13b8
PlotSquared Version
6.9.3-Premium
Checklist
- I have included a Plot debugpaste.
- I am using the newest build from https://www.spigotmc.org/resources/77506/ and the issue still persists.
Anything else?
No response
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start at the auto-merge claim path and trace where the plots.merge.9 permission limit is evaluated. Reproduce with auto-merge enabled and a nine-plot limit; done means automatic merging stops at nine plots while claiming remains functional, with regression coverage if the existing tests provide a suitable location.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- java
- Domain
- game-dev
- Issue type
- Bug
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100