search

aspnet / JsonPatch

[Archived] JSON PATCH library. Project moved to https://github.com/aspnet/AspNetCore
Apache License 2.0
103 stars 48 forks source link

Discussion: ExpandoObjectAdapter does not assume case insensitivity #103

Closed jbagga closed 6 years ago

jbagga commented 7 years ago

When applying a json patch to ExpandoObjects, case insensitive search will not be used to match the path segment to the object's dictionary keys. Instead, the path segment will be resolved using the NamingStrategy defined in the IContractResolver.

See announcement: https://github.com/aspnet/Announcements/issues/269

rynowak commented 7 years ago

FYI we've done this change to be consistent with how JSON.NET deals with casing and dictionaries. The right way to configure this is to use the naming strategy.

Some of the decisions in this repo were taken which MVC was still using the pascal case contract resolver and don't really make much sense anymore.

aspnet-hello commented 6 years ago

This issue was moved to aspnet/Home#2430