Daniel, gràcies per anar commitant, estas fent molta feina.
Uns comentaris..
- sobre HibernateEJB i HibernateUtils: quina es la responsabilitat de HibernateEJB? i&nbps; HibernateUtils?&nbps; Diria que les dues tenen la mateixa responsabilitat; gestionar sessions Hibernate. Potser es podria moure els metodes de HibernateEJB a HibernateUtils i prescindir de l'herencia amb HibernateEJB.
- sobre HibernateEJB, no sembla&nbps; que getArxiuDTO i getHistoric no hagin d'estar en aquesta classe. Per que no crear un HistoricEJB i un ArxiuEJB ?
- els tests d'integracio no em funcionen. Sembla que falta spring.cfg.xml
Et semblen bé els canvis? si és així els puc fer jo si prefereixes. Sino, pots aportar el teu punt de vista ..
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
Inicialment varem crear HibernateUtils però després es va crear HibenateEJB per mantenir un poc la similitud amb el back, que està fet així.
Em semblen bé els canvis que comentes. Pots fer-los si vols, no hi ha cap problema.
Sobre els tests, el fitxer spring.cfg.xml ja no existeix, s'empre sac-api-context.xml. A la classe BeanUtils pots veure la ruta del fitxer. De totes formes, aquests tests els hem fet per anar provant el codi que fèiem des del mateix Eclipse, no son un conjunt complet i purista de tests unitaris/integració.
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
http://rolsac.svn.sourceforge.net/viewvc/rolsac?view=revision&revision=593
Falta corregir un error de seguertat al obtenir el nombre d'unitats administratives i les insercions d'estadístiques.
http://rolsac.svn.sourceforge.net/viewvc/rolsac?view=revision&revision=594
Afegit EJB remot per insercions d'estadístiques. S'han mogut els mètodes de EJBUtils a HibernateEJB.
http://rolsac.svn.sourceforge.net/viewvc/rolsac?view=revision&revision=595
Corregit error de seguretat al consultar el nombre d'unitats administratives provocat per incompatibilitat de XDoclet amb tipus enum de Java 1.5.
View and moderate all "tickets13 Discussion" comments posted by this user
Mark all as spam, and block user from posting to "Tickets1.3"
Originally by: ejaen
Daniel, gràcies per anar commitant, estas fent molta feina.
Uns comentaris..
- sobre HibernateEJB i HibernateUtils: quina es la responsabilitat de HibernateEJB? i&nbps; HibernateUtils?&nbps; Diria que les dues tenen la mateixa responsabilitat; gestionar sessions Hibernate. Potser es podria moure els metodes de HibernateEJB a HibernateUtils i prescindir de l'herencia amb HibernateEJB.
- sobre HibernateEJB, no sembla&nbps; que getArxiuDTO i getHistoric no hagin d'estar en aquesta classe. Per que no crear un HistoricEJB i un ArxiuEJB ?
- els tests d'integracio no em funcionen. Sembla que falta spring.cfg.xml
Et semblen bé els canvis? si és així els puc fer jo si prefereixes. Sino, pots aportar el teu punt de vista ..
Hola Enric,
Inicialment varem crear HibernateUtils però després es va crear HibenateEJB per mantenir un poc la similitud amb el back, que està fet així.
Em semblen bé els canvis que comentes. Pots fer-los si vols, no hi ha cap problema.
Sobre els tests, el fitxer spring.cfg.xml ja no existeix, s'empre sac-api-context.xml. A la classe BeanUtils pots veure la ruta del fitxer. De totes formes, aquests tests els hem fet per anar provant el codi que fèiem des del mateix Eclipse, no son un conjunt complet i purista de tests unitaris/integració.
View and moderate all "tickets13 Discussion" comments posted by this user
Mark all as spam, and block user from posting to "Tickets1.3"
Originally by: ejaen
Ok, m'ho apuntaré com a tasca a fer.
La implementació del back antic no és un bon exemple a seguir, desde el meu punt de vista. Té moltes god classes.
Diff:
http://rolsac.sourceforge.net/mantis/view.php?id=369