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" >