Make existing Methods compatible

Author: Roman Ettlinger

src/abstractions/MultipartBody.cs

@@ -63,7 +63,21 @@ public void AddOrReplacePart<T>(string partName, string contentType, T partValue
     /// <returns>The value of the part.</returns>
     public T? GetPartValue<T>(string partName)
     {
-        return GetPartValue<T>(partName, null);
+        var value = GetPartValue<T>(partName, null);
+
+        if(EqualityComparer<T?>.Default.Equals(value, default))
+        {
+            foreach(var key in _parts.Keys)
+            {
+                if(key.Item1 == partName)
+                {
+                    value = GetPartValue<T>(partName, key.Item2);
+                    break;
+                }
+            }
+        }
+
+        return value;
     }
     /// <summary>
     /// Gets the value of a part from the multipart body.
@@ -95,7 +109,21 @@ public void AddOrReplacePart<T>(string partName, string contentType, T partValue
     /// <returns>True if the part was removed, false otherwise.</returns>   
     public bool RemovePart(string partName)
     {
-        return RemovePart(partName, null);
+        bool success = RemovePart(partName, null);
+
+        if(!success)
+        {
+            foreach(var key in _parts.Keys)
+            {
+                if(key.Item1 == partName)
+                {
+                    success = RemovePart(partName, key.Item2);
+                    break;
+                }
+            }
+        }
+
+        return success;
     }
 
     /// <summary>

tests/abstractions/MultipartBodyTests.cs

@@ -114,13 +114,16 @@ public void AllowsDuplicateEntries()
     }
 
     [Fact]
-    public void ImplementationBreaksExisting()
+    public void MultiPartBodyExistingGetAndRemoveStillWork()
     {
         var body = new MultipartBody();
 
         body.AddOrReplacePart("file", "application/json", "fileContent", "file.json");
 
         // existing usecase, file should still be able to be retreived
         Assert.Equal("fileContent", body.GetPartValue<string>("file"));
+
+        // existing usecase, file should be sucesfully removed
+        Assert.True(body.RemovePart("file"));
     }
 }