IntellectualSites / IntellectualSites/PlotSquared

Plot limit can be overwritten.

Open
#4,339 4 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Requires Testing
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
  1. 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
  2. To reproduce the whole thing. Do you need at least 2 plots that merged.
  3. A villager should be spawned where the road used to be.
  4. Hit this Villager so that it moves around a bit as the counts are then recalculated.
  5. As soon as step 3 happens, another mob or misc can be spawned/set.
  6. 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
Anything else?

No response

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. 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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.