Re: [Jaxor-devel] Re: Associating Lists

[Carl F. <car...@gm...>]

I have checked the code in for OneToMany. There are two issues that I
am unhappy about. First, you must have an <entity-ref> in the .jaxor
of the foreign object (see email.jaxor) and for setXXXXXList(list) to
work.  It throws a SQLValidationException and tells you that the
column was expected to have a value, but for some reason this doesn't
"smell" good.

Second, I have added this method to ForeignFieldLazyLoader:

    private boolean isForeignKey(EntityRow targetEntity) {
        for(Iterator iterator =
targetEntity.getForeignFields().iterator(); iterator.hasNext();) {
            ForeignFieldLazyLoader loader = (ForeignFieldLazyLoader)
iterator.next();
                if (loader.getResolver().isUnique())
                return true;
        }
        return false;
    }

The idea is to find out if the targetEntity is related to the
sourceEntity. I have tested against ManyToManyTest and against my new
OneToManyTest and all lights are green.  This method works, and I
don't know why. I was expecting to need to check directly against the
sourceEntity as well, but apparently I don't.

The concern is that it will set the list for the wrong foreign
field...  I am sorry if this seems un-smart, but it seems like magic
to me :-) Can anyone come up with a test that will cause this method
to return true, thus allowing a save to the wrong table?

On Sat, 7 Aug 2004 14:49:55 -0700 (PDT), Michael Rettig
<mik...@ya...> wrote:
> Carl,
> 
> You should have write permissions. I added you to the
> project as a developer.
> 
> http://sourceforge.net/project/memberlist.php?group_id=59712
> 
> Did you checkout the project using ssh?
> 
> http://sourceforge.net/cvs/?group_id=59712
> 
> Mike
> 
> --- Carl Fyffe <car...@gm...> wrote:
> 
> > Since a Many to Many is really just a One to Many on
> > two sides, I
> > thought I would solve the One to Many problem first.
> >  That is done.  I
> > would like to check the code in, but I don't have
> > write permission in
> > the repository.
> >
> > On Sat, 7 Aug 2004 10:08:14 -0700 (PDT), Michael
> > Rettig
> > <mik...@ya...> wrote:
> > >
> > > --- Carl Fyffe <car...@gm...> wrote:
> > >
> > > > Another idea that I have had would be to start
> > from
> > > > the linking
> > > > object.  Put something into the group_members
> > that
> > > > would allow the
> > > > user to create the links.
> > > >
> > > > GroupMemberFinder.newInstances(group, userList);
> > > > // or
> > > > GroupMemberFinder.newInstances(user, groupList);
> > > >
> > > > First would take a list of users and link them
> > to
> > > > the group, the
> > > > second would take a list of groups and link to
> > the
> > > > user. What would
> > > > the xml look like to make this happen....
> > > >
> > > > <list-ref table="user_table" type="fk">
> > > >     <key source="user_name"/>
> > > > </list-ref>
> > > >
> > > > <list-ref table="groups" type="fk">
> > > >     <key source="group_name"/>
> > > > </list-ref>
> > > >
> > > > I like the look of this and will try to get it
> > > > working this weekend.
> > >
> > > Yes, I like the look of this too.  I'm eager to
> > see it
> > > in action.
> > >
> > > Regards,
> > >
> > >
> > > Mike
> > >
> > > __________________________________
> > > Do you Yahoo!?
> > > New and Improved Yahoo! Mail - 100MB free storage!
> > > http://promotions.yahoo.com/new_mail
> > >
> >
> 
>                 
> __________________________________
> Do you Yahoo!?
> Take Yahoo! Mail with you! Get it on your mobile phone.
> http://mobile.yahoo.com/maildemo
>