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
