You are viewing a plain text version of this content. The canonical link for it is here.
Posted to cvs@cocoon.apache.org by cz...@apache.org on 2004/09/01 15:07:09 UTC
svn commit: rev 37338 - in cocoon/trunk/src/blocks/portal/java/org/apache/cocoon/portal: coplet layout layout/impl
Author: cziegeler
Date: Wed Sep 1 06:07:07 2004
New Revision: 37338
Modified:
cocoon/trunk/src/blocks/portal/java/org/apache/cocoon/portal/coplet/CopletInstanceData.java
cocoon/trunk/src/blocks/portal/java/org/apache/cocoon/portal/layout/AbstractLayout.java
cocoon/trunk/src/blocks/portal/java/org/apache/cocoon/portal/layout/Item.java
cocoon/trunk/src/blocks/portal/java/org/apache/cocoon/portal/layout/Layout.java
cocoon/trunk/src/blocks/portal/java/org/apache/cocoon/portal/layout/impl/CompositeLayoutImpl.java
cocoon/trunk/src/blocks/portal/java/org/apache/cocoon/portal/layout/impl/CopletLayout.java
Log:
Fix copy
Modified: cocoon/trunk/src/blocks/portal/java/org/apache/cocoon/portal/coplet/CopletInstanceData.java
==============================================================================
--- cocoon/trunk/src/blocks/portal/java/org/apache/cocoon/portal/coplet/CopletInstanceData.java (original)
+++ cocoon/trunk/src/blocks/portal/java/org/apache/cocoon/portal/coplet/CopletInstanceData.java Wed Sep 1 06:07:07 2004
@@ -106,4 +106,13 @@
return clone;
}
+ public CopletInstanceData copy() {
+ try {
+ return (CopletInstanceData)this.clone();
+ } catch (CloneNotSupportedException cnse) {
+ // ignore
+ return null;
+ }
+ }
+
}
Modified: cocoon/trunk/src/blocks/portal/java/org/apache/cocoon/portal/layout/AbstractLayout.java
==============================================================================
--- cocoon/trunk/src/blocks/portal/java/org/apache/cocoon/portal/layout/AbstractLayout.java (original)
+++ cocoon/trunk/src/blocks/portal/java/org/apache/cocoon/portal/layout/AbstractLayout.java Wed Sep 1 06:07:07 2004
@@ -89,9 +89,9 @@
}
/* (non-Javadoc)
- * @see org.apache.cocoon.portal.layout.Layout#copy(java.util.Map)
+ * @see org.apache.cocoon.portal.layout.Layout#copy()
*/
- public Layout copy(Map copletInstanceDatas) {
+ public Layout copy() {
try {
return (Layout)this.clone();
} catch (CloneNotSupportedException cnse) {
Modified: cocoon/trunk/src/blocks/portal/java/org/apache/cocoon/portal/layout/Item.java
==============================================================================
--- cocoon/trunk/src/blocks/portal/java/org/apache/cocoon/portal/layout/Item.java (original)
+++ cocoon/trunk/src/blocks/portal/java/org/apache/cocoon/portal/layout/Item.java Wed Sep 1 06:07:07 2004
@@ -15,8 +15,6 @@
*/
package org.apache.cocoon.portal.layout;
-import java.util.Map;
-
/**
*
@@ -68,11 +66,11 @@
return clone;
}
- public Item copy(CompositeLayout parent, Map copletInstanceDatas) {
+ public Item copy(CompositeLayout parent) {
try {
Item clone = (Item)this.clone();
if ( this.layout != null ) {
- clone.layout = this.layout.copy(copletInstanceDatas);
+ clone.layout = this.layout.copy();
}
clone.parentLayout = parent;
return clone;
Modified: cocoon/trunk/src/blocks/portal/java/org/apache/cocoon/portal/layout/Layout.java
==============================================================================
--- cocoon/trunk/src/blocks/portal/java/org/apache/cocoon/portal/layout/Layout.java (original)
+++ cocoon/trunk/src/blocks/portal/java/org/apache/cocoon/portal/layout/Layout.java Wed Sep 1 06:07:07 2004
@@ -15,8 +15,6 @@
*/
package org.apache.cocoon.portal.layout;
-import java.util.Map;
-
import org.apache.cocoon.portal.factory.Producible;
@@ -49,11 +47,9 @@
void setParent(Item item);
/**
- * Make a copy of this layout object and of all it's children.,
- * Each reference to a coplet instance data is replaced to
- * an instance from the provided map. (If the map is null,
- * the reference is copied).
+ * Make a copy of this layout object and of all it's children.
+ * This includes copies of items and copletinstancedatas.
*/
- Layout copy(Map copletInstanceDatas);
+ Layout copy();
}
Modified: cocoon/trunk/src/blocks/portal/java/org/apache/cocoon/portal/layout/impl/CompositeLayoutImpl.java
==============================================================================
--- cocoon/trunk/src/blocks/portal/java/org/apache/cocoon/portal/layout/impl/CompositeLayoutImpl.java (original)
+++ cocoon/trunk/src/blocks/portal/java/org/apache/cocoon/portal/layout/impl/CompositeLayoutImpl.java Wed Sep 1 06:07:07 2004
@@ -18,7 +18,6 @@
import java.util.ArrayList;
import java.util.Iterator;
import java.util.List;
-import java.util.Map;
import org.apache.cocoon.portal.layout.AbstractLayout;
import org.apache.cocoon.portal.layout.CompositeLayout;
@@ -117,12 +116,12 @@
/* (non-Javadoc)
* @see org.apache.cocoon.portal.layout.Layout#copy(java.util.Map)
*/
- public Layout copy(Map copletInstanceDatas) {
- CompositeLayoutImpl clone = (CompositeLayoutImpl)super.copy(copletInstanceDatas);
+ public Layout copy() {
+ CompositeLayoutImpl clone = (CompositeLayoutImpl)super.copy();
final Iterator i = this.items.iterator();
while ( i.hasNext() ) {
final Item current = (Item)i.next();
- final Item clonedItem = current.copy(clone, copletInstanceDatas);
+ final Item clonedItem = current.copy(clone);
clone.addItem(clonedItem);
}
return clone;
Modified: cocoon/trunk/src/blocks/portal/java/org/apache/cocoon/portal/layout/impl/CopletLayout.java
==============================================================================
--- cocoon/trunk/src/blocks/portal/java/org/apache/cocoon/portal/layout/impl/CopletLayout.java (original)
+++ cocoon/trunk/src/blocks/portal/java/org/apache/cocoon/portal/layout/impl/CopletLayout.java Wed Sep 1 06:07:07 2004
@@ -54,15 +54,15 @@
}
/* (non-Javadoc)
- * @see org.apache.cocoon.portal.layout.Layout#copy(java.util.Map)
+ * @see org.apache.cocoon.portal.layout.Layout#copy()
*/
public Layout copy(Map copletInstanceDatas) {
- CopletLayout clone = (CopletLayout)super.copy(copletInstanceDatas);
+ CopletLayout clone = (CopletLayout)super.copy();
if (copletInstanceDatas == null) {
clone.copletInstanceData = this.copletInstanceData;
} else {
if ( this.copletInstanceData != null ) {
- clone.copletInstanceData = (CopletInstanceData)copletInstanceDatas.get(this.copletInstanceData.getId());
+ clone.copletInstanceData = this.copletInstanceData.copy();
}
}