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

[TASK] - Modificar la clase org.jugni.apps.pico.modelos.Identificador #46

Closed
jselvamadrigal opened this issue Jun 17, 2020 · 2 comments
Closed
Labels
bug Something isn't working

Comments

@jselvamadrigal
Copy link
Collaborator

Las entidades deben de tener identificadores propios estos pueden ser de tipo simple o compuesto, por lo tanto la clase Identificador no cumple con ese requisito y su utilidad no esta definida claramente.

La clase también define el GeneratedValue con una estrategia de SEQUENCE lo que provoca que todas las entidades que la hereden tengan un identificador no continuo.

Al remover la clase se modificaran los modelos que extiendan de dicha clase y donde exista una referencia a la misma.

@jselvamadrigal jselvamadrigal added the bug Something isn't working label Jun 17, 2020
@gacsnic
Copy link
Collaborator

gacsnic commented Jun 18, 2020

hasta el dia de hoy no se había planteado identificadores de tipo compuesto, se tendría que considerar si es necesario establecer en las entidades identificadores compuesto.

La estrategia de GeneratedValue se puede sobrescribir en cada identidad o modificar la estrategia de la clase padre.

@jselvamadrigal jselvamadrigal changed the title [TASK] - Eliminar clase org.jugni.apps.pico.modelos.Identificador [TASK] - Modificar la clase org.jugni.apps.pico.modelos.Identificador Jun 20, 2020
@berroteran
Copy link
Member

este issue se cierra por falta de coherencia.

La descripción no tiene nada de que ver en el titulo,
Lo que describen en el ticket ya se está tratando en otro issue.

recuerden un Ticket por Issue, una sola característica o un solo problema.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants