Michael,
It appears that a similar change has already been made in cvs. Your patch
was to add this:
<cmp-field>
<field-name><XDtClass:classTagValue
tagName="jboss:cmp-field" paramName="field-name"/></field-name>
<column-name><XDtClass:classTagValue
tagName="jboss:cmp-field" paramName="column-name"/></column-name>
+ <XDtClass:ifHasClassTag tagName="jboss:cmp-field"
+ paramName="sql-type">
+ <sql-type><XDtClass:classTagValue
+
tagName="jboss:cmp-field" paramName="sql-type"/></sql-type>
+ </XDtClass:ifHasClassTag>
+ <XDtClass:ifHasClassTag tagName="jboss:cmp-field"
+ paramName="jdbc-type">
+ <jdbc-type><XDtClass:classTagValue
+
tagName="jboss:cmp-field" paramName="jdbc-type"/></jdbc-type>
+ </XDtClass:ifHasClassTag>
</cmp-field>
</XDtClass:forAllClassTags>
but cvs already has:
<cmp-field>
<field-name><XDtMethod:propertyName/></field-name>
<XDtMethod:ifHasMethodTag tagName="jboss:column-name">
<column-name><XDtMethod:methodTagValue
tagName="jboss:column-name" paramName="name" paramNum="0"/></column-name>
</XDtMethod:ifHasMethodTag>
<XDtMethod:ifHasMethodTag tagName="jboss:sql-type">
<sql-type><XDtMethod:methodTagValue
tagName="jboss:sql-type" paramName="type" paramNum="0"/></sql-type>
</XDtMethod:ifHasMethodTag>
<XDtMethod:ifHasMethodTag tagName="jboss:jdbc-type">
<jdbc-type><XDtMethod:methodTagValue
tagName="jboss:jdbc-type" paramName="type" paramNum="0"/></jdbc-type>
</XDtMethod:ifHasMethodTag>
</cmp-field>
(sorry about the formatting). So to me it looks as if its just a
difference in syntax (and the level at which tags are specified). Seeing
as though thats already in cvs, I'm hesitant to remove it.
Could you have a look at it and let me know if it doesn't achieve your
means. If it doesn't, or you think its not the best way to do it, I'd be
happy to change it after discussion.
cheers
dim
On Fri, 16 Nov 2001, Ara Abrahamian wrote:
> Oops, you're right Michael. I closed the ones I applied. I guess Dmitri
> will apply your patch (I'm not a jboss guru, that's why I didn't apply
> your patch).
>
> Ara.
>
> > -----Original Message-----
> > From: [EMAIL PROTECTED] [mailto:xdoclet-devel-
> > [EMAIL PROTECTED]] On Behalf Of Michael Larionov
> > Sent: Monday, November 19, 2001 9:10 PM
> > To: Ara Abrahamian; [EMAIL PROTECTED]
> > Subject: Re: [Xdoclet-devel] v1.1
> >
> > I noticed there are 9 open patch requests on the SourceForge page:
> > http://sourceforge.net/tracker/index.php?group_id=31602&atid=402706
> > and among them the patch I particular care for #480511.
> >
> > Thanks!
> >
> > Michael.
> >
> > ----- Original Message -----
> > From: "Ara Abrahamian" <[EMAIL PROTECTED]>
> > To: <[EMAIL PROTECTED]>
> > Sent: Friday, November 16, 2001 11:43 AM
> > Subject: [Xdoclet-devel] v1.1
> >
> >
> > > Yesterday I fixed many bugs, applied patches, etc. AFAIK the only
> items
> > > remaining are:
> > >
> > > - [ #472764 ] Wrong class names in ejb- jar.xml
> > > Or making ejb subtasks optional.
> > > - [ #480307 ] NullPointerException
> > > I have no idea about this.
> > > - [ #482902 ] StringIndexOutOfBoundsException
> > > I have no idea about this too!
> > >
> > > The rest is updating javadocs and maybe polishing docs/etc and
> updating
> > > <document/> in core/script/build.xml. Some docs are also missing
> > > (webwork/jmx/weblogic-web).
> > >
> > > I think we'll be ready for v1.1 in a few days. Guys! Let's make this
> > > release ASAP, I really want to concentrate on the new xjavadoc
> module
> > > and help Aslak, it's huge and interesting.
> > >
> > > Ara.
> > >
> > >
> > >
> > > _________________________________________________________
> > > Do You Yahoo!?
> > > Get your free @yahoo.com address at http://mail.yahoo.com
> > >
> > >
> > > _______________________________________________
> > > 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
>
>
> _________________________________________________________
> Do You Yahoo!?
> Get your free @yahoo.com address at http://mail.yahoo.com
>
>
> _______________________________________________
> 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
