Index: components/jetspeed-page-manager/src/main/java/org/apache/jetspeed/om/page/impl/FragmentPropertyMap.java
===================================================================
--- components/jetspeed-page-manager/src/main/java/org/apache/jetspeed/om/page/impl/FragmentPropertyMap.java (revision 775409)
+++ components/jetspeed-page-manager/src/main/java/org/apache/jetspeed/om/page/impl/FragmentPropertyMap.java (working copy)
@@ -18,13 +18,12 @@
import java.util.AbstractMap;
import java.util.AbstractSet;
+import java.util.HashSet;
import java.util.Collection;
import java.util.Iterator;
import java.util.Map;
import java.util.Set;
-import org.apache.jetspeed.page.impl.DatabasePageManagerUtils;
-
/**
* FragmentPropertyMap
*
@@ -85,7 +84,7 @@
private class FragmentPropertiesEntrySet extends AbstractSet
{
- private Collection entries = DatabasePageManagerUtils.createCollection();
+ private Set entries = new HashSet();
/* (non-Javadoc)
* @see java.util.Set#add(java.lang.Object)
Index: components/jetspeed-page-manager/src/main/java/org/apache/jetspeed/page/impl/DatabasePageManagerUtils.java
===================================================================
--- components/jetspeed-page-manager/src/main/java/org/apache/jetspeed/page/impl/DatabasePageManagerUtils.java (revision 775409)
+++ components/jetspeed-page-manager/src/main/java/org/apache/jetspeed/page/impl/DatabasePageManagerUtils.java (working copy)
@@ -21,10 +21,8 @@
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
-import org.apache.ojb.broker.util.collections.RemovalAwareCollection;
-import org.apache.ojb.broker.util.collections.RemovalAwareList;
+import org.apache.jetspeed.util.ojb.CollectionUtils;
-
/**
* DatabasePageManagerUtils
*
@@ -44,14 +42,28 @@
* @see JS2-590
* @return
*/
+
+ @SuppressWarnings("unchecked")
public static final Collection createCollection()
{
- return java.util.Collections.synchronizedCollection(new RemovalAwareCollection());
+ // return synchronized manageable/removal aware collection;
+ // requires setting collection-class attributes for collection-
+ // descriptor OJB mappings:
+ //
+ //
+ //
+ return CollectionUtils.createSynchronizedCollection();
}
+ @SuppressWarnings("unchecked")
public static final List createList()
{
- return java.util.Collections.synchronizedList(new RemovalAwareList());
- }
-
+ // return synchronized manageable/removal aware list;
+ // requires setting collection-class attributes for collection-
+ // descriptor OJB mappings:
+ //
+ //
+ //
+ return CollectionUtils.createSynchronizedList();
+ }
}
Index: components/jetspeed-page-manager/src/main/java/org/apache/jetspeed/page/document/impl/NodeImpl.java
===================================================================
--- components/jetspeed-page-manager/src/main/java/org/apache/jetspeed/page/document/impl/NodeImpl.java (revision 775409)
+++ components/jetspeed-page-manager/src/main/java/org/apache/jetspeed/page/document/impl/NodeImpl.java (working copy)
@@ -113,7 +113,7 @@
{
if (metadataFields == null)
{
- metadataFields = DatabasePageManagerUtils.createList();
+ metadataFields = DatabasePageManagerUtils.createCollection();
}
pageMetadata = newPageMetadata(metadataFields);
}
Index: components/jetspeed-page-manager/src/main/resources/JETSPEED-INF/ojb/page-manager-repository.xml
===================================================================
--- components/jetspeed-page-manager/src/main/resources/JETSPEED-INF/ojb/page-manager-repository.xml (revision 775409)
+++ components/jetspeed-page-manager/src/main/resources/JETSPEED-INF/ojb/page-manager-repository.xml (working copy)
@@ -57,6 +57,7 @@
auto-retrieve="true"
auto-delete="object"
auto-update="object"
+ collection-class="org.apache.jetspeed.util.ojb.CollectionUtils$SynchronizedRemovalAwareCollection"
>
@@ -176,6 +177,7 @@
auto-retrieve="true"
auto-delete="object"
auto-update="object"
+ collection-class="org.apache.jetspeed.util.ojb.CollectionUtils$SynchronizedRemovalAwareList"
>
@@ -187,6 +189,7 @@
auto-retrieve="true"
auto-delete="object"
auto-update="object"
+ collection-class="org.apache.jetspeed.util.ojb.CollectionUtils$SynchronizedRemovalAwareList"
>
@@ -198,6 +201,7 @@
auto-retrieve="true"
auto-delete="object"
auto-update="object"
+ collection-class="org.apache.jetspeed.util.ojb.CollectionUtils$SynchronizedRemovalAwareList"
>
@@ -209,6 +213,7 @@
auto-retrieve="true"
auto-delete="object"
auto-update="object"
+ collection-class="org.apache.jetspeed.util.ojb.CollectionUtils$SynchronizedRemovalAwareList"
>
@@ -487,6 +492,7 @@
auto-retrieve="true"
auto-delete="object"
auto-update="object"
+ collection-class="org.apache.jetspeed.util.ojb.CollectionUtils$SynchronizedRemovalAwareCollection"
>
@@ -497,6 +503,7 @@
auto-retrieve="true"
auto-delete="object"
auto-update="object"
+ collection-class="org.apache.jetspeed.util.ojb.CollectionUtils$SynchronizedRemovalAwareList"
>
@@ -717,6 +724,7 @@
auto-retrieve="true"
auto-delete="object"
auto-update="object"
+ collection-class="org.apache.jetspeed.util.ojb.CollectionUtils$SynchronizedRemovalAwareCollection"
>
@@ -801,6 +809,7 @@
auto-retrieve="true"
auto-delete="object"
auto-update="object"
+ collection-class="org.apache.jetspeed.util.ojb.CollectionUtils$SynchronizedRemovalAwareList"
>
@@ -811,6 +820,7 @@
auto-retrieve="true"
auto-delete="object"
auto-update="object"
+ collection-class="org.apache.jetspeed.util.ojb.CollectionUtils$SynchronizedRemovalAwareCollection"
>
@@ -930,6 +940,7 @@
auto-retrieve="true"
auto-delete="object"
auto-update="object"
+ collection-class="org.apache.jetspeed.util.ojb.CollectionUtils$SynchronizedRemovalAwareList"
>
@@ -941,6 +952,7 @@
auto-retrieve="true"
auto-delete="object"
auto-update="object"
+ collection-class="org.apache.jetspeed.util.ojb.CollectionUtils$SynchronizedRemovalAwareList"
>
@@ -952,6 +964,7 @@
auto-retrieve="true"
auto-delete="object"
auto-update="object"
+ collection-class="org.apache.jetspeed.util.ojb.CollectionUtils$SynchronizedRemovalAwareList"
>
@@ -1199,6 +1212,7 @@
auto-retrieve="true"
auto-delete="object"
auto-update="object"
+ collection-class="org.apache.jetspeed.util.ojb.CollectionUtils$SynchronizedRemovalAwareCollection"
>
@@ -1209,6 +1223,7 @@
auto-retrieve="true"
auto-delete="object"
auto-update="object"
+ collection-class="org.apache.jetspeed.util.ojb.CollectionUtils$SynchronizedRemovalAwareList"
>
@@ -1429,6 +1444,7 @@
auto-retrieve="true"
auto-delete="object"
auto-update="object"
+ collection-class="org.apache.jetspeed.util.ojb.CollectionUtils$SynchronizedRemovalAwareCollection"
>
@@ -1501,6 +1517,7 @@
auto-retrieve="true"
auto-delete="object"
auto-update="object"
+ collection-class="org.apache.jetspeed.util.ojb.CollectionUtils$SynchronizedRemovalAwareList"
>
@@ -1645,6 +1662,7 @@
auto-retrieve="true"
auto-delete="object"
auto-update="object"
+ collection-class="org.apache.jetspeed.util.ojb.CollectionUtils$SynchronizedRemovalAwareList"
>
@@ -1656,6 +1674,7 @@
auto-retrieve="true"
auto-delete="object"
auto-update="object"
+ collection-class="org.apache.jetspeed.util.ojb.CollectionUtils$SynchronizedRemovalAwareList"
>
@@ -1667,6 +1686,7 @@
auto-retrieve="true"
auto-delete="object"
auto-update="object"
+ collection-class="org.apache.jetspeed.util.ojb.CollectionUtils$SynchronizedRemovalAwareList"
>
@@ -1797,6 +1817,7 @@
auto-retrieve="true"
auto-delete="object"
auto-update="object"
+ collection-class="org.apache.jetspeed.util.ojb.CollectionUtils$SynchronizedRemovalAwareList"
>
@@ -1873,6 +1894,7 @@
auto-retrieve="true"
auto-delete="object"
auto-update="object"
+ collection-class="org.apache.jetspeed.util.ojb.CollectionUtils$SynchronizedRemovalAwareCollection"
>
@@ -1992,6 +2014,7 @@
auto-retrieve="true"
auto-delete="object"
auto-update="object"
+ collection-class="org.apache.jetspeed.util.ojb.CollectionUtils$SynchronizedRemovalAwareList"
>
@@ -2003,6 +2026,7 @@
auto-retrieve="true"
auto-delete="object"
auto-update="object"
+ collection-class="org.apache.jetspeed.util.ojb.CollectionUtils$SynchronizedRemovalAwareList"
>
@@ -2241,6 +2265,7 @@
auto-retrieve="true"
auto-delete="object"
auto-update="object"
+ collection-class="org.apache.jetspeed.util.ojb.CollectionUtils$SynchronizedRemovalAwareList"
>
@@ -2252,6 +2277,7 @@
auto-retrieve="true"
auto-delete="object"
auto-update="object"
+ collection-class="org.apache.jetspeed.util.ojb.CollectionUtils$SynchronizedRemovalAwareList"
>
@@ -2289,6 +2315,7 @@
auto-retrieve="true"
auto-delete="object"
auto-update="object"
+ collection-class="org.apache.jetspeed.util.ojb.CollectionUtils$SynchronizedRemovalAwareList"
>