Merge pull request #65 from microsoft/feature/nrt-composed

- relaxes nullability tolerance when merging objects for composed types

CHANGELOG.md

@@ -11,6 +11,12 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ### Changed
 
+## [1.0.0-rc.6] - 2023-01-27
+
+### Changed
+
+- Relaxed nullability tolerance when merging objects for composed types.
+
 ## [1.0.0-rc.5] - 2023-01-26
 
 ### Changed

src/Microsoft.Kiota.Abstractions.csproj

@@ -14,7 +14,7 @@
     <EmbedUntrackedSources>true</EmbedUntrackedSources>
     <Deterministic>true</Deterministic>
     <VersionPrefix>1.0.0</VersionPrefix>
-    <VersionSuffix>rc.5</VersionSuffix>
+    <VersionSuffix>rc.6</VersionSuffix>
     <GeneratePackageOnBuild>true</GeneratePackageOnBuild>
     <SignAssembly>false</SignAssembly>
     <DelaySign>false</DelaySign>

src/serialization/ISerializationWriter.cs

@@ -127,7 +127,7 @@ public interface ISerializationWriter : IDisposable
         /// <param name="key">The key to be used for the written value. May be null.</param>
         /// <param name="value">The model object to be written.</param>
         /// <param name="additionalValuesToMerge">The additional values to merge to the main value when serializing an intersection wrapper.</param>
-        void WriteObjectValue<T>(string? key, T? value, params IParsable[] additionalValuesToMerge) where T : IParsable;
+        void WriteObjectValue<T>(string? key, T? value, params IParsable?[] additionalValuesToMerge) where T : IParsable;
         /// <summary>
         /// Writes the specified enum value to the stream with an optional given key.
         /// </summary>

src/serialization/ParseNodeHelper.cs

@@ -16,7 +16,7 @@ public static class ParseNodeHelper {
     /// Merges the given fields deserializers for an intersection type into a single collection.
     /// </summary>
     /// <param name="targets">The collection of deserializers to merge.</param>
-    public static IDictionary<string, Action<IParseNode>> MergeDeserializersForIntersectionWrapper(params IParsable[] targets) {
+    public static IDictionary<string, Action<IParseNode>> MergeDeserializersForIntersectionWrapper(params IParsable?[] targets) {
         if (targets == null)
         {
             throw new ArgumentNullException(nameof(targets));
@@ -26,7 +26,8 @@ public static IDictionary<string, Action<IParseNode>> MergeDeserializersForInter
             throw new ArgumentException("At least one target must be provided.", nameof(targets));
         }
 
-        return targets.SelectMany(static x => x.GetFieldDeserializers())
+        return targets.Where(static x => x != null)
+                        .SelectMany(static x => x!.GetFieldDeserializers())
                         .GroupBy(static x => x.Key)
                         .Select(static x => x.First())
                         .ToDictionary(static x => x.Key, static x => x.Value);