SkytAsul / BeautyQuests

A quests plugin based on a simple-to-use graphical interface (Minecraft)
MIT License
149 stars 28 forks source link

Support Folia - Branch develop #314

Closed Euphillya closed 2 months ago

Euphillya commented 3 months ago

Change all schedulers using Energie Use PaperLib to fix player teleportation

Please report any possible bugs arising from this patch.

Changement de toutes les schedulers en utilisant Energie Utiliser PaperLib pour fixer la téléportation des joueurs

Merci de me soumettre tout possible bug provenant de ce patch.

SkytAsul commented 3 months ago

Globalement, je ne comprend pas pourquoi tu dois obligatoirement passer par le callback pour récupérer une instance de la tâche. Normalement, toutes les méthodes de type scheduleTask devraient renvoyer la tâche. C'est comme ça que fonctionnent les APIs de Spigot et de Folia.

SkytAsul commented 3 months ago

Et est-ce que tu penses que tu pourrais intégrer la méthode teleportAsync dans ton API Energie ? J'aimerais bien ne pas avoir à shade 2 nouvelles librairies, dont une qui ne servirait qu'à une seule méthode...

Euphillya commented 3 months ago

Et est-ce que tu penses que tu pourrais intégrer la méthode teleportAsync dans ton API Energie ? J'aimerais bien ne pas avoir à shade 2 nouvelles librairies, dont une qui ne servirait qu'à une seule méthode...

Non car ce n'est pas le but de la lib, par contre, j'ai donc juste fait une exception avec NoSuchMethodError pour que les version après la 1.13.2+ (paper) puisse utiliser le tpAsync, sinon on utilise le sync. Ca a quand même été intégré par une personne, donc j'ai modifié