You are viewing a plain text version of this content. The canonical link for it is here.
Posted to wsrp4j-dev@portals.apache.org by es...@apache.org on 2007/08/21 06:47:45 UTC

svn commit: r567950 - in /portals/wsrp4j/trunk/commons-producer/src/java/org/apache/wsrp4j/commons/producer/provider/driver: ConsumerConfiguredPortletImpl.java ProducerOfferedPortletImpl.java

Author: esm
Date: Mon Aug 20 21:47:45 2007
New Revision: 567950

URL: http://svn.apache.org/viewvc?rev=567950&view=rev
Log:
Code formatting.

Modified:
    portals/wsrp4j/trunk/commons-producer/src/java/org/apache/wsrp4j/commons/producer/provider/driver/ConsumerConfiguredPortletImpl.java
    portals/wsrp4j/trunk/commons-producer/src/java/org/apache/wsrp4j/commons/producer/provider/driver/ProducerOfferedPortletImpl.java

Modified: portals/wsrp4j/trunk/commons-producer/src/java/org/apache/wsrp4j/commons/producer/provider/driver/ConsumerConfiguredPortletImpl.java
URL: http://svn.apache.org/viewvc/portals/wsrp4j/trunk/commons-producer/src/java/org/apache/wsrp4j/commons/producer/provider/driver/ConsumerConfiguredPortletImpl.java?rev=567950&r1=567949&r2=567950&view=diff
==============================================================================
--- portals/wsrp4j/trunk/commons-producer/src/java/org/apache/wsrp4j/commons/producer/provider/driver/ConsumerConfiguredPortletImpl.java (original)
+++ portals/wsrp4j/trunk/commons-producer/src/java/org/apache/wsrp4j/commons/producer/provider/driver/ConsumerConfiguredPortletImpl.java Mon Aug 20 21:47:45 2007
@@ -26,8 +26,8 @@
  * @see     ConsumerConfiguredPortlet PortletImpl
  * @version $Id$
  */
