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 we...@apache.org on 2003/10/17 21:49:07 UTC

cvs commit: jakarta-jetspeed-2/portal/src/java/org/apache/jetspeed/om/impl SecurityRoleRefImpl.java

weaver      2003/10/17 12:49:07

  Modified:    portal/src/java/org/apache/jetspeed/om/impl
                        SecurityRoleRefImpl.java
  Log:
  removed unneeded methods
  
  Revision  Changes    Path
  1.2       +7 -26     jakarta-jetspeed-2/portal/src/java/org/apache/jetspeed/om/impl/SecurityRoleRefImpl.java
  
  Index: SecurityRoleRefImpl.java
  ===================================================================
  RCS file: /home/cvs/jakarta-jetspeed-2/portal/src/java/org/apache/jetspeed/om/impl/SecurityRoleRefImpl.java,v
  retrieving revision 1.1
  retrieving revision 1.2
  diff -u -r1.1 -r1.2
  --- SecurityRoleRefImpl.java	10 Oct 2003 05:12:09 -0000	1.1
  +++ SecurityRoleRefImpl.java	17 Oct 2003 19:49:07 -0000	1.2
  @@ -56,10 +56,10 @@
   import java.io.Serializable;
   import java.util.Locale;
   
  +import org.apache.jetspeed.Jetspeed;
   import org.apache.jetspeed.om.common.MutableDescription;
   import org.apache.jetspeed.om.common.MutableDescriptionSet;
   import org.apache.jetspeed.om.common.SecurityRoleRefComposite;
  -import org.apache.jetspeed.om.impl.*;
   import org.apache.jetspeed.util.HashCodeBuilder;
   import org.apache.pluto.om.common.Description;
   import org.apache.pluto.om.common.DescriptionSet;
  @@ -80,8 +80,6 @@
       protected long portletId;
       private String link;
       private String name;
  -    private String description;
  -
       private MutableDescriptionSet descriptions;
   
       /**
  @@ -166,18 +164,6 @@
       }
   
       /**
  -     * @deprecated
  -     * @see org.apache.pluto.om.common.SecurityRoleRefCtrl#setDescription(java.lang.String)
  -     */
  -    public void setDescription(String arg0)
  -    {
  -        // TODO Is this really needed anymore???
  -        System.out.println("Setting description for sec ref..." + arg0);
  -        addDescription(new DescriptionImpl(Locale.getDefault(), arg0, MutableDescription.TYPE_SEC_ROLE_REF));
  -        System.out.println("Description Set for sec ref " + arg0);
  -    }
  -
  -    /**
        * @see org.apache.jetspeed.om.common.SecurityRoleRefComposite#setDescriptionSet(org.apache.pluto.om.common.DescriptionSet)
        */
       public void setDescriptionSet(DescriptionSet descriptions)
  @@ -188,19 +174,14 @@
   
       /**
        * We should be using one of the more locale-specific methods.
  -     * This is here only preserver unit testing until Castor is re-mapped.
        * 
  -     * @deprecated
  -     * @return
  +     * 
  +     * @see org.apache.pluto.om.common.SecurityRoleRefCtrl#setDescription(java.lang.String)
  +     * @param arg0
        */
  -    public String getDescription()
  +    public void setDescription(String arg0)
       {
  -        Description desc = getDescription(Locale.getDefault());
  -        if (desc != null)
  -        {
  -            return desc.getDescription();
  -        }
  -        return null;
  +       DescriptionImpl di = new DescriptionImpl(Jetspeed.getDefaultLocale(), arg0, MutableDescription.TYPE_SEC_ROLE_REF);
       }
   
   }
  
  
  

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