- Fixes#3860
- Guice requires javax annotations be used for Inject annotated constructors
- Guice does not require not-null annotations, this is assumed
- Also to do with #3413
* feat: Move ExpireManager to Guice by removing legacy IMP references
* Mark ExpireManager IMP as deprecated and add comments
* Add import for PlotPlatform for function reference
* Add ExpireManager instance call and optimize performance
* Fix#3820
* Reword message and add link to completed translations
* Fix#3820
* Reword message and add link to completed translations
* Add line breaks for better code readability
* Add line breaks for better code readability
Co-authored-by: Alexander Brandes <mc.cache@web.de>
* feature: add flag for beacon effects of other plots
Took 7 minutes
* fix: don't add the beacon effect flag if the platform is not paper
Took 2 minutes
* fix: creating consistency in documentation
Took 7 minutes
* feature: add global paper setting for the beacon-effect
* fix: description of the plot flag and rewrite of the listener
* fix: renamed flag and added road support
* fix: renamed flag and changed flag functionality
* fix: added spigot fallback listener and fixed NPE
* refactor: Address checkstyle violations
* addition: remove beacon effects when player leaves a plot
* fix: updated javadoc and fixed usage of the PlotListener#addEffect method
* chore: Run license updater
Co-authored-by: BuildTools <unconfigured@null.spigotmc.org>
Co-authored-by: Alexander Brandes <mc.cache@web.de>
- No need to wrap the final analysis into another async method. It's already contained in an async task
- Don't overstretch a chunk bounday when setting blocks to newBlocks array
* Implement build height limits into more events
* Implement util method to check if y is within world build limits
* Extract duplicate player-build-height-checking to plot area
- Extract to core module to allow any possible future platforms to access the method, as they will always duplicate
* Add code tag to javadoc
Co-authored-by: Alexander Brandes <mc.cache@web.de>
Co-authored-by: Alexander Brandes <mc.cache@web.de>
* Ensure forceSync is provided when constructing chunk coordinators
* Re-add old factory create method and deprecate for removal
* Remove old create method that Guice doesn't like
* Implement tile entities to generation using Populators
- Fixes#3051
* Javadocs
* Don't do the big error if heads don't work
* Address comments regarding javadocs/comments
* Ensure Location is still sealed, and add api description annotation to public methods in UncheckedWorldLocation
* Clean up HybridGen
- There's no need for while loops acting as a modulo after we've already performed a modulo
- Make the code-sections calculating if positions are in the wall/road more readable
- Collaps duplicate if-elseif bodies
* Better exception handling when setting data to LimitedRegion during chunk population
* Address comments
Co-authored-by: Alexander Brandes <mc.cache@web.de>
* Better naming for "legacy" block state populator
Co-authored-by: Alexander Brandes <mc.cache@web.de>
* Only load world cofigurations if WorldEdit has fully enabled (safe to do)
- Fixes#3664
(cherry picked from commit f2e1e99be3b4f1fd5ce00e32ea7773dd4d1855aa)
* Fix imports
* QueryCapability is effectively a "dumb" method
* Update Core/src/main/java/com/plotsquared/core/PlotSquared.java
Co-authored-by: Alexander Brandes <mc.cache@web.de>
