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

Mob cap on merged plots is not working #3050

Closed
2 tasks done
Xaver106 opened this issue May 19, 2021 · 0 comments · Fixed by #3643
Closed
2 tasks done

Mob cap on merged plots is not working #3050

Xaver106 opened this issue May 19, 2021 · 0 comments · Fixed by #3643
Labels
Approved A project maintainer or developer has reproduced this issue Bug Something isn't working

Comments

@Xaver106
Copy link
Contributor

Server Implementation

Paper

Server Version

1.16.5

Describe the bug

The mob cap on merged plots does not work. It looks like it handels the mob cap on eac sub plot individuatly and at the same time on the whole plot.
Setting the mob cap lets you place more than the allowed mobs on the plot. Sometimes you can't and have to move to another sub plot and you can place one more.
The /p caps command apperently works for the sub plots, as it clearly doesn't show all mobs and changes if you move to a different sub plot.
This behavier happens to all caps other than the misc-cap.

To Reproduce

  1. Claim a plot and merge more plots to it
  2. Set mob-cap to something
  3. Place some mobs on the plot. Try to fly arround while placing them
  4. check /p caps and see how it behaves on different subplots

Expected behaviour

caps are either the whole plot or each sub plot, but no a mixture of both.

Screenshots / Videos

Error log (if applicable)

Plot Debugpaste

https://athion.net/ISPaster/paste/view/71147f64df514e6e98d8c595927a36f3

PlotSquared Version

PlotSquared-5.13.11-Premium

Checklist

Anything else?

No response

@Xaver106 Xaver106 added the Requires Testing This is a new issue which needs to be approved before labeled with "bug" label May 19, 2021
@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 May 21, 2021
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

Successfully merging a pull request may close this issue.

2 participants