The following comment has been added to this issue:

     Author: Andrew Stevens
    Created: Sun, 9 Mar 2003 3:51 AM
       Body:
According to the DTD, the primkey-mapping element is optional,
<!ELEMENT entity-deployment (primkey-mapping?, cmp-field-mapping*, finder-method*, 
env-entry-mapping*, ejb-ref-mapping*, resource-ref-mapping*)>
Your patch would always include one regardless of whether it was needed.

However, I believe swapping the lines

       <primkey-mapping>
        <XDtClass:ifDoesntHaveClassTag tagName="ejb:bean" paramName="primkey-field" 
superclasses="true">
          <cmp-field-mapping>
...
          </cmp-field-mapping>
        </XDtClass:ifDoesntHaveClassTag>
       </primkey-mapping>

round to instead be

        <XDtClass:ifDoesntHaveClassTag tagName="ejb:bean" paramName="primkey-field" 
superclasses="true">
       <primkey-mapping>
          <cmp-field-mapping>
...
          </cmp-field-mapping>
       </primkey-mapping>
        </XDtClass:ifDoesntHaveClassTag>

should give the desired behaviour i.e. a primkey-mapping is included if and only if 
there's no single primkey-field specified.  Could you give that a try and let me know 
if it's okay?

---------------------------------------------------------------------
View the issue:

  http://opensource.atlassian.com/projects/xdoclet/secure/ViewIssue.jspa?key=XDT-319


Here is an overview of the issue:
---------------------------------------------------------------------
        Key: XDT-319
    Summary: Incorrect format for EJB compound PK fields in orion-ejb-jar.xml
       Type: Bug

     Status: Assigned
   Priority: Major

 Time Spent: Unknown
   Estimate: 0 minutes

    Project: XDoclet
  Component: Orion Module

   Assignee: xdoclet-devel (Use for new issues)
   Reporter: Daniele Arduini

    Created: Fri, 21 Feb 2003 11:57 AM
    Updated: Fri, 21 Feb 2003 11:59 AM
Environment: xdoclet-cvs HEAD (2003-02-21)

Description:
The generated orion-ejb-jar.xml deployment descriptor is not correct
for Entity EJBs with a compound PK, so the server will ignore any
setup and generate its defaults fields names.

The attached patch works for me, please verify.
Apply from the xdoclet root with:

patch -p0 < xdoclet-orion.patch



---------------------------------------------------------------------
JIRA INFORMATION:
This message is automatically generated by JIRA.

If you think it was sent incorrectly contact one of the administrators:
   http://opensource.atlassian.com/projects/xdoclet/Administrators.jspa

If you want more information on JIRA, or have a bug to report see:
   http://www.atlassian.com/software/jira



-------------------------------------------------------
This SF.net email is sponsored by: Etnus, makers of TotalView, The debugger 
for complex code. Debugging C/C++ programs can leave you feeling lost and 
disoriented. TotalView can help you find your way. Available on major UNIX 
and Linux platforms. Try it free. www.etnus.com
_______________________________________________
xdoclet-devel mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/xdoclet-devel

Reply via email to