Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[JAVA] Enumérations représentant totalement l'entité #401

Open
wants to merge 4 commits into
base: develop
Choose a base branch
from

Conversation

dchallas
Copy link

@dchallas dchallas commented Oct 21, 2024

Permet d'avoir toutes les données statiques dans l’énumération.
Attention, le tri des values a été supprimé pour respecter l'ordre dans les fichiers TMD, ceci afin de permettre la gestion des cycles (par exemple les parents).
La régression attendue peut se produire pour les projets qui utilisent l'ordinal de l'enum comme PK (comportement par defaut d'hibernate), mais il est préférable dans ce cas que le projet gère l'ordre, la solution étant donc de trier les values dans le TMD.

     var refs = GetAllValues(classe)
 --           .OrderBy(x => x.Name, StringComparer.Ordinal)
            .ToList();

@gideruette gideruette closed this Oct 21, 2024
@gideruette gideruette reopened this Oct 21, 2024
@gideruette
Copy link
Collaborator

gideruette commented Oct 21, 2024

Le nom de propriété de l'association n'est pas correct dans l'enum : typeDroit au lieu de typeDroitCode, et il manque l'import de typeDroitCode dans droitCode (l'import de l'enum cible dans l'enum source)

@dchallas
Copy link
Author

dchallas commented Oct 23, 2024

Le nom de propriété de l'association n'est pas correct dans l'enum : typeDroit au lieu de typeDroitCode,

Correction commitée

et il manque l'import de typeDroitCode dans droitCode (l'import de l'enum cible dans l'enum source)

dans l'exemple par default, pas de soucis car dans le même package, mais effectivement => Correction commitée

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.

2 participants