-public class ConsumerConfiguredPortletImpl extends PortletImpl
-        implements ConsumerConfiguredPortlet {
+public class ConsumerConfiguredPortletImpl extends PortletImpl implements ConsumerConfiguredPortlet 
+{
 
     /**
      * Portlet handle of the parent ProducerOfferedPortlet object
@@ -40,7 +40,8 @@
      * @param handle String representing the portlet handle of the corresponding
      *               parent ProducerOfferedPortlet.
      */
-    public void setParentHandle(String handle) {
+    public void setParentHandle( String handle ) 
+    {
         this.parentHandle = handle;
     }
 
@@ -51,7 +52,8 @@
      * @return String representing the portlet handle of the corresponding
      * parent ProducerOfferedPortlet.
      */
-    public String getParentHandle() {
+    public String getParentHandle() 
+    {
         return this.parentHandle;
     }
 
@@ -61,9 +63,10 @@
      *
      * @return    Object representing the cloned object.
      */
-    public Object clone() {
-        ConsumerConfiguredPortlet clone = (ConsumerConfiguredPortletImpl)super.clone();
-        clone.setParentHandle(this.parentHandle);
+    public Object clone() 
+    {
+        ConsumerConfiguredPortlet clone = (ConsumerConfiguredPortletImpl) super.clone();
+        clone.setParentHandle( this.parentHandle );
         return clone;
     }
 

Modified: portals/wsrp4j/trunk/commons-producer/src/java/org/apache/wsrp4j/commons/producer/provider/driver/ProducerOfferedPortletImpl.java
URL: http://svn.apache.org/viewvc/portals/wsrp4j/trunk/commons-producer/src/java/org/apache/wsrp4j/commons/producer/provider/driver/ProducerOfferedPortletImpl.java?rev=567950&r1=567949&r2=567950&view=diff
==============================================================================
--- portals/wsrp4j/trunk/commons-producer/src/java/org/apache/wsrp4j/commons/producer/provider/driver/ProducerOfferedPortletImpl.java (original)
+++ portals/wsrp4j/trunk/commons-producer/src/java/org/apache/wsrp4j/commons/producer/provider/driver/ProducerOfferedPortletImpl.java Mon Aug 20 21:47:45 2007
@@ -18,10 +18,8 @@
 import java.util.Iterator;
 import java.util.LinkedList;
 
-import org.apache.wsrp4j.commons.producer.provider.interfaces.
-        ConsumerConfiguredPortlet;
-import org.apache.wsrp4j.commons.producer.provider.interfaces.
-        ProducerOfferedPortlet;
+import org.apache.wsrp4j.commons.producer.provider.interfaces.ConsumerConfiguredPortlet;
+import org.apache.wsrp4j.commons.producer.provider.interfaces.ProducerOfferedPortlet;
 
 /**
  * <p>This class implements the ProducerOfferedPortlet interface encapsulating
@@ -30,30 +28,26 @@
  * @see     ProducerOfferedPortlet
  * @version $Id$
  */
-public class ProducerOfferedPortletImpl extends PortletImpl
-        implements ProducerOfferedPortlet {
-
+public class ProducerOfferedPortletImpl extends PortletImpl implements ProducerOfferedPortlet 
+{    
     // handle ID
     private String handle = "";
 
-    // boolean, if registration is required or not
-    private boolean requiresRegistration = false;
-
-    // list of clone handles for this ProducerOfferedPortlet
-    public LinkedList cloneHandles;
+    /**
+     * References clones of this producer portlet
+     * TODO: revisit public access
+     */
+    public LinkedList /* ConsumerConfiguredPortlet */ cloneHandles = new LinkedList();
 
     /**
-     * constructor
+     * Is registration required
      */
+    private boolean requiresRegistration = false;
+
     public ProducerOfferedPortletImpl() {
         this.cloneHandles = new LinkedList();
     }
 
-    /**
-     * Returns the portletHandle.
-     *
-     * @return    A String representing the portletHandle.
-     */
     public String getPortletHandle() {
         return this.handle;
     }
@@ -63,7 +57,8 @@
      *
      * @return    Boolean flag.
      */
-    public boolean isRegistrationRequired() {
+    public boolean isRegistrationRequired() 
+    {
         return this.requiresRegistration;
     }
 
@@ -72,7 +67,8 @@
      *
      * @param     required  Boolean to be assigned to the requiresReg.-flag.
      */
-    public void setRegistrationRequired(boolean required) {
+    public void setRegistrationRequired( boolean required ) 
+    {
         this.requiresRegistration = required;
     }
 
@@ -87,10 +83,10 @@
 
     /**
      * Adds a clone referencing a ConsumerConfiguredPortlet.
-     *
      */
-    public void addClone(ConsumerConfiguredPortlet cce) {
-        this.cloneHandles.add(cce);
+    public void addClone( ConsumerConfiguredPortlet cce ) 
+    {
+        this.cloneHandles.add( cce );
     }
 
     /**
@@ -99,8 +95,9 @@
      * @param   cce  String representing the portlet-handle of the
      *                 corresponding ConsumerConfiguredPortlet.
      */
-    public void deleteClone(ConsumerConfiguredPortlet cce) {
-        this.cloneHandles.remove(cce);
+    public void deleteClone( ConsumerConfiguredPortlet cce ) 
+    {
+        this.cloneHandles.remove( cce );
     }
 
     /**
@@ -108,7 +105,8 @@
      *
      * @return  Iterator
      */
-    public Iterator getClones() {
+    public Iterator getClones() 
+    {
         return this.cloneHandles.iterator();
     }
 
@@ -118,12 +116,13 @@
      *
      * @return    Object representing the cloned object.
      */
-    public Object clone() {
+    public Object clone() 
+    {
         // TODO: Is it correct to construct a ConsumerConfiguredPortlet?
-        ConsumerConfiguredPortlet clone = new ConsumerConfiguredPortletImpl();
-
+        
+        ConsumerConfiguredPortlet clone = new ConsumerConfiguredPortletImpl();        
         clone.setParentHandle(this.handle);
-
+        
         return clone;
     }