Skip to content

Ajout de l'injecteur de namespaces items adder#1262

Open
iambibi wants to merge 19 commits intoServerOpenMC:masterfrom
iambibi:feat/itemsadder-namespace-injector
Open

Ajout de l'injecteur de namespaces items adder#1262
iambibi wants to merge 19 commits intoServerOpenMC:masterfrom
iambibi:feat/itemsadder-namespace-injector

Conversation

@iambibi
Copy link
Copy Markdown
Member

@iambibi iambibi commented May 1, 2026

Petit résumé de la PR:

Le but de la PR c'est de centraliser les resources sur un depot et d'avoir un systeme d'injection des namespaces assez propre.
c'est aussi de la recherche, si je suis face a un probleme qui me bloque. J'irais en faire une issue sur ItemsAdder.
(a la suite, si cela semble affirmatif, je metterais un systeme de placeholder afin de remplacer un placeholder par une valeur dynamique (ect DreamItem.getRarity -> tooltip item/couleur nom item)

É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

Decrivez vos changements

@iambibi iambibi added this to the 2.5.0 milestone May 1, 2026
@iambibi iambibi self-assigned this May 1, 2026
@iambibi iambibi added the 📦 Feature Ajout d'une fonctionnalité label May 1, 2026
@iambibi
Copy link
Copy Markdown
Member Author

iambibi commented May 1, 2026

euh ouais j'ai pas pensé au nombre de lignes que ça allait introduire dcp

@gab4000
Copy link
Copy Markdown
Contributor

gab4000 commented May 1, 2026

OUAH nan mais qu'est-ce que c'est que ce truc là ?!

@iambibi
Copy link
Copy Markdown
Member Author

iambibi commented May 1, 2026

OUAH nan mais qu'est-ce que c'est que ce truc là ?!

euh bah c'est les namespaces items adder

@iambibi
Copy link
Copy Markdown
Member Author

iambibi commented May 1, 2026

Normalement la pr doit être en draft j'ai pas encore commencé, mais jpp sur tel

@iambibi iambibi marked this pull request as draft May 2, 2026 09:49
@iambibi iambibi linked an issue May 2, 2026 that may be closed by this pull request
@gab4000
Copy link
Copy Markdown
Contributor

gab4000 commented May 3, 2026

G pas compris le but de cette PR, ça permet d'éviter d'avoir à gérer les ressources items adder en dehors du plugin ?

@iambibi
Copy link
Copy Markdown
Member Author

iambibi commented May 3, 2026

G pas compris le but de cette PR, ça permet d'éviter d'avoir à gérer les ressources items adder en dehors du plugin ?

  • ça permet ce que tu viens de dire : devoir update le pack a chaque fois, mais c'est pas tellement la raison premiere
  • on pourra implementer ce qu'on veut dans les contents, par exemple, si tu as des raretés d'item et que sur items adder tu as des tooltip qui changent en fonction de la rareté, et que tu changes la rareté d'un item dans l'enum, ben ça le fera automatiquement dans le content itemsadder. (puis on peut aller bien plus loin en générant de A à Z nos namespaces avec nos CustomItem et tt mais c'est pas réelement utile et ça peut rajouter des bugs inutiles).

Voila les deux raisons majeures

@iambibi
Copy link
Copy Markdown
Member Author

iambibi commented May 3, 2026

puis la tu vois je viens de penser a un truc qui me bloque pour faire mes blocs constitué de 2 bloc IA.
Meme si je n'ai pas forcément besoin de cela, ça me permet d'acceder facilement aux resources pdt le bootstrap et de dire au plugin si ce bloc est posé, pose le bloc la, si ce bloc est cassé, casse celui la ect

@gab4000
Copy link
Copy Markdown
Contributor

gab4000 commented May 3, 2026

hmmmmm, je vois

@iambibi iambibi modified the milestones: 2.5.0, 2.5.0-beta-1 May 4, 2026
@iambibi
Copy link
Copy Markdown
Member Author

iambibi commented May 6, 2026

ouf que 2 conflits ahah

iambibi added 3 commits May 7, 2026 20:59
…espace-injector

# Conflicts:
#	src/main/java/fr/openmc/core/ListenersManager.java
#	src/main/java/fr/openmc/core/features/animations/AnimationsManager.java
#	src/main/java/fr/openmc/core/features/dream/listeners/others/SingularityCraftListener.java
#	src/main/java/fr/openmc/core/features/dream/registries/items/consumable/ChipsLait2Margouta.java
@gab4000
Copy link
Copy Markdown
Contributor

gab4000 commented May 7, 2026

Pourquoi ne pas enlever les méthodes getRarety et juste utiliser getMeta().getRarety() ?

@gab4000
Copy link
Copy Markdown
Contributor

gab4000 commented May 7, 2026

Et pareil pour les autres méthodes personnalisées

@iambibi
Copy link
Copy Markdown
Member Author

iambibi commented May 7, 2026

Pourquoi ne pas enlever les méthodes getRarety et juste utiliser getMeta().getRarety() ?

les métas doivent etre lire dans le bootstrap, c'est a dire avant que le plugin se lance.
c'est juste plus simple de lire getRarity au lieu de getMeta().getRarity().
Mais cependant c'est vrai que ça sert a rien de repeter 60 fois la meme chose

@iambibi
Copy link
Copy Markdown
Member Author

iambibi commented May 7, 2026

Et pareil pour les autres méthodes personnalisées

hein?

@gab4000
Copy link
Copy Markdown
Contributor

gab4000 commented May 7, 2026

les métas doivent etre lire dans le bootstrap, c'est a dire avant que le plugin se lance.
c'est juste plus simple de lire getRarity au lieu de getMeta().getRarity().
Mais cependant c'est vrai que ça sert a rien de repeter 60 fois la meme chose

Doivent être lire ? Le bac de français en sueur mdr

@gab4000
Copy link
Copy Markdown
Contributor

gab4000 commented May 7, 2026

hein?

Deux !
Nan en vrai je parle des autres méthodes genre getTransferable

@iambibi
Copy link
Copy Markdown
Member Author

iambibi commented May 7, 2026

les métas doivent etre lire dans le bootstrap, c'est a dire avant que le plugin se lance.
c'est juste plus simple de lire getRarity au lieu de getMeta().getRarity().
Mais cependant c'est vrai que ça sert a rien de repeter 60 fois la meme chose

Doivent être lire ? Le bac de français en sueur mdr

oui bah hein

@iambibi
Copy link
Copy Markdown
Member Author

iambibi commented May 7, 2026

Et pareil pour les autres méthodes personnalisées

pour les memes raisons que getRarity, deux minutes je fini un truc la

@iambibi iambibi marked this pull request as ready for review May 7, 2026 20:31
@iambibi
Copy link
Copy Markdown
Member Author

iambibi commented May 7, 2026

A review, actuellement j'attends la PR afin d'avancer sur ma lignée de choses sur items adder, (ajouter un logger, ajouter un bloc de 2 bloc)

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

Labels

📦 Feature Ajout d'une fonctionnalité

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FEAT] Injecter des namespace item adder

2 participants