Skip to content

Rewrite des Registres des Features et des Hooks - #1453

Draft
iambibi wants to merge 9 commits into
ServerOpenMC:masterfrom
iambibi:chore/feature-hook-registry
Draft

Rewrite des Registres des Features et des Hooks#1453
iambibi wants to merge 9 commits into
ServerOpenMC:masterfrom
iambibi:chore/feature-hook-registry

Conversation

@iambibi

@iambibi iambibi commented Aug 17, 2026

Copy link
Copy Markdown
Member

Petit résumé de la PR:

tres expérimental, et je le fais par envie de faire du rewrite.
un grand pas, qui nettoie un probleme qui pars du début :

  • le static partout dans le projet
    le but de cette PR et de tuer ce probleme en le remplaçant par qlqchose de plus objet.
    A voir si ça passera dans la V2.5 mais ça peut attendre jusqu'a l'infini

Étape nécessaire afin que la PR soit fini (si PR en draft)

  • Suivre le Code de Conduite
  • Enlever tous les imports non utilisés
  • Bien documenter la feature
  • Fournir un profileur (si besoin/demandé par un admin)
  • Avoir une milestone associée à la PR
  • Valider tout les checks
  • Tester et valider la feature/changement
  • Les Issues corrigée(s) en commun :

Decrivez vos changements

@iambibi iambibi self-assigned this Aug 17, 2026
@iambibi iambibi added the 🧹Rewrite Souvent dédié à un rewrite ou tout simplement un changement interne qui change peu aux joueurs label Aug 17, 2026
@iambibi iambibi linked an issue Aug 18, 2026 that may be closed by this pull request
@iambibi

iambibi commented Aug 18, 2026

Copy link
Copy Markdown
Member Author

ultra répétitif, et je vois mes heures passés sur ce rewrite qui me semble de plus en plus géant au fur et a mesure que j'avance.
je vais pas planifier ça pour la V2.5, du moins je continuerais la PR la quand j'aurais envie de souffrir mais bon xD.
Au moins si des devs critiquent le plugin sur le point du static, regarder cette PR, je y mets mon temps et ma patience, mais je vois pas d'interet pour les joueurs d'avoir ça xD

@iambibi

iambibi commented Aug 18, 2026

Copy link
Copy Markdown
Member Author

ou sinon, je push la version avec que l'adminshop et les villes sans du static, tres inutile et pas envie de refaire un style "OMCPlayer" fait hyper cool mais pas utiliser car pas implémenter partout dans le plugin 😅

@gab4000

gab4000 commented Aug 18, 2026

Copy link
Copy Markdown
Contributor

Je ne comprends vraiment pas l'intérêt de cette PR... pourquoi te faire autant chier a faire du full object alors que tu peux faire du static ? Pourquoi l'enlever ?

@iambibi

iambibi commented Aug 19, 2026

Copy link
Copy Markdown
Member Author

Le static est hyper moche hein...

@iambibi

iambibi commented Aug 19, 2026

Copy link
Copy Markdown
Member Author

C'est bien connu, les rewrites sont fait pour augmenter la qualité du code hein 😂

@iambibi

iambibi commented Aug 19, 2026

Copy link
Copy Markdown
Member Author

Puis tu accèdes à une méthode mais imagine si la feature n'est pas initialisé mdrr

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

Labels

🧹Rewrite Souvent dédié à un rewrite ou tout simplement un changement interne qui change peu aux joueurs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[CHANGE] Registre de Features et de Hooks

2 participants