Constructor is Method here right ? Anyway yes I have to have a deeper look for those two. Regards Vincent
> -----Original Message----- > From: Aslak Hellesøy [mailto:[EMAIL PROTECTED]] > Sent: mardi 23 avril 2002 19:47 > To: Vincent Harcq; [EMAIL PROTECTED] > Cc: [EMAIL PROTECTED] > Subject: RE: [Xdoclet-devel] xDoclet broken > > > If you take a look at it, it would be nice to do it in such a > way that it works for fields and constructors too. The > following method in DocletSupport is no good: > > public static XTag getCurrentTag() > { > if( currentMethodTag != null ) > return currentMethodTag; > else > return currentClassTag; > } > > -as it ignores field/method tags. I don't think constructor > and field tags are used a lot, but our core should support it. > > /Aslak > > > -----Original Message----- > > From: [EMAIL PROTECTED] > > [mailto:[EMAIL PROTECTED]]On Behalf > Of Vincent > > Harcq > > Sent: 23. april 2002 00:54 > > To: [EMAIL PROTECTED] > > Cc: [EMAIL PROTECTED] > > Subject: Re: [Xdoclet-devel] xDoclet broken > > > > > > I am on it > > It works ok locally > > Basically I extend what Aslak changes to support test cases > plus last > > patches I have done recently. The mistake underlying is imho in > > getTagValue that needs a for_type up to the end else > > getCurrentMethod/ClassTag mixing each others when having > > forAllMethodTag inside forAllClassTag or-and the opposite. > This will > > be for tomorrow. Refactoring will be needed as it really becomes a > > headache. Vincent > > > > > Hello, folks. I'll do a more accurate report when I get home. But > > > I'm sure there is something very broken with xDoclet now. > Here are > > > the > > > symptoms: > > > > > > any tag that uses delimiters is not working. I've traced this bug > > > down to the modifications done in this patch: > > > > > > http://cvs.sourceforge.net/cgi- > > bin/viewcvs.cgi/xdoclet/xdoclet/core/src/xdoclet/XDocletTagSupport > > .java.diff > > ?r1=1.36&r2=1.37 > > > > > > Taka a look at line 404. This modification has to be undone. With > > > this is place, getTagValue() doesn't handle: > > > > > > 1) Multiple tag/parameters alternatives > > > 2) Delimited tag values > > > 3) the @jboss:table-name "abc" case > > > > > > This was done by Vincent, right? Vincent, please correct > me if I'm > > > being unfair to you, but I think this modification really > could not > > > be done. > > > > > > There are sure other changes that broke something. For > example, my > > > ejb-jar.xml <query> elements are completly broken. It's > messing up > > > parameters. Again, I'll do some more detailed > report/fixing when I > > > get home. > > > > > > -- > > > Pazu > > > > > > > > > _______________________________________________ > > > 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 > > > _______________________________________________ Xdoclet-devel mailing list [EMAIL PROTECTED] https://lists.sourceforge.net/lists/listinfo/xdoclet-devel
