Skip to content

Correctif où la singularité ne pouvait pas etre ouvrir sans regarder de bloc#1220

Closed
gtolontop wants to merge 2 commits intoServerOpenMC:masterfrom
gtolontop:fix/singularity-no-block
Closed

Correctif où la singularité ne pouvait pas etre ouvrir sans regarder de bloc#1220
gtolontop wants to merge 2 commits intoServerOpenMC:masterfrom
gtolontop:fix/singularity-no-block

Conversation

@gtolontop
Copy link
Copy Markdown
Contributor

Corrige le bug où la singularité ne pouvait pas être utilisée sans regarder un bloc.

Problème

Dans InteractListener.onInteract(), un check event.useInteractedBlock() == Event.Result.DENY provoquait un return immédiat lorsque le joueur ne regardait aucun bloc. Cela empêchait tout item custom implémentant UsableItem (comme la Singularité) de fonctionner en l'air.

Correction

  • Suppression du check useInteractedBlock() == DENY qui bloquait inutilement l'événement pour les items custom

Fixes #1213

@iambibi iambibi added this to the 2.2.9 milestone Mar 23, 2026
@iambibi iambibi added the ✨ Fixes Quelque chose a été patch label Mar 23, 2026
@iambibi
Copy link
Copy Markdown
Member

iambibi commented Mar 23, 2026

Donne des noms lisible pour les joueurs, et demande à ta fichue IA de suivre la template

@gtolontop
Copy link
Copy Markdown
Contributor Author

bah wtf je suis juste le mm plan depuis tous les jours

@gtolontop
Copy link
Copy Markdown
Contributor Author

je suis littéralement le même plan de PR depuis le premier jour et jfais bien gaffe a tjr faire le même pour être cohérent ptdrrrrr faut pas me dire mtn que faut changer

@iambibi
Copy link
Copy Markdown
Member

iambibi commented Mar 23, 2026

oui mais si je te fais la remarque c'est que c'est casse couille, meme apres 1 an

@gtolontop
Copy link
Copy Markdown
Contributor Author

oui mais si tu mle dis pas avant ça sert a rien dle dire comme ça psq moi jle trouvais cool dcp bah comme on mdis rien jprend ça pour un "il est carré"

@gtolontop
Copy link
Copy Markdown
Contributor Author

tu peux pas d'un coup dire "il est guez change" en ragant (tu m'as compris)

@gtolontop
Copy link
Copy Markdown
Contributor Author

on me l'aurais dja dis plusieurs fois why not mais on me l'as jamais dis

@gtolontop
Copy link
Copy Markdown
Contributor Author

et on est sur un repo github jvais pas faire des commit de golmon déjà j'ai changé de anglais > français jvais pas faire un commit avec "fix bug item utilisable que sur un bloc"

@gtolontop
Copy link
Copy Markdown
Contributor Author

tema l'image stp

@iambibi
Copy link
Copy Markdown
Member

iambibi commented Mar 23, 2026

tu peux pas d'un coup dire "il est guez change" en ragant (tu m'as compris)

ah mais je rageais pas xD, c'était juste un commentaire rien de plus

@gtolontop
Copy link
Copy Markdown
Contributor Author

surtout que j'explique entièrement le soucis dans le commit xd donc si vrm il veulent le détaille il est dedans je l'écris moi même et jle fais corriger par une AI pour qu'il soit propre et correcte a lire pour tous le monde :)

@iambibi
Copy link
Copy Markdown
Member

iambibi commented Mar 23, 2026

et on est sur un repo github jvais pas faire des commit de golmon déjà j'ai changé de anglais > français jvais pas faire un commit avec "fix bug item utilisable que sur un bloc"

image Regarde la diff, entre fix ggngng et Ajout de tata pour tata. Pour la release sur le discord c'est bien plus propre xD

@gtolontop
Copy link
Copy Markdown
Contributor Author

tabuseeee mes pr sont plus complete xd

@gtolontop
Copy link
Copy Markdown
Contributor Author

j'ai pas dis "fix item"

@iambibi
Copy link
Copy Markdown
Member

iambibi commented Mar 23, 2026

tabuseeee mes pr sont plus complete xd

j'ai pas parlé de complet ou non, juste le mec ne sait pas ce que cette pr fait, fin la je la push, les mecs le lisent, ils auront mm pas compris que ça visait a corriger le fait que la singu n'etait pas ouvrable lorsqu'on regardait aucun block

@gtolontop
Copy link
Copy Markdown
Contributor Author

Problème
Dans InteractListener.onInteract(), un check event.useInteractedBlock() == Event.Result.DENY provoquait un return immédiat lorsque le joueur ne regardait aucun bloc. Cela empêchait tout item custom implémentant UsableItem (comme la Singularité) de fonctionner en l'air.

je peux pas faire plus clair que sque j'ai mis dans la truc stp

@iambibi
Copy link
Copy Markdown
Member

iambibi commented Mar 23, 2026

Problème Dans InteractListener.onInteract(), un check event.useInteractedBlock() == Event.Result.DENY provoquait un return immédiat lorsque le joueur ne regardait aucun bloc. Cela empêchait tout item custom implémentant UsableItem (comme la Singularité) de fonctionner en l'air.

je peux pas faire plus clair que sque j'ai mis dans la truc stp

un effort stp, personne clique sur la PR, juste le nom doit suffir a la personne pour savoir ce qui a été corrigé, imagine t Rylo le mec qui a fait son issue, et tu voudrais savoir quand c'est fixé, jamais il va comprendre que fix: supprimer le check useInteractedBlock dans InteractListener c'est pour la singu.
Stp fait Correctif où la singularité ne pouvait pas etre ouvrir sans regarder de bloc

@iambibi iambibi changed the title fix: supprimer le check useInteractedBlock dans InteractListener Correctif où la singularité ne pouvait pas etre ouvrir sans regarder de bloc Mar 23, 2026
@PuppyTransGirl
Copy link
Copy Markdown
Member

PuppyTransGirl commented Mar 23, 2026

ça fix pas le problème, il faut enlever le ignoredCancelled = true

@PuppyTransGirl
Copy link
Copy Markdown
Member

quand tu cliques dans les air le PlayerInteractEvent est envoyé aux plugins mais déjà cancel

@PuppyTransGirl
Copy link
Copy Markdown
Member

@gtolontop
Copy link
Copy Markdown
Contributor Author

Problème Dans InteractListener.onInteract(), un check event.useInteractedBlock() == Event.Result.DENY provoquait un return immédiat lorsque le joueur ne regardait aucun bloc. Cela empêchait tout item custom implémentant UsableItem (comme la Singularité) de fonctionner en l'air.
je peux pas faire plus clair que sque j'ai mis dans la truc stp

un effort stp, personne clique sur la PR, juste le nom doit suffir a la personne pour savoir ce qui a été corrigé, imagine t Rylo le mec qui a fait son issue, et tu voudrais savoir quand c'est fixé, jamais il va comprendre que fix: supprimer le check useInteractedBlock dans InteractListener c'est pour la singu. Stp fait Correctif où la singularité ne pouvait pas etre ouvrir sans regarder de bloc

ça change plus de fonctionnement que juste la singularité d'ou mon titre :)

@gtolontop
Copy link
Copy Markdown
Contributor Author

ça fix pas le problème, il faut enlever le ignoredCancelled = true

j'y avais pas pensé ta ptet raison jvais check

@iambibi
Copy link
Copy Markdown
Member

iambibi commented Mar 26, 2026

ça fix pas le problème, il faut enlever le ignoredCancelled = true

j'y avais pas pensé ta ptet raison jvais check

tu as testé? montre screen de toi ds singu en regardant ds le ciel?

@iambibi iambibi removed this from the 2.2.9 milestone Mar 26, 2026
@iambibi iambibi added this to the 2.2.10 milestone Mar 26, 2026
@AxenoDev AxenoDev modified the milestones: 2.2.10, 2.2.11 Apr 5, 2026
@iambibi
Copy link
Copy Markdown
Member

iambibi commented Apr 7, 2026

@gtolontop merci de répondre sous délai de 3 semaines
sinon on reprendra ton travail

@gtolontop
Copy link
Copy Markdown
Contributor Author

tu peux dja le reprendre

@gtolontop
Copy link
Copy Markdown
Contributor Author

bah tu reprend pas ??? faut savoir ?

@gtolontop
Copy link
Copy Markdown
Contributor Author

tu as bsn que je t'explique une notion ?

@iambibi
Copy link
Copy Markdown
Member

iambibi commented Apr 8, 2026

bah tu reprend pas ??? faut savoir ?

ben si je le reprends ptdr pas mtn car j'ai d'autres choses prioritaires mais tkt pas m'apprends moi comment a m'assigner pls

@iambibi
Copy link
Copy Markdown
Member

iambibi commented Apr 8, 2026

honnetement c'est dommage, tu avais l'air serieux

@gtolontop
Copy link
Copy Markdown
Contributor Author

bah tu reprend pas ??? faut savoir ?

ben si je le reprends ptdr pas mtn car j'ai d'autres choses prioritaires mais tkt pas m'apprends moi comment a m'assigner pls

Jte ferais un tuto

@gtolontop
Copy link
Copy Markdown
Contributor Author

honnetement c'est dommage, tu avais l'air serieux

Jte rappel que j'ai arrêté de contribuer quand vous avez commencé à m'engueuler pour des truc compléments guez en me disant ct dla d j'ai stop de contribuer et on revient me voir avec un msg de neuil "dans 3 semaines c'est repris si tu fais pas" vla la frappe

@gtolontop
Copy link
Copy Markdown
Contributor Author

Surtout que ta aucune idée si j'ai lancer le serveur ou pas et tu me dis que l'AI sais pas lancer le serveur alors que jlai lancé plusieurs fois et que c'est déplacé bref

@iambibi
Copy link
Copy Markdown
Member

iambibi commented Apr 8, 2026

honnetement c'est dommage, tu avais l'air serieux

Jte rappel que j'ai arrêté de contribuer quand vous avez commencé à m'engueuler pour des truc compléments guez en me disant ct dla d j'ai stop de contribuer et on revient me voir avec un msg de neuil "dans 3 semaines c'est repris si tu fais pas" vla la frappe

Mon role avec Axeno c'est de maintenir le projet, tu as proposé une idée, on t'a dit non avec des arguments. Tu t'es faché. On ne peut pas dire toujours oui à toutes les idées.
Puis honnetement j'ai jamais engueuler, c'est sur quand tu repetes la meme chose c'est redondant. Tu faisais juste du rage bait a ce stade.

Nous on gere juste le repo si une PR reste stagnante, on mets une dead line, et voila.
Je ne vais pas rentrer dans ton jeu honnetement j'ai d'autres choses a faire que ça.
Bonne continuation dans tes projets personnels.

@ServerOpenMC ServerOpenMC locked as too heated and limited conversation to collaborators Apr 8, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

✨ Fixes Quelque chose a été patch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] Impossible d'utiliser la singularité en regardant aucun bloc

4 participants