Mateus Dalepiane

Results 6 comments of Mateus Dalepiane

@JamesNK what do you think about this fix?

@kvprasoon I took your changes and figured a way to work around this problem (hope that's ok). Here's my code: https://github.com/mdalepiane/PowerShell/blob/jsonnull/src/Microsoft.PowerShell.Commands.Utility/commands/utility/WebCmdlet/JsonObject.cs#L725 What do you think of this approach? GitHubmdalepiane/PowerShellPowerShell for...

I've found an issue in Newtonsoft.Json repo ([#1997](https://github.com/JamesNK/Newtonsoft.Json/issues/1997)) that's exactly our problem here. It seems that NullValueHandling.Ignore is not yet supported for Dictionary objects. I'm looking at a fix for...

@iSazonov while working on that I figured out there's a catch: when preprocessing the objects, before calling JsonConvert, we actually convert JObject and CustomObjects into dictionaries. So the workaround will...

> Please create tests for all cases. Done! I added the following tests to cover all places where code was added: - TestDictionaryConvertToJsonWithIgnoreNullPropertiesFalse - TestDictionaryConvertToJsonWithIgnoreNullPropertiesTrue - TestObjectConvertToJsonWithIgnoreNullPropertiesFalse - TestObjectConvertToJsonWithIgnoreNullPropertiesTrue -...