Combine ExpandoObjectAdapter and DictionaryAdapter

aspnet / JsonPatch

[Archived] JSON PATCH library. Project moved to https://github.com/aspnet/AspNetCore

Apache License 2.0

103 stars 48 forks source link

Closed jbagga closed 7 years ago

jbagga commented 7 years ago

Refactoring some code to combine the two (similar) adapters, ensuring both types use the JsonDictionaryContract and resolve the dictionary keys.

May need more tests. This is just to get feedback on some generic type conversion. See my comment below.

cc @rynowak

jbagga commented 7 years ago

@kichalla @rynowak Updated

jbagga commented 7 years ago

@rynowak Changed key and value conversion code

rynowak commented 7 years ago

This is looking pretty good now. Just waiting on a new error message and some tests.

jbagga commented 7 years ago

@rynowak Updated with tests