@lluismf showed me HibernateOpenmrsObjectDAO which I didnt know existed
(see the discussion from replace occurences of vector with arraylist: )
he suggested to
Make all the Hibernate*DAO extend HibernateOpenmrsObjectDAO Remove all the sessionFactory.getCurrentSession() prefixes. Invoke get/saveOrUpdate/… of the parent. Remove the sessionFactory instance variable because it won’t be used anymore
I just wanted some more discussion on this. Why was this class introduced and not used? Was it a requirement from a module?
Do you also think this should be done? Any thoughts?