jasonw4331 / MyPlot

Plot and protection plugin for PocketMine-MP
https://discord.gg/Qqcpp9Q
MIT License
101 stars 89 forks source link

/p reset\dispose bug with merge plots #448

Closed efor89 closed 2 years ago

efor89 commented 2 years ago

Issue description

hi jason i have the problem that only 1 plot is remove from the max plots if the plot is a merge. if my max plots is 4 and i have a 4er merge and i use reset after this i can only claim 1 plot.

Steps to reproduce the issue

  1. /p claim
  2. /p merge confirm
  3. /p reset confirm
  4. /p list
  5. /p claim

OS and versions

Other Plugins

Console error, backtrace or other files

no error

HighTecDev commented 2 years ago

A Merge always counts as one Plot

efor89 commented 2 years ago

no a 4er merge are 4 plots if my max plots 4 i can only have 1 4er merge and with reset only the origin is remove from max plots so 3 are in max plots

Wertzui123 commented 2 years ago

Should a merge count as one or multiple plots? @jasonwynn10

jasonw4331 commented 2 years ago

merged plots count as individual plots for the permission calculation system