MLG-Fortress / PopulationDensity

The now-official repository of the PopulationDensity Bukkit plugin
https://dev.bukkit.org/projects/population-density
10 stars 10 forks source link

Extends protection to cover the base under the region post. #82

Closed zedwick closed 4 years ago

zedwick commented 4 years ago

As the Y returned by World#getHighestBlockYAt now reflects the highest non-air block, including transparent blocks, we need to extend the protection area down by 1 block (as the highest block is now 1 block above the sign). Resolves #73

RoboMWM commented 4 years ago

Ok, this is interesting. The docs always stated that it returned the highest non-transparent block, but this wasn't the case (at least with leaves) for several versions.

Can you link me to the CB commit that changed this? Or if it's something that changed from 1.14->1.15 (or some other minor change)?

Nevermind, you stated the opposite of what I thought you stated.

RoboMWM commented 4 years ago

Related: https://hub.spigotmc.org/jira/browse/SPIGOT-5523