Referencing Binary Format: Remove references to collections

src/main/java/org/joda/beans/ser/bin/BeanReferences.java

@@ -132,13 +132,13 @@ private void findReferencesBean(
             return;
         }
 
-        // has this object been seen before, if so no need to check it again
-        int result = objects.compute(base, BeanReferences::incrementOrOne);
-        if (result > 1) {
-            return;
-        }
-
         if (base instanceof Bean) {
+            // has this object been seen before, if so no need to check it again
+            int result = objects.compute(base, BeanReferences::incrementOrOne);
+            if (result > 1) {
+                return;
+            }
+
             addClassInfo(base, declaredClass);
             Bean bean = (Bean) base;
             if (settings.getConverter().isConvertible(bean.getClass())) {
@@ -166,11 +166,23 @@ private void findReferencesBean(
         } else if (parentIterator != null) {
             SerIterator childIterator = settings.getIteratorFactory().createChild(base, parentIterator);
             if (childIterator != null) {
+                if (childIterator.metaTypeRequired()) {
+                    objects.compute(childIterator.metaTypeName(), BeanReferences::incrementOrOne);
+                }
+                // shouldn't try and reuse references to collections
                 findReferencesIterable(childIterator, objects);
             } else {
+                int result = objects.compute(base, BeanReferences::incrementOrOne);
+                if (result > 1) {
+                    return;
+                }
                 addClassInfo(base, declaredClass);
             }
         } else {
+            int result = objects.compute(base, BeanReferences::incrementOrOne);
+            if (result > 1) {
+                return;
+            }
             addClassInfo(base, declaredClass);
         }
     }

src/test/java/org/joda/beans/ser/SerTestHelper.java

@@ -19,10 +19,12 @@
 import java.util.Currency;
 import java.util.EnumSet;
 import java.util.HashMap;
+import java.util.HashSet;
 import java.util.List;
 import java.util.Locale;
 import java.util.Map;
 import java.util.TimeZone;
+import java.util.TreeSet;
 
 import org.joda.beans.sample.Address;
 import org.joda.beans.sample.Company;
@@ -228,6 +230,17 @@ public static ImmGenericCollections<JodaConvertInterface> testGenericInterfaces(
             .build();
     }
 
+    public static ImmGenericCollections<Object> testGenericInterfacesCollections() {
+        return ImmGenericCollections.builder()
+            .map(ImmutableMap.of(
+                "First", Arrays.asList("A", "B"),
+                "First1", ImmutableList.of("A", "B"),
+                "Third1", new TreeSet<>(ImmutableList.of("A", "B")),
+                "Third", new HashSet<>(Arrays.asList("A", "B")),
+                "Second", testCollections()))
+            .build();
+    }
+
     public static ImmKeyList testIntermediateInterfaces() {
         // second serialized as JodaConvertInterface, non-bean
         // third and fourth are serialized as an intermediate Joda-Convert interface INamedKey

src/test/java/org/joda/beans/ser/bin/TestSerializeReferencingBin.java

@@ -87,6 +87,19 @@ public void test_writeJodaConvertInterface() {
         BeanAssert.assertBeanEquals(bean, array);
     }
 
+    @Test
+    public void test_writeJodaConvertInterfaceCollections() {
+        ImmGenericCollections<?> array = SerTestHelper.testGenericInterfacesCollections();
+
+        byte[] bytes = JodaBeanSer.COMPACT.binWriterReferencing().write(array);
+        //System.out.println(JodaBeanBinReader.visualize(bytes));
+
+        ImmGenericCollections<?> bean =
+                (ImmGenericCollections<?>) JodaBeanSer.COMPACT.binReader().read(bytes);
+        //System.out.println(bean);
+        BeanAssert.assertBeanEquals(bean, array);
+    }
+
     @Test
     public void test_writeIntermediateInterface() {
         ImmKeyList array = SerTestHelper.testIntermediateInterfaces();