Djaytan / mc-jobs-reborn-patch-place-break

A place-break patch extension of JobsReborn plugin for Bukkit servers (Minecraft).
MIT License
5 stars 2 forks source link

Add safety checks & advanced tests cases for tags moving scenarios #587

Open Djaytan opened 7 months ago

Djaytan commented 7 months ago

Safety check

Associate to each tag a block type. Then, at PPB exploit check time, if the tag's block type doesn't match currently broken one (e.g. a sugarcane is broken whereas the tag initially apply to a stone block) then do not consider the action as an exploit one. This change will works only if the tags moving mechanism is flawless.

Advanced testing for tags moving scenarios

Review and eventually improve the tags moving behavior (does it works completely as intended, even for corner cases?). What about moving tags associated to air blocks (i.e. ephemeral tags and "ghost" ones?).