search

FelipeSulser / ServiciosSociales

Para SistInfoInet
3 stars 0 forks source link

Felipe cambios relaciones #15

Closed FelipeSulser closed 9 years ago

FelipeSulser commented 9 years ago

Fix importante, por favor hagan merge.

HaritzPuerto commented 9 years ago

@FelipeSulser Puedes poner aquí los issue que arregla? eg: fixes #NúmeroIssue Voy a mirarlo

FelipeSulser commented 9 years ago

No arregla ninguno existente, son todos nuevos que he encontrado: Por ejemplo que faltaban muchas not null, el dni no tenía unique etc Además de las not null de las relaciones Y tambien que la relacion Ciudadano - Expediente FALTABA

estebanpw commented 9 years ago

Me estoy bajando el netbeans en mi máquina

cuando lo setee up mergeraré, aunque seguro otro lo hace antes

HaritzPuerto commented 9 years ago

@FelipeSulser vivienda actual no debería estar en expediente?

HaritzPuerto commented 9 years ago

@FelipeSulser Expediente.java tiene: @OneToMany //no le falta un (mappedBy="expediente") @JoinColumn //se puede quitar no? private List intervenciones;

Intervenciones.java tiene:

@ManyToOne
private Expediente expediente;
HaritzPuerto commented 9 years ago

Intervecion.java @ManyToOne @JoinColumn(nullable=false) private Expediente expediente;

Expediente.java @OneToMany(mappedBy="expediente") private List intervenciones; @FelipeSulser

HaritzPuerto commented 9 years ago

Ya he mirado todos los commits. Me parece correcto. Voy a mirar si falta algo, hacer una prueba en mi pc y acepto el pull request

HaritzPuerto commented 9 years ago

Hay que hacer el cambio #18 así que cierro la pull request, lo cambio y hago una pull request