Skip to content

Conversation

@CrAfTsArMy
Copy link

No description provided.

liyanic and others added 4 commits January 24, 2026 16:38
* Fix: Made the LocationWrapper extended LocationWrapperReduced to prevent duplicated code.

* Fix: Added missing repository

* Fix: Added the possibility to specify a max player value by a pattern and fixed a performance issue.

* Fix: Re-added and improved the hashCode and equals methods for LocationWrapper

* Fix: Refactored the toString method in LocationWrapperReduced and added a getter for defaultMaxPlayerCount.

---------

Co-authored-by: yan13try <102859802+yan13try@users.noreply.github.com>
* SQLTemplate now has a purpose
* Massively duplicated code removed
* No more silent catch and log exceptions
* No more logging within the lib code
* Fix: Made the LocationWrapper extended LocationWrapperReduced to prevent duplicated code.

* Fix: Added missing repository

* Fix: Added the possibility to specify a max player value by a pattern and fixed a performance issue.

* Fix: Re-added and improved the hashCode and equals methods for LocationWrapper

* Fix: Refactored the toString method in LocationWrapperReduced and added a getter for defaultMaxPlayerCount.

* Fix: Reworked the spawnRotatingCircle to use the Bukkit Scheduler.

---------

Co-authored-by: yan13try <102859802+yan13try@users.noreply.github.com>
Co-authored-by: Philipp <62473021+CrAfTsArMy@users.noreply.github.com>
@CrAfTsArMy CrAfTsArMy requested a review from liyanic January 24, 2026 16:04
@CrAfTsArMy CrAfTsArMy requested a review from liyanic January 24, 2026 16:06
@empfi

This comment was marked as abuse.

Copy link

@empfi empfi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Super

Copy link

@Idiotischer Idiotischer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please switch to Particle.<Particle_TYPE>.builder().allPlayers().spawn()

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants