Re: [Jaxor-devel] Re: Associating Lists
Brought to you by:
mrettig
Menu
▾
▴
Re: [Jaxor-devel] Re: Associating Lists
|
From: Carl F. <car...@gm...> - 2004-08-09 14:39:46
|
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
> > > > > > 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
> > > > >
> > > >
> > > >
> > > >
> > > -------------------------------------------------------
> > > > This SF.Net email is sponsored by OSTG. Have you
> > > > noticed the changes on
> > > > Linux.com, ITManagersJournal and NewsForge in the
> > > > past few weeks? Now,
> > > > one more big change to announce. We are now OSTG-
> > > > Open Source Technology
> > > > Group. Come see the changes on the new OSTG site.
> > > > www.ostg.com
> > > > _______________________________________________
> > > > Jaxor-devel mailing list
> > > > Jax...@li...
> > > >
> > > https://lists.sourceforge.net/lists/listinfo/jaxor-devel
> > > >
> > >
> > >
> > > __________________________________
> > > Do you Yahoo!?
> > > Y! Messenger - Communicate in real time. Download now.
> > > http://messenger.yahoo.com
> > >
> > >
> > >
> > >
> > > -------------------------------------------------------
> > > This SF.Net email is sponsored by OSTG. Have you noticed the changes on
> > > Linux.com, ITManagersJournal and NewsForge in the past few weeks? Now,
> > > one more big change to announce. We are now OSTG- Open Source Technology
> > > Group. Come see the changes on the new OSTG site. www.ostg.com
> > > _______________________________________________
> > > Jaxor-devel mailing list
> > > Jax...@li...
> > > https://lists.sourceforge.net/lists/listinfo/jaxor-devel
> > >
> >
>
|
