Put some fields as "final"
Made all abstract classes' constructors "protected" (them being public serves no purpose since they cannot be instantiated outside of child classes)
Did some minor code prettifying
The island will only be saved if isChanged() is true. It is set to true
if any setter is used or a field is changed.
This should help servers shutdown a lot faster and background database
saves go quicker. Only applies to Islands!
https://github.com/BentoBoxWorld/BentoBox/issues/1629
* Improves the meta data API for Island
* Unified API into MetaDataAble Interface
All classes now use the same interface and Optionals. Reduces code
duplication and makes the API the same across the board.
* Version 1.15.6
Fixed since JavaDocs
This is modeled after the Bukkit metadata API with the difference that
it is persistent, i.e., metadata is stored to the database. Metadata can
be placed on Islands or Users.
This API should be useful for addons that do not want or need to create
their own database tables and instead just want to tag the user with
some data, or tag the island with some data. It is intended for small
amounts of data, like boolean tags or other values.
* Break out events and add individual handlers
https://github.com/BentoBoxWorld/BentoBox/issues/1613
* Corrected AddonEvents to have proper static handlers
* Made constructors protected.
Abstract classes should not have public constructors. Constructors of
abstract classes can only be called in constructors of their subclasses.
So there is no point in making them public. The protected modifier
should be enough.
* Try loading adjacent chunks, but too slow.
* NMS void delete
* Revert the extra chunk in IslandDeletion.
That was for another deletion approach.
* Added NMS abstraction.
* Debug
* Clearer code.
* Clarify exceptions
* Persistent entities should never be removed.
There is no valid reason why BentoBox should remove the Persistent entity. Otherwise, what is the point of being persistent if someone can just remove it?
* Changes which mobs can be removed outside island
Protect renamed and persistent entities from being removed if they wander outside island bounds.
Except for animals, as they cannot be removed only if they are renamed.
There was not specific reason why this was being done and in testing, I
cannot find a reason why it is needed (yet) so I am removing it due to
the lag reported in
https://github.com/BentoBoxWorld/BentoBox/issues/1580
* Add getKey and getValue for Pair calls for nicer access.
* Fixes ConcurrentModificationException in HeadGetter.
Fixes an issue when elements with the same name were overwritten by HeadGetter.
