In fact somebody point me the jndiname local/remote/both problem last week. Did not had the time to commit yet. Also agree for jboss dd.
Please do if you can. Thanks >===== Original Message From David Budworth <[EMAIL PROTECTED]> ===== >Howdy Vincent, > >I was also working on the same problem, and just noticed you checked it >in. > >I am curious. HomeTagsHandler::jndiName will "create" a LocalJNDI name >if it isn't already set. > >So making the template check for existance of the class tag isn't quite >enough. > >What I was going to change it to was: ><XDtEjb:ifRemoteEjb> > <jndi-name><XDtEjbHome:jndiName type="remote"/></jndi-name> ></XDtEjb:ifRemoteEjb> ><XDtEjb:ifLocalEjb> > <local-jndi-name><XDtEjbHome:jndiName type="local"/></local-jndi-name> ></XDtEjb:ifLocalEjb> > >But before doing that, I realized that the jndiName function makes some >bad assumptions as to how to "guess" the jndi names. > >One being that, depending on the ejb:bean view-type parameter, the >generated JNDI name changes >ie: >Given EJB named "Account" > >view-type="remote": >jndiName = "Account" > >view-type="local": >jndiName = "Account" > >view-type="both": >jndiName = "Account" >localJndiName = "AccountLocal" > > >I believe that this is wrong, since if you change the view type from >local to "both", you end up with your jndi name getting changed (thus >breaking any code that calls a "local" only method (ie. CMR)) > >What I think it should be, is: >jndi-local-name || compName(clazz,"local") >and >jndi-name || compName(clazz,"remote") > >This keeps the JNDI names from magically changing on you when you change >the view-type of an ejb. > > >I've already made these changes, and they do work (at least for jboss >they work). > >I just wanted to avoid stepping on toes here, as well as make sure I'm >not completly on crack as to my view of how it should work. > >Thanks, > >-David > >p.s. If this didn't make any sense, tell me, and I'll try to explain my >point better after some sleep. > >On Tue, 15 Jan 2002, Vincent Harcq wrote: > >> Update of /cvsroot/xdoclet/xdoclet/core/resources/xdoclet/ejb/vendor >> In directory usw-pr-cvs1:/tmp/cvs-serv1453 >> >> Modified Files: >> jboss_xml.j >> Log Message: >> [JBoss] Add local-jndi-name for session/entity bean DD >> >> _______________________________________________ >> Xdoclet-devel mailing list >> [EMAIL PROTECTED] >> https://lists.sourceforge.net/lists/listinfo/xdoclet-devel _______________________________________________ Xdoclet-devel mailing list [EMAIL PROTECTED] https://lists.sourceforge.net/lists/listinfo/xdoclet-devel
