Merge pull request #285 from microsoftgraph/bugfix/missing-collection-serialization

bugfix/missing collection serialization

Author: baywet

src/GraphODataTemplateWriter/CodeHelpers/Java/TypeHelperJava.cs

@@ -754,7 +754,7 @@ public static string CreatePackageDefForEnum(this CustomT4Host host)
         }
         public static string GetPackagePrefix(this OdcmObject obj)
         {
-            switch(obj)
+            switch (obj)
             {
                 case OdcmEnum e:
                     return "models.generated";
@@ -860,7 +860,7 @@ public static string CreatePackageDefForBaseMethodRequestBuilder(this CustomT4Ho
             sb.Append("\n");
 
             var imports = host.CurrentType.AsOdcmMethod().WithOverloads().SelectMany(x => ImportClassesOfMethodParameters(x));
-            sb.Append(imports.Any() ? imports.Aggregate((x, y) => $"{x}{Environment.NewLine}{y}"): string.Empty);
+            sb.Append(imports.Any() ? imports.Aggregate((x, y) => $"{x}{Environment.NewLine}{y}") : string.Empty);
             return sb.ToString();
         }
 
@@ -1463,14 +1463,15 @@ public static string CreatePropertyDef(IEnumerable<OdcmProperty> properties, boo
                 var propertyFormat = format;
                 if (property.IsCollection)
                 {
-                    if (!property.IsNavigation())
+                    if (property.IsNavigation())
                     {
-                        propertyType = "java.util.List<" + property.GetTypeString() + ">";
+                        propertyType = TypeCollectionPage(property);
+                        if (!property.ContainsTarget)
+                            propertyFormat = collectionFormat;
                     }
                     else
                     {
-                        propertyType = TypeCollectionPage(property);
-                        propertyFormat = collectionFormat;
+                        propertyType = "java.util.List<" + property.GetTypeString() + ">";
                     }
                 }
                 else

test/Typewriter.Test/TestDataJava/com/microsoft/graph/models/extensions/CloudCommunications.java

@@ -36,12 +36,16 @@ public class CloudCommunications extends Entity implements IJsonBackedObject {
      * The Calls.
      * 
      */
+    @SerializedName("calls")
+    @Expose
     public CallCollectionPage calls;
 
     /**
      * The Call Records.
      * 
      */
+    @SerializedName("callRecords")
+    @Expose
     public CallRecordCollectionPage callRecords;
 
 

test/Typewriter.Test/TestDataJava/com/microsoft/graph/models/extensions/Schedule.java

@@ -44,12 +44,16 @@ public class Schedule extends Entity implements IJsonBackedObject {
      * The Times Off.
      * 
      */
+    @SerializedName("timesOff")
+    @Expose
     public TimeOffCollectionPage timesOff;
 
     /**
      * The Time Off Requests.
      * 
      */
+    @SerializedName("timeOffRequests")
+    @Expose
     public TimeOffRequestCollectionPage timeOffRequests;
 
 

test/Typewriter.Test/TestDataJava/com/microsoft/graph2/callrecords/models/extensions/CallRecord.java

@@ -111,12 +111,16 @@ public class CallRecord extends Entity implements IJsonBackedObject {
      * The Sessions.
      * 
      */
+    @SerializedName("sessions")
+    @Expose
     public SessionCollectionPage sessions;
 
     /**
      * The Recipients.
      * 
      */
+    @SerializedName("recipients")
+    @Expose
     public EntityType2CollectionPage recipients;
 
 

test/Typewriter.Test/TestDataJava/com/microsoft/graph2/callrecords/models/extensions/Session.java

@@ -84,6 +84,8 @@ public class Session extends Entity implements IJsonBackedObject {
      * The Segments.
      * 
      */
+    @SerializedName("segments")
+    @Expose
     public SegmentCollectionPage segments;