Re: [Jaxor-devel] Re: Associating Lists
Brought to you by:
mrettig
Menu
▾
▴
Re: [Jaxor-devel] Re: Associating Lists
|
From: Michael R. <mik...@ya...> - 2004-08-09 15:26:34
|
This seems like a sensible approach. Though, I don't
think it should need to implement EntityInterface. If
it does, then that really smells.
You may want to create a new listener interface for a
list (i.e. ListChangeListener) to listen for additions
and removals. This would allow you to set/unset the
keys properly.
Mike
--- Carl Fyffe <car...@gm...> wrote:
> Is the answer to have EntityListAdapterBase
> implement ValueChangeListener?
>
> I am afraid that if that is the answer, then it will
> also have to
> implement EntityInterface. Is that the case?
>
> On Mon, 9 Aug 2004 01:24:43 -0400, Carl Fyffe
> <car...@gm...> wrote:
> > How do I watch the List to see if it changes?
> That seems to be what
> > needs to be done to fix
> testAddingAnEntityToAListAlreadySetOnEntity()
> > I keep thinking that a change to List.vm may fix
> the problem, but I
> > haven't figured out how. I took a swing and seem
> to have struck out.
> > Any pointers for my next time at bat?
> >
> >
> >
> > On Sun, 8 Aug 2004 12:14:11 -0400, Carl Fyffe
> <car...@gm...> wrote:
> > > Thank you! I will get right on them. I also have
> some news on the ManyToMany.
> > >
> > > ManyToMany does not work anything like
> OneToMany. Yes at the heart it
> > > is just two OneToMany's pointing in different
> directions, but they
> > > point in the wrong directions.
> > >
> > > This is REALLY what I want to have work:
> > > group.setUserList(members);
> > >
> > > But that means that UserList will have to know
> about GroupMember and I
> > > don't see anyway to make that happen without
> making a change to the
> > > DTD and thus the generated .java files. I am
> trying to avoid that if
> > > possible.
> > >
> > > Next, I tried this:
> > > GroupMemberEntity gme = new GroupMemberBase();
> > > gme.setGroupEntity(group);
> > > gme.setUserList(members);
> > >
> > > But like I said earlier, the OneToMany's point
> in the wrong direction.
> > > User's don't have one GroupMember, they have
> lots. So if you have a
> > > user and you want to set a list of GroupMembers,
> that works, but we
> > > have a list of Users. This is the other thought
> that I have had:
> > >
> > > group.setUserList(members,
> GroupMemberEntity.class);
> > >
> > > Pass in some reference that would allow the
> setter to know what to
> > > link with. Again that would require a change in
> the DTD and I think
> > > the other way is cleaner so I would prefer to go
> with the other idea
> > > if I have to change the DTD.
> > >
> > > Can you give me a hint at which would be a
> better "interface" for the developer?
> > >
> > > I will start working on your tests and continue
> thinking about the above.
> > >
> > > On Sat, 7 Aug 2004 19:34:30 -0700 (PDT), Michael
> Rettig
> > >
> > >
> > > <mik...@ya...> wrote:
> > > > Carl,
> > > >
> > > > That was quick. I added a couple new failing
> tests. If
> > > > we have a list that is already set on on
> entity,
> > > > then we need to set the foreign fields for
> each entity
> > > > that is added to the list. Also, if we change
> the
> > > > value of the key on the source entity, then we
> need to
> > > > update the keys on the list entities.
> > > >
> > > > I checked in the tests. Do you want to take a
> stab at
> > > > them?
> > > >
> > > > --- Carl Fyffe <car...@gm...> wrote:
> > > >
> > > > > 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
>
=== message truncated ===
__________________________________
Do you Yahoo!?
New and Improved Yahoo! Mail - 100MB free storage!
http://promotions.yahoo.com/new_mail
|
