I can think of three ways to provide this funtionality: 1: Put it in the properties file as done now. The full name including the methods is then the only way since you never will know if getSession() is the correcy method. Disadvantage is that I somehow do not like this. It feels wrong. It'n not a property it's a method call which should in the ideal world be derived somehow (introspecion?).
2: put it in a merge file. In my opinion this is a cleaner sollution then the first but also splits the generation of the file. The merge then becomes a 'required' item. While as far as I have worked with the merge files, they are intended as an extension and thus provide additional functionality to middlegen. Not core functionality, like the hibernateDAO is. 3. Set it via the gui and store the setting in the prefs file. I put this file under version control so I can recreate the middle/xdoclet generation on the production system. The disadvantage is, that the default will, in most cases not work since they are not 'local' like the other settings stored in the prefs file. So taking everything in consideration, the third option would be my choice. What are your cons and pros? As for the fix in CVS, is there a bugfix release planned, like 2.1.1? I do not want to go into production with a CVS snapshot as you can understand. For development I can opf course live with it. Regards, Wouter -----Original Message----- From: [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED] On Behalf Of Ben Litchfield Sent: 15 January 2005 03:42 To: middlegen-user@lists.sourceforge.net Subject: Re: [Middlegen-user] Hibernate, dao and DBHelper class template error > I run the subtask hibernatDAO in the middlegen task. As mentioned before > on this list. You need to specify the variable "getHibernateSession" when > using hibernate 2.x. (I am using 2.1) > > <hibernateDAO getHibernateSession="${getHibernateSession}.getSession();"/> > > I have a few comments on the template. > First you need to specify the ';'. Which is not very need but okay. I would like to hear your comments on a better way to provide this functionality. Would it be better as a merge file? > The second is that the template produces a java file that is incorrect. This was fixed in CVS a while back but has not been released yet. Ben ------------------------------------------------------- The SF.Net email is sponsored by: Beat the post-holiday blues Get a FREE limited edition SourceForge.net t-shirt from ThinkGeek. It's fun and FREE -- well, almost....http://www.thinkgeek.com/sfshirt _______________________________________________ middlegen-user mailing list middlegen-user@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/middlegen-user ------------------------------------------------------- The SF.Net email is sponsored by: Beat the post-holiday blues Get a FREE limited edition SourceForge.net t-shirt from ThinkGeek. It's fun and FREE -- well, almost....http://www.thinkgeek.com/sfshirt _______________________________________________ middlegen-user mailing list middlegen-user@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/middlegen-user
