IntellectualSites / IntellectualSites/PlotSquared
Plot limit can be overwritten.
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.20.4
Describe the bug
The limitation can be circumvented by having the entity on a merged property where the road previously existed and walking around, so the plot cap is always recalculated and the transition to plot 1 to 2 where the road was ignored.
Fawe: 2.9.1-SNAPSHOT-659;c93ec87
To Reproduce
- Setup these Flags in words.yml:
flags:
animal-cap: 1
mob-cap: 1
hostile-cap: 1
entity-cap: 1
vehicle-cap: 1
misc-cap: 1 - To reproduce the whole thing. Do you need at least 2 plots that merged.
- A villager should be spawned where the road used to be.
- Hit this Villager so that it moves around a bit as the counts are then recalculated.
- As soon as step 3 happens, another mob or misc can be spawned/set.
- The whole thing can be done permanently without limits.
Reminder: There cannot be a mob on plot 1 and 2, it must be between 1 and 2 where the road was before.
Detailed instructions can be seen in the video above.
Expected behaviour
The limitation should be maintained and not be recalculated again and again as soon as a mob runs over the merged property.
In general it should not be possible for the cap to be overwritten.
Screenshots / Videos
How to bypass cap:
https://www.youtube.com/watch?v=KCqGM4RalOE&feature=youtu.be
ArmorStands are ignored by caps and can be placed infinitely.
https://www.youtube.com/watch?v=0YuhzfdSx0I
Error log (if applicable)
No response
Plot Debugpaste
https://athion.net/ISPaster/paste/view/756733ea33a144f7b23cf2e472a6b4dc
PlotSquared Version
7.3.4-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 by reproducing the merged-plot scenario with PlotSquared 7.3.4-Premium, Paper 1.20.4, and the listed cap flags. Verify the villager movement and ArmorStand cases against the expected behavior that caps cannot be bypassed or overwritten. No source files or tests are named, so locating the relevant cap-counting code will require project familiarity.
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
- Needs clarification
- Newbie friendliness
- 30/100