You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@turbine.apache.org by ep...@apache.org on 2003/08/24 21:38:02 UTC

cvs commit: jakarta-turbine-fulcrum/security/src/java/org/apache/fulcrum/security/adapter/turbine RoleAdapter.java GroupAdapter.java PermissionAdapter.java UserAdapter.java

epugh       2003/08/24 12:38:02

  Modified:    security/src/java/org/apache/fulcrum/security/adapter/turbine
                        RoleAdapter.java GroupAdapter.java
                        PermissionAdapter.java UserAdapter.java
  Log:
  Various changes to deal with tranlating from int's in turbine land to longs in fulcrum land.
  
  Revision  Changes    Path
  1.3       +2 -2      jakarta-turbine-fulcrum/security/src/java/org/apache/fulcrum/security/adapter/turbine/RoleAdapter.java
  
  Index: RoleAdapter.java
  ===================================================================
  RCS file: /home/cvs/jakarta-turbine-fulcrum/security/src/java/org/apache/fulcrum/security/adapter/turbine/RoleAdapter.java,v
  retrieving revision 1.2
  retrieving revision 1.3
  diff -u -r1.2 -r1.3
  --- RoleAdapter.java	23 Aug 2003 15:25:54 -0000	1.2
  +++ RoleAdapter.java	24 Aug 2003 19:38:02 -0000	1.3
  @@ -121,14 +121,14 @@
        */
       public int getId()
       {
  -        return role.getId();
  +        return new Integer(role.getId()+"").intValue();
       }
       /* (non-Javadoc)
        * @see org.apache.turbine.om.security.SecurityEntity#getIdAsObj()
        */
       public Integer getIdAsObj()
       {
  -        return new Integer(role.getId());
  +        return new Integer(role.getId()+"");
       }
       /* (non-Javadoc)
        * @see org.apache.turbine.om.security.SecurityEntity#setId(int)
  
  
  
  1.3       +2 -2      jakarta-turbine-fulcrum/security/src/java/org/apache/fulcrum/security/adapter/turbine/GroupAdapter.java
  
  Index: GroupAdapter.java
  ===================================================================
  RCS file: /home/cvs/jakarta-turbine-fulcrum/security/src/java/org/apache/fulcrum/security/adapter/turbine/GroupAdapter.java,v
  retrieving revision 1.2
  retrieving revision 1.3
  diff -u -r1.2 -r1.3
  --- GroupAdapter.java	23 Aug 2003 15:25:54 -0000	1.2
  +++ GroupAdapter.java	24 Aug 2003 19:38:02 -0000	1.3
  @@ -101,14 +101,14 @@
        */
       public int getId()
       {
  -        return group.getId();
  +		return new Integer(group.getId()+"").intValue();
       }
       /* (non-Javadoc)
        * @see org.apache.turbine.om.security.SecurityEntity#getIdAsObj()
        */
       public Integer getIdAsObj()
       {
  -        return new Integer(group.getId());
  +        return new Integer(group.getId()+"");
       }
       /* (non-Javadoc)
        * @see org.apache.turbine.om.security.SecurityEntity#setId(int)
  
  
  
  1.3       +2 -2      jakarta-turbine-fulcrum/security/src/java/org/apache/fulcrum/security/adapter/turbine/PermissionAdapter.java
  
  Index: PermissionAdapter.java
  ===================================================================
  RCS file: /home/cvs/jakarta-turbine-fulcrum/security/src/java/org/apache/fulcrum/security/adapter/turbine/PermissionAdapter.java,v
  retrieving revision 1.2
  retrieving revision 1.3
  diff -u -r1.2 -r1.3
  --- PermissionAdapter.java	23 Aug 2003 15:25:54 -0000	1.2
  +++ PermissionAdapter.java	24 Aug 2003 19:38:02 -0000	1.3
  @@ -67,14 +67,14 @@
        */
       public int getId()
       {
  -        return permission.getId();
  +        return new Integer(permission.getId()+"").intValue();
       }
       /* (non-Javadoc)
        * @see org.apache.turbine.om.security.SecurityEntity#getIdAsObj()
        */
       public Integer getIdAsObj()
       {
  -        return new Integer(permission.getId());
  +        return new Integer(permission.getId()+"");
       }
       /* (non-Javadoc)
        * @see org.apache.turbine.om.security.SecurityEntity#setId(int)
  
  
  
  1.3       +2 -2      jakarta-turbine-fulcrum/security/src/java/org/apache/fulcrum/security/adapter/turbine/UserAdapter.java
  
  Index: UserAdapter.java
  ===================================================================
  RCS file: /home/cvs/jakarta-turbine-fulcrum/security/src/java/org/apache/fulcrum/security/adapter/turbine/UserAdapter.java,v
  retrieving revision 1.2
  retrieving revision 1.3
  diff -u -r1.2 -r1.3
  --- UserAdapter.java	23 Aug 2003 15:25:54 -0000	1.2
  +++ UserAdapter.java	24 Aug 2003 19:38:02 -0000	1.3
  @@ -357,14 +357,14 @@
        */
       public int getId()
       {
  -        return user.getId();
  +		return new Integer(user.getId()+"").intValue();
       }
       /* (non-Javadoc)
        * @see org.apache.turbine.om.security.SecurityEntity#getIdAsObj()
        */
       public Integer getIdAsObj()
       {
  -        return new Integer(user.getId());
  +        return new Integer(user.getId()+"");
       }
       /* (non-Javadoc)
        * @see org.apache.turbine.om.security.SecurityEntity#setId(int)
  
  
  

Re: cvs commit: jakarta-turbine-fulcrum/security/src/java/org/apache/fulcrum/security/adapter/turbine RoleAdapter.java GroupAdapter.java PermissionAdapter.java UserAdapter.java

Posted by "Henning P. Schmiedehausen" <hp...@intermeta.de>.
"Eric Pugh" <ep...@upstate.com> writes:

>And, since you suggested that we remove the whole int's longs in favor of
>objects, it will have to be redone any way..  Some times in the thick of
>working you don't always thing the most perfect way of doing something,
>instead you are just trying to make compile errors go away.  I am definitly
>aware of where the warts are.  Its a first cut.

Hi,

I thought so. I do have, however, objections to simply put "warts"
into the HEAD branch of the CVS. If you want to work on a "first cut",
simply branch off, work on the branch, then merge back onto the main
stream, once the code is stable. That's the nature of CVS.

I've made the experience time and again, that without superhuman
discipline, such warts tend to stay, simply because something more
interesting pops up, one gets distracted or simply gets bored and
start to work on something else. And so this code that is intended as
a "first cut that should be smoothed later" stays.

	Regards
		Henning


>Eric

>> -----Original Message-----
>> From: Henning P. Schmiedehausen [mailto:hps@intermeta.de]
>> Sent: Monday, August 25, 2003 10:10 AM
>> To: turbine-dev@jakarta.apache.org
>> Subject: Re: cvs commit:
>> jakarta-turbine-fulcrum/security/src/java/org/apache/fulcrum/s
>> ecurity/ad
>> apter/turbine RoleAdapter.java GroupAdapter.java
>> PermissionAdapter.java
>> UserAdapter.java
>>
>>
>> epugh@apache.org writes:
>>
>> >  +        return new Integer(role.getId()+"").intValue();
>>
>> Sorry, but this is Bullshit.
>>
>> Either do it right new Long(role.getId()).intValue() or leave it. But
>> working on some toString magic and hoping that it somehow works right
>> is just asking for trouble.
>>
>> It will explode anyway the second you use "larger than int" Ids.
>>
>> Turbine can deal with long id's, it would just need some adapters
>> (internally the ints are Integer objects, so we could add some glue
>> methods which accept longs and convert them into Long objects). If you
>> look at the SecurityMap implementation, you can see this.
>>
>> I'd really prefer that you look at the source and try to understand
>> what some of the code does before you go off into coder land and start
>> to hack away. This is how some of the more cumbersome code in the 2.x
>> Turbine core came into existence. I'd like to avoid this for the next
>> turbine releases.
>>
>> 	Regards
>> 		Henning
>>
>>
>>
>> >       }
>> >       /* (non-Javadoc)
>> >        * @see
>> org.apache.turbine.om.security.SecurityEntity#getIdAsObj()
>> >        */
>> >       public Integer getIdAsObj()
>> >       {
>> >  -        return new Integer(role.getId());
>> >  +        return new Integer(role.getId()+"");
>> >       }
>> >       /* (non-Javadoc)
>> >        * @see
>> org.apache.turbine.om.security.SecurityEntity#setId(int)
>> >
>> >
>> >
>> >  1.3       +2 -2
>> jakarta-turbine-fulcrum/security/src/java/org/apache/fulcrum/s
>> ecurity/adapter/turbine/GroupAdapter.java
>> >
>> >  Index: GroupAdapter.java
>> >  ===================================================================
>> >  RCS file:
>> /home/cvs/jakarta-turbine-fulcrum/security/src/java/org/apache
>> /fulcrum/security/adapter/turbine/GroupAdapter.java,v
>> >  retrieving revision 1.2
>> >  retrieving revision 1.3
>> >  diff -u -r1.2 -r1.3
>> >  --- GroupAdapter.java	23 Aug 2003 15:25:54 -0000	1.2
>> >  +++ GroupAdapter.java	24 Aug 2003 19:38:02 -0000	1.3
>> >  @@ -101,14 +101,14 @@
>> >        */
>> >       public int getId()
>> >       {
>> >  -        return group.getId();
>> >  +		return new Integer(group.getId()+"").intValue();
>> >       }
>> >       /* (non-Javadoc)
>> >        * @see
>> org.apache.turbine.om.security.SecurityEntity#getIdAsObj()
>> >        */
>> >       public Integer getIdAsObj()
>> >       {
>> >  -        return new Integer(group.getId());
>> >  +        return new Integer(group.getId()+"");
>> >       }
>> >       /* (non-Javadoc)
>> >        * @see
>> org.apache.turbine.om.security.SecurityEntity#setId(int)
>> >
>> >
>> >
>> >  1.3       +2 -2
>> jakarta-turbine-fulcrum/security/src/java/org/apache/fulcrum/s
>> ecurity/adapter/turbine/PermissionAdapter.java
>> >
>> >  Index: PermissionAdapter.java
>> >  ===================================================================
>> >  RCS file:
>> /home/cvs/jakarta-turbine-fulcrum/security/src/java/org/apache
>> /fulcrum/security/adapter/turbine/PermissionAdapter.java,v
>> >  retrieving revision 1.2
>> >  retrieving revision 1.3
>> >  diff -u -r1.2 -r1.3
>> >  --- PermissionAdapter.java	23 Aug 2003 15:25:54 -0000	1.2
>> >  +++ PermissionAdapter.java	24 Aug 2003 19:38:02 -0000	1.3
>> >  @@ -67,14 +67,14 @@
>> >        */
>> >       public int getId()
>> >       {
>> >  -        return permission.getId();
>> >  +        return new Integer(permission.getId()+"").intValue();
>> >       }
>> >       /* (non-Javadoc)
>> >        * @see
>> org.apache.turbine.om.security.SecurityEntity#getIdAsObj()
>> >        */
>> >       public Integer getIdAsObj()
>> >       {
>> >  -        return new Integer(permission.getId());
>> >  +        return new Integer(permission.getId()+"");
>> >       }
>> >       /* (non-Javadoc)
>> >        * @see
>> org.apache.turbine.om.security.SecurityEntity#setId(int)
>> >
>> >
>> >
>> >  1.3       +2 -2
>> jakarta-turbine-fulcrum/security/src/java/org/apache/fulcrum/s
>> ecurity/adapter/turbine/UserAdapter.java
>> >
>> >  Index: UserAdapter.java
>> >  ===================================================================
>> >  RCS file:
>> /home/cvs/jakarta-turbine-fulcrum/security/src/java/org/apache
>> /fulcrum/security/adapter/turbine/UserAdapter.java,v
>> >  retrieving revision 1.2
>> >  retrieving revision 1.3
>> >  diff -u -r1.2 -r1.3
>> >  --- UserAdapter.java	23 Aug 2003 15:25:54 -0000	1.2
>> >  +++ UserAdapter.java	24 Aug 2003 19:38:02 -0000	1.3
>> >  @@ -357,14 +357,14 @@
>> >        */
>> >       public int getId()
>> >       {
>> >  -        return user.getId();
>> >  +		return new Integer(user.getId()+"").intValue();
>> >       }
>> >       /* (non-Javadoc)
>> >        * @see
>> org.apache.turbine.om.security.SecurityEntity#getIdAsObj()
>> >        */
>> >       public Integer getIdAsObj()
>> >       {
>> >  -        return new Integer(user.getId());
>> >  +        return new Integer(user.getId()+"");
>> >       }
>> >       /* (non-Javadoc)
>> >        * @see
>> org.apache.turbine.om.security.SecurityEntity#setId(int)
>> >
>> >
>> >
>>
>> >---------------------------------------------------------------------
>> >To unsubscribe, e-mail: turbine-dev-unsubscribe@jakarta.apache.org
>> >For additional commands, e-mail: turbine-dev-help@jakarta.apache.org
>>
>> --
>> Dipl.-Inf. (Univ.) Henning P. Schmiedehausen          INTERMETA GmbH
>> hps@intermeta.de        +49 9131 50 654 0   http://www.intermeta.de/
>>
>> Java, perl, Solaris, Linux, xSP Consulting, Web Services
>> freelance consultant -- Jakarta Turbine Development  -- hero for hire
>>
>> "Dominate!! Dominate!! Eat your young and aggregate! I have
>> grotty silicon!"
>>       -- AOL CD when played backwards  (User Friendly - 200-10-15)
>>
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail: turbine-dev-unsubscribe@jakarta.apache.org
>> For additional commands, e-mail: turbine-dev-help@jakarta.apache.org


>---------------------------------------------------------------------
>To unsubscribe, e-mail: turbine-dev-unsubscribe@jakarta.apache.org
>For additional commands, e-mail: turbine-dev-help@jakarta.apache.org

-- 
Dipl.-Inf. (Univ.) Henning P. Schmiedehausen          INTERMETA GmbH
hps@intermeta.de        +49 9131 50 654 0   http://www.intermeta.de/

Java, perl, Solaris, Linux, xSP Consulting, Web Services 
freelance consultant -- Jakarta Turbine Development  -- hero for hire

"Dominate!! Dominate!! Eat your young and aggregate! I have grotty silicon!" 
      -- AOL CD when played backwards  (User Friendly - 200-10-15)

RE: cvs commit: jakarta-turbine-fulcrum/security/src/java/org/apache/fulcrum/security/adapter/turbine RoleAdapter.java GroupAdapter.java PermissionAdapter.java UserAdapter.java

Posted by Eric Pugh <ep...@upstate.com>.
I know..  I specified in the message to quinton that is sucked..  I wa
planning on going back and fixing it..  And I know the larger then ints
would fail.  I specifically said so..  I was trying to get an example of the
code working first, before spending a ton of time fixing it up.  There are
also missing licenses etc that all need to be done.

And, since you suggested that we remove the whole int's longs in favor of
objects, it will have to be redone any way..  Some times in the thick of
working you don't always thing the most perfect way of doing something,
instead you are just trying to make compile errors go away.  I am definitly
aware of where the warts are.  Its a first cut.

Eric

> -----Original Message-----
> From: Henning P. Schmiedehausen [mailto:hps@intermeta.de]
> Sent: Monday, August 25, 2003 10:10 AM
> To: turbine-dev@jakarta.apache.org
> Subject: Re: cvs commit:
> jakarta-turbine-fulcrum/security/src/java/org/apache/fulcrum/s
> ecurity/ad
> apter/turbine RoleAdapter.java GroupAdapter.java
> PermissionAdapter.java
> UserAdapter.java
>
>
> epugh@apache.org writes:
>
> >  +        return new Integer(role.getId()+"").intValue();
>
> Sorry, but this is Bullshit.
>
> Either do it right new Long(role.getId()).intValue() or leave it. But
> working on some toString magic and hoping that it somehow works right
> is just asking for trouble.
>
> It will explode anyway the second you use "larger than int" Ids.
>
> Turbine can deal with long id's, it would just need some adapters
> (internally the ints are Integer objects, so we could add some glue
> methods which accept longs and convert them into Long objects). If you
> look at the SecurityMap implementation, you can see this.
>
> I'd really prefer that you look at the source and try to understand
> what some of the code does before you go off into coder land and start
> to hack away. This is how some of the more cumbersome code in the 2.x
> Turbine core came into existence. I'd like to avoid this for the next
> turbine releases.
>
> 	Regards
> 		Henning
>
>
>
> >       }
> >       /* (non-Javadoc)
> >        * @see
> org.apache.turbine.om.security.SecurityEntity#getIdAsObj()
> >        */
> >       public Integer getIdAsObj()
> >       {
> >  -        return new Integer(role.getId());
> >  +        return new Integer(role.getId()+"");
> >       }
> >       /* (non-Javadoc)
> >        * @see
> org.apache.turbine.om.security.SecurityEntity#setId(int)
> >
> >
> >
> >  1.3       +2 -2
> jakarta-turbine-fulcrum/security/src/java/org/apache/fulcrum/s
> ecurity/adapter/turbine/GroupAdapter.java
> >
> >  Index: GroupAdapter.java
> >  ===================================================================
> >  RCS file:
> /home/cvs/jakarta-turbine-fulcrum/security/src/java/org/apache
> /fulcrum/security/adapter/turbine/GroupAdapter.java,v
> >  retrieving revision 1.2
> >  retrieving revision 1.3
> >  diff -u -r1.2 -r1.3
> >  --- GroupAdapter.java	23 Aug 2003 15:25:54 -0000	1.2
> >  +++ GroupAdapter.java	24 Aug 2003 19:38:02 -0000	1.3
> >  @@ -101,14 +101,14 @@
> >        */
> >       public int getId()
> >       {
> >  -        return group.getId();
> >  +		return new Integer(group.getId()+"").intValue();
> >       }
> >       /* (non-Javadoc)
> >        * @see
> org.apache.turbine.om.security.SecurityEntity#getIdAsObj()
> >        */
> >       public Integer getIdAsObj()
> >       {
> >  -        return new Integer(group.getId());
> >  +        return new Integer(group.getId()+"");
> >       }
> >       /* (non-Javadoc)
> >        * @see
> org.apache.turbine.om.security.SecurityEntity#setId(int)
> >
> >
> >
> >  1.3       +2 -2
> jakarta-turbine-fulcrum/security/src/java/org/apache/fulcrum/s
> ecurity/adapter/turbine/PermissionAdapter.java
> >
> >  Index: PermissionAdapter.java
> >  ===================================================================
> >  RCS file:
> /home/cvs/jakarta-turbine-fulcrum/security/src/java/org/apache
> /fulcrum/security/adapter/turbine/PermissionAdapter.java,v
> >  retrieving revision 1.2
> >  retrieving revision 1.3
> >  diff -u -r1.2 -r1.3
> >  --- PermissionAdapter.java	23 Aug 2003 15:25:54 -0000	1.2
> >  +++ PermissionAdapter.java	24 Aug 2003 19:38:02 -0000	1.3
> >  @@ -67,14 +67,14 @@
> >        */
> >       public int getId()
> >       {
> >  -        return permission.getId();
> >  +        return new Integer(permission.getId()+"").intValue();
> >       }
> >       /* (non-Javadoc)
> >        * @see
> org.apache.turbine.om.security.SecurityEntity#getIdAsObj()
> >        */
> >       public Integer getIdAsObj()
> >       {
> >  -        return new Integer(permission.getId());
> >  +        return new Integer(permission.getId()+"");
> >       }
> >       /* (non-Javadoc)
> >        * @see
> org.apache.turbine.om.security.SecurityEntity#setId(int)
> >
> >
> >
> >  1.3       +2 -2
> jakarta-turbine-fulcrum/security/src/java/org/apache/fulcrum/s
> ecurity/adapter/turbine/UserAdapter.java
> >
> >  Index: UserAdapter.java
> >  ===================================================================
> >  RCS file:
> /home/cvs/jakarta-turbine-fulcrum/security/src/java/org/apache
> /fulcrum/security/adapter/turbine/UserAdapter.java,v
> >  retrieving revision 1.2
> >  retrieving revision 1.3
> >  diff -u -r1.2 -r1.3
> >  --- UserAdapter.java	23 Aug 2003 15:25:54 -0000	1.2
> >  +++ UserAdapter.java	24 Aug 2003 19:38:02 -0000	1.3
> >  @@ -357,14 +357,14 @@
> >        */
> >       public int getId()
> >       {
> >  -        return user.getId();
> >  +		return new Integer(user.getId()+"").intValue();
> >       }
> >       /* (non-Javadoc)
> >        * @see
> org.apache.turbine.om.security.SecurityEntity#getIdAsObj()
> >        */
> >       public Integer getIdAsObj()
> >       {
> >  -        return new Integer(user.getId());
> >  +        return new Integer(user.getId()+"");
> >       }
> >       /* (non-Javadoc)
> >        * @see
> org.apache.turbine.om.security.SecurityEntity#setId(int)
> >
> >
> >
>
> >---------------------------------------------------------------------
> >To unsubscribe, e-mail: turbine-dev-unsubscribe@jakarta.apache.org
> >For additional commands, e-mail: turbine-dev-help@jakarta.apache.org
>
> --
> Dipl.-Inf. (Univ.) Henning P. Schmiedehausen          INTERMETA GmbH
> hps@intermeta.de        +49 9131 50 654 0   http://www.intermeta.de/
>
> Java, perl, Solaris, Linux, xSP Consulting, Web Services
> freelance consultant -- Jakarta Turbine Development  -- hero for hire
>
> "Dominate!! Dominate!! Eat your young and aggregate! I have
> grotty silicon!"
>       -- AOL CD when played backwards  (User Friendly - 200-10-15)
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: turbine-dev-unsubscribe@jakarta.apache.org
> For additional commands, e-mail: turbine-dev-help@jakarta.apache.org


Re: cvs commit: jakarta-turbine-fulcrum/security/src/java/org/apache/fulcrum/security/adapter/turbine RoleAdapter.java GroupAdapter.java PermissionAdapter.java UserAdapter.java

Posted by "Henning P. Schmiedehausen" <hp...@intermeta.de>.
epugh@apache.org writes:

>  +        return new Integer(role.getId()+"").intValue();

Sorry, but this is Bullshit. 

Either do it right new Long(role.getId()).intValue() or leave it. But
working on some toString magic and hoping that it somehow works right
is just asking for trouble.

It will explode anyway the second you use "larger than int" Ids.

Turbine can deal with long id's, it would just need some adapters
(internally the ints are Integer objects, so we could add some glue
methods which accept longs and convert them into Long objects). If you
look at the SecurityMap implementation, you can see this.

I'd really prefer that you look at the source and try to understand
what some of the code does before you go off into coder land and start
to hack away. This is how some of the more cumbersome code in the 2.x
Turbine core came into existence. I'd like to avoid this for the next
turbine releases.

	Regards
		Henning



>       }
>       /* (non-Javadoc)
>        * @see org.apache.turbine.om.security.SecurityEntity#getIdAsObj()
>        */
>       public Integer getIdAsObj()
>       {
>  -        return new Integer(role.getId());
>  +        return new Integer(role.getId()+"");
>       }
>       /* (non-Javadoc)
>        * @see org.apache.turbine.om.security.SecurityEntity#setId(int)
>  
>  
>  
>  1.3       +2 -2      jakarta-turbine-fulcrum/security/src/java/org/apache/fulcrum/security/adapter/turbine/GroupAdapter.java
>  
>  Index: GroupAdapter.java
>  ===================================================================
>  RCS file: /home/cvs/jakarta-turbine-fulcrum/security/src/java/org/apache/fulcrum/security/adapter/turbine/GroupAdapter.java,v
>  retrieving revision 1.2
>  retrieving revision 1.3
>  diff -u -r1.2 -r1.3
>  --- GroupAdapter.java	23 Aug 2003 15:25:54 -0000	1.2
>  +++ GroupAdapter.java	24 Aug 2003 19:38:02 -0000	1.3
>  @@ -101,14 +101,14 @@
>        */
>       public int getId()
>       {
>  -        return group.getId();
>  +		return new Integer(group.getId()+"").intValue();
>       }
>       /* (non-Javadoc)
>        * @see org.apache.turbine.om.security.SecurityEntity#getIdAsObj()
>        */
>       public Integer getIdAsObj()
>       {
>  -        return new Integer(group.getId());
>  +        return new Integer(group.getId()+"");
>       }
>       /* (non-Javadoc)
>        * @see org.apache.turbine.om.security.SecurityEntity#setId(int)
>  
>  
>  
>  1.3       +2 -2      jakarta-turbine-fulcrum/security/src/java/org/apache/fulcrum/security/adapter/turbine/PermissionAdapter.java
>  
>  Index: PermissionAdapter.java
>  ===================================================================
>  RCS file: /home/cvs/jakarta-turbine-fulcrum/security/src/java/org/apache/fulcrum/security/adapter/turbine/PermissionAdapter.java,v
>  retrieving revision 1.2
>  retrieving revision 1.3
>  diff -u -r1.2 -r1.3
>  --- PermissionAdapter.java	23 Aug 2003 15:25:54 -0000	1.2
>  +++ PermissionAdapter.java	24 Aug 2003 19:38:02 -0000	1.3
>  @@ -67,14 +67,14 @@
>        */
>       public int getId()
>       {
>  -        return permission.getId();
>  +        return new Integer(permission.getId()+"").intValue();
>       }
>       /* (non-Javadoc)
>        * @see org.apache.turbine.om.security.SecurityEntity#getIdAsObj()
>        */
>       public Integer getIdAsObj()
>       {
>  -        return new Integer(permission.getId());
>  +        return new Integer(permission.getId()+"");
>       }
>       /* (non-Javadoc)
>        * @see org.apache.turbine.om.security.SecurityEntity#setId(int)
>  
>  
>  
>  1.3       +2 -2      jakarta-turbine-fulcrum/security/src/java/org/apache/fulcrum/security/adapter/turbine/UserAdapter.java
>  
>  Index: UserAdapter.java
>  ===================================================================
>  RCS file: /home/cvs/jakarta-turbine-fulcrum/security/src/java/org/apache/fulcrum/security/adapter/turbine/UserAdapter.java,v
>  retrieving revision 1.2
>  retrieving revision 1.3
>  diff -u -r1.2 -r1.3
>  --- UserAdapter.java	23 Aug 2003 15:25:54 -0000	1.2
>  +++ UserAdapter.java	24 Aug 2003 19:38:02 -0000	1.3
>  @@ -357,14 +357,14 @@
>        */
>       public int getId()
>       {
>  -        return user.getId();
>  +		return new Integer(user.getId()+"").intValue();
>       }
>       /* (non-Javadoc)
>        * @see org.apache.turbine.om.security.SecurityEntity#getIdAsObj()
>        */
>       public Integer getIdAsObj()
>       {
>  -        return new Integer(user.getId());
>  +        return new Integer(user.getId()+"");
>       }
>       /* (non-Javadoc)
>        * @see org.apache.turbine.om.security.SecurityEntity#setId(int)
>  
>  
>  

>---------------------------------------------------------------------
>To unsubscribe, e-mail: turbine-dev-unsubscribe@jakarta.apache.org
>For additional commands, e-mail: turbine-dev-help@jakarta.apache.org

-- 
Dipl.-Inf. (Univ.) Henning P. Schmiedehausen          INTERMETA GmbH
hps@intermeta.de        +49 9131 50 654 0   http://www.intermeta.de/

Java, perl, Solaris, Linux, xSP Consulting, Web Services 
freelance consultant -- Jakarta Turbine Development  -- hero for hire

"Dominate!! Dominate!! Eat your young and aggregate! I have grotty silicon!" 
      -- AOL CD when played backwards  (User Friendly - 200-10-15)