> -----Original Message-----
> From: [EMAIL PROTECTED]
> [mailto:[EMAIL PROTECTED]]On Behalf Of Vincent
> Harcq
> Sent: 23. april 2002 19:53
> To: 'Aslak Helles�y'; [EMAIL PROTECTED]
> Cc: [EMAIL PROTECTED]
> Subject: RE: [Xdoclet-devel] xDoclet broken
>
>
> Constructor is Method here right ?
Not quite... In the xjavadoc API (and also in the corresponding javadoc API)
there are 4 different subinterfaces of XMember: XConstructor, XMethod,
XField. Then there is XClass too. The superinterface of all these is
XProgramElement. Check xjavadoc API.
Therefore we need to store currentFieldTag, currentMethodTag,
currentClassTag and currentConstructorTag in DocletSupport. When any of
these are set, we should also set currentTag, and getCurrentTag will just
return currentTag (assuming it has been properly set).
I'm gonna attack the module stuff now, so I don't have time to look at it
ATM...
/Aslak
> 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
_______________________________________________
Xdoclet-devel mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/xdoclet-devel
