Foundatio.Repositories
Foundatio.Repositories copied to clipboard
System.Text.Json Patching Support
I removed some usages of JSON.NET down to where our only usage is pretty much the JSON patch support.
Spent some time taking a look at json-everything's JsonPatch support and it only uses JSON Pointers for paths and does not support some of the things we are doing where we are allowing full JSON Path support where we can do things like $.books[?(@.author == 'John Steinbeck')] to remove all array elements from the books array that has an author sub-property that matches John Steinbeck. So basically we would need to make major changes to their libraries or talk them into supporting JSON Path expressions even though the unofficial JSON Patch spec only supports JSON Pointer.
Originally posted by @ejsmith in https://github.com/FoundatioFx/Foundatio.Repositories/issues/96#issuecomment-1079584216
Found this issue where someone asked him and he said no to supporting JSON Path.
Originally posted by @ejsmith in https://github.com/FoundatioFx/Foundatio.Repositories/issues/96#issuecomment-1079585033
Comment for previous history:
For Json Patching it would be nice to see if we can get it to work without a dependency or we can internalize the dependency. I see several spots doing patching
- https://github.com/gregsdennis/json-everything
- https://github.com/ladeak/JsonMergePatch
- .NET 7 is adding patching to System.Net.Http.Json, we could see how they did it and perhaps do it here.
Originally posted by @niemyjski in https://github.com/FoundatioFx/Foundatio.Repositories/issues/96#issuecomment-1086149286