Hi Teresa,

I'm confused. What does this comment mean?

Thanks,

Craig

On Nov 2, 2007, at 10:42 PM, Teresa Kan (JIRA) wrote:


[ https://issues.apache.org/jira/browse/OPENJPA-357? page=com.atlassian.jira.plugin.system.issuetabpanels:comment- tabpanel#action_12539796 ]

Teresa Kan commented on OPENJPA-357:
------------------------------------

 *private* *boolean* includes(FieldMetaData fmd) {
   String fmdName = fmd.getFullName(*false*);
*   if* ((fmd.isInDefaultFetchGroup() && hasFetchGroup(FetchGroup.*
NAME_DEFAULT*))|| hasFetchGroup(FetchGroup.*NAME_ALL*)


*return* *true*;

// check any LoadFetchGroup associates with this field. We need to

// include this LoadFetchGroup for fetch

String[] fgs = fmd.getCustomFetchGroups();

*for* (*int* i = 0; i < fgs.length; i++) {

*if* (hasFetchGroup(fgs[i])) {

String fg = fmd.getLoadFetchGroup(); //kan

*if* (fg != *null*) { //kan

*if* (!hasFetchGroup(fg)) { //kan

addFetchGroup(fg); //kan

}

}

*return* *true*;

}

}

*return* *false*;

}


On 9/8/07, Pinaki Poddar (JIRA) <[EMAIL PROTECTED]> wrote:


Use case where FetchGroup A includes FecthGroup B is not honoured by fetch operations --------------------------------------------------------------------- ----------------

                Key: OPENJPA-357
URL: https://issues.apache.org/jira/browse/ OPENJPA-357
            Project: OpenJPA
         Issue Type: Bug
   Affects Versions: 0.9.7, 1.0.0
           Reporter: Pinaki Poddar
           Assignee: Pinaki Poddar
            Fix For: 1.0.1, 1.1.0


Given:
Fetch group A contains {x,y}
Fetch group B contains {z}.
Fetch group A includes Fetch group B
Assertions:
1. Any fetch operation with fetch group A should fetch {x,y,z}.
2. Any fetch operation with fetch group B should fetch {z}.
Currently,
Assertion #2 fails.

--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


Craig Russell
Architect, Sun Java Enterprise System http://java.sun.com/products/jdo
408 276-5638 mailto:[EMAIL PROTECTED]
P.S. A good JDO? O, Gasp!

Attachment: smime.p7s
Description: S/MIME cryptographic signature

Reply via email to