You are viewing a plain text version of this content. The canonical link for it is here.
Posted to jetspeed-dev@portals.apache.org by David Sean Taylor <da...@bluesunrise.com> on 2001/11/12 05:34:48 UTC

RE: cvs commit:jakarta-jetspeed/src/java/org/apache/jetspeed/om/profileBaseProfileLocator.java QueryLocator.java

> >                       BaseJetspeedAnonProfilePeer.java
> >                       BaseJetspeedGroupProfilePeer.java
> >                       BaseJetspeedRoleProfilePeer.java
> >                       BaseJetspeedUserProfilePeer.java

The 4 files above were originally generated, but they are modified.
We added our business model logic there. Thats how its supposed to work and
Torque will not overwrite them on subsequent code generations.
As for the 4 other files, I will look into modifying the build to generate
them automatically.

-- David



> -----Original Message-----
> From: Jon Stevens [mailto:jon@latchkey.com]
> Sent: Sunday, November 11, 2001 8:11 PM
> To: jetspeed-dev
> Subject: Re: cvs
> commit:jakarta-jetspeed/src/java/org/apache/jetspeed/om/profil
> eBaseProfi
> leLocator.java QueryLocator.java
>
>
> on 11/11/01 7:37 PM, "taylor@apache.org" <ta...@apache.org> wrote:
>
> >                       BaseJetspeedAnonProfile.java
> >                       BaseJetspeedAnonProfilePeer.java
> >                       BaseJetspeedGroupProfile.java
> >                       BaseJetspeedGroupProfilePeer.java
> >                       BaseJetspeedRoleProfile.java
> >                       BaseJetspeedRoleProfilePeer.java
> >                       BaseJetspeedUserProfile.java
> >                       BaseJetspeedUserProfilePeer.java
>
> -1. You shouldn't be checking in the generated base classes either.
>
> -jon
>
>
> --
> To unsubscribe, e-mail:
> <ma...@jakarta.apache.org>
> For additional commands, e-mail:
> <ma...@jakarta.apache.org>
>
>



--
To unsubscribe, e-mail:   <ma...@jakarta.apache.org>
For additional commands, e-mail: <ma...@jakarta.apache.org>

Re: cvscommit:jakarta-jetspeed/src/java/org/apache/jetspeed/om/profileBaseProfi leLocator.java QueryLocator.java

Posted by Jon Stevens <jo...@latchkey.com>.
on 11/11/01 9:12 PM, "David Sean Taylor" <da...@bluesunrise.com> wrote:

> I will change the jetspeed build to automatically generate the
> base om classes.
> 
> -- David

Thank you.

-jon


--
To unsubscribe, e-mail:   <ma...@jakarta.apache.org>
For additional commands, e-mail: <ma...@jakarta.apache.org>

RE: cvscommit:jakarta-jetspeed/src/java/org/apache/jetspeed/om/profileBaseProfileLocator.java QueryLocator.java

Posted by David Sean Taylor <da...@bluesunrise.com>.
If you are looking for the answer "Because I am ignorant of how Torque
works" - well sorry, but I do understand how Torque uses base-generated
classes and business objects. I also know that its common practice not to
checkin generated files.

However, with all the problems I went thru today with dependencies between
projects, I wanted to get the files checked in so that others could work
with me on the problem. Once the problems I am experienced with torque are
worked out, I will change the jetspeed build to automatically generate the
base om classes.

-- David


> -----Original Message-----
> From: Jon Stevens [mailto:jon@latchkey.com]
> Sent: Sunday, November 11, 2001 9:02 PM
> To: jetspeed-dev
> Subject:
> Re:cvscommit:jakarta-jetspeed/src/java/org/apache/jetspeed/om/
> profileBas
> eProfileLocator.java QueryLocator.java
>
>
> on 11/11/01 8:59 PM, "David Sean Taylor"
> <da...@bluesunrise.com> wrote:
>
> > Yes, I misread your email. The 4 files I was referring to are
> > Jetspeed*ProfilePeer.java - not the base classes.
> >
> > -- David
>
> So, then why did you check the Base classes into CVS?
>
> -jon
>
>
> --
> To unsubscribe, e-mail:
> <ma...@jakarta.apache.org>
> For additional commands, e-mail:
> <ma...@jakarta.apache.org>
>
>



--
To unsubscribe, e-mail:   <ma...@jakarta.apache.org>
For additional commands, e-mail: <ma...@jakarta.apache.org>

Re: cvscommit:jakarta-jetspeed/src/java/org/apache/jetspeed/om/profileBaseProfi leLocator.java QueryLocator.java

Posted by Jon Stevens <jo...@latchkey.com>.
on 11/11/01 8:59 PM, "David Sean Taylor" <da...@bluesunrise.com> wrote:

> Yes, I misread your email. The 4 files I was referring to are
> Jetspeed*ProfilePeer.java - not the base classes.
> 
> -- David

So, then why did you check the Base classes into CVS?

-jon


--
To unsubscribe, e-mail:   <ma...@jakarta.apache.org>
For additional commands, e-mail: <ma...@jakarta.apache.org>

RE: cvscommit:jakarta-jetspeed/src/java/org/apache/jetspeed/om/profileBaseProfileLocator.java QueryLocator.java

Posted by David Sean Taylor <da...@bluesunrise.com>.
> No. You are wrong. You put the business logic into the BO
> classes, not the
> Base classes.

Yes, I misread your email. The 4 files I was referring to are
Jetspeed*ProfilePeer.java - not the base classes.

-- David

> -----Original Message-----
> From: Jon Stevens [mailto:jon@latchkey.com]
> Sent: Sunday, November 11, 2001 8:38 PM
> To: jetspeed-dev
> Subject: Re:
> cvscommit:jakarta-jetspeed/src/java/org/apache/jetspeed/om/pro
> fileBasePr
> ofileLocator.java QueryLocator.java
>
>
> on 11/11/01 8:34 PM, "David Sean Taylor"
> <da...@bluesunrise.com> wrote:
>
> > The 4 files above were originally generated, but they are modified.
> > We added our business model logic there. Thats how its
> supposed to work and
> > Torque will not overwrite them on subsequent code generations.
> > As for the 4 other files, I will look into modifying the
> build to generate
> > them automatically.
> >
> > -- David
>
> No. You are wrong. You put the business logic into the BO
> classes, not the
> Base classes.
>
> -jon
>
>
> --
> To unsubscribe, e-mail:
> <ma...@jakarta.apache.org>
> For additional commands, e-mail:
> <ma...@jakarta.apache.org>
>
>



--
To unsubscribe, e-mail:   <ma...@jakarta.apache.org>
For additional commands, e-mail: <ma...@jakarta.apache.org>

Re: cvs commit:jakarta-jetspeed/src/java/org/apache/jetspeed/om/profileBaseProfileL ocator.java QueryLocator.java

Posted by Jon Stevens <jo...@latchkey.com>.
on 11/11/01 8:34 PM, "David Sean Taylor" <da...@bluesunrise.com> wrote:

> The 4 files above were originally generated, but they are modified.
> We added our business model logic there. Thats how its supposed to work and
> Torque will not overwrite them on subsequent code generations.
> As for the 4 other files, I will look into modifying the build to generate
> them automatically.
> 
> -- David

No. You are wrong. You put the business logic into the BO classes, not the
Base classes.

-jon


--
To unsubscribe, e-mail:   <ma...@jakarta.apache.org>
For additional commands, e-mail: <ma...@jakarta.apache.org>