Re: [Xdoclet-user] CMP VO setter generation in 1.2.x

M.-Leander Reimer Mon, 22 Nov 2004 07:20:55 -0800

Hi,

the relevant XDT file is:

xdoclet-1.2.2\modules\ejb\src\xdoclet\modules\ejb\entity\resources\entity-value.xdt

I don't know the line, but look for the following code:

<XDtMethod:setterMethod/>( valueHolder.<XDtMethod:getterMethod/>() );

and replace it with:

if (valueHolder.<XDtMethod:propertyName/>HasBeenSet()) {
    <XDtMethod:setterMethod/>( valueHolder.<XDtMethod:getterMethod/>());
}

That should do it.

Regards,
Leander


Mohammed Khaffar schrieb:

How do you fix this bug? Do I have to modify the .xdt file? If so where do I
find that file?

A newbie to XDoclet
Thanks

-----Original Message-----
From: [EMAIL PROTECTED]
[mailto:[EMAIL PROTECTED] Behalf Of Mike
Moulton
Sent: 14 November 2004 03:00
To: [EMAIL PROTECTED]
Subject: Re: [Xdoclet-user] CMP VO setter generation in 1.2.x


This is a know issue that currently has to open issues in Jira (XDT-158
XDT-872
http://opensource.atlassian.com/projects/xdoclet/browse/XDT-872).

This has been a known issue for a while, but it hasn't seem to gotten
any attention. Personally I always fix this bug in each new release
before I can use xdoclet.

Can anyone in the know comment on if this is actually a bug, and when
it might be able to be fixed.

Thanks,
-- Mike


On Nov 12, 2004, at 4:56 PM, Craig Crutcher wrote:

So it looks like the VO setter created in the xdoclet-generated CMP
file doesnt check for dirty fields. The code generated is of the form:

  public void setYourOwnVO( YourOwnVO valueHolder )
  {
     setFoo( valueHolder.getFoo() );
     setBar( valueHolder.getBar() );
  }

The setter should check to make sure a field in the VO has been set
before attempting to use it on the entity. It seems the code should
look something like:

  public void setYourOwnVO( YourOwnVO valueHolder )
  {
     if( valueHolder.fooHasBeenSet() )
        setFoo( valueHolder.getFoo() );
     if( valueHolder.barHasBeenSet() )
        setBar( valueHolder.getBar() );
  }

Is the current coding intentional? Just checking since its been this
way for quite a while and no one has complained. -cc.


-------------------------------------------------------
This SF.Net email is sponsored by: InterSystems CACHE
FREE OODBMS DOWNLOAD - A multidimensional database that combines
robust object and relational technologies, making it a perfect match
for Java, C++,COM, XML, ODBC and JDBC. www.intersystems.com/match8
_______________________________________________
xdoclet-user mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/xdoclet-user

  :  mike moulton
  :  meltmedia
  :  1429 north 1st street
  :  phoenix  az  85004
  :
  :  [EMAIL PROTECTED]
  :  mmoulton66    | aim
  :  602.340.9440 | ofc
  :  602.432.2568 | cel
  :  602.340.1003 | fax
  :
  :  meltmedia.com

-------------------------------------------------------
This SF.Net email is sponsored by: InterSystems CACHE
FREE OODBMS DOWNLOAD - A multidimensional database that combines
robust object and relational technologies, making it a perfect match
for Java, C++,COM, XML, ODBC and JDBC. www.intersystems.com/match8
_______________________________________________
xdoclet-user mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/xdoclet-user

------------------------------------------------------- SF email is sponsored by - The IT Product Guide Read honest & candid reviews on hundreds of IT Products from real users. Discover which products truly live up to the hype. Start reading now. http://productguide.itmanagersjournal.com/ _______________________________________________ xdoclet-user mailing list [EMAIL PROTECTED] https://lists.sourceforge.net/lists/listinfo/xdoclet-user

Re: [Xdoclet-user] CMP VO setter generation in 1.2.x

Reply via email to