Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

JsonPatchDocument ignores PropertyNameCaseInsensitive setting #41

Closed
twenzel opened this issue Jan 16, 2025 · 1 comment · Fixed by #42
Closed

JsonPatchDocument ignores PropertyNameCaseInsensitive setting #41

twenzel opened this issue Jan 16, 2025 · 1 comment · Fixed by #42

Comments

@twenzel
Copy link
Contributor

twenzel commented Jan 16, 2025

The Apply method ignores the Options.PropertyNameCaseInsensitive setting to find the target.

At the moment the PocoAdapter.FindPropertyByName method uses the PropertyNamingPolicy to find the property. But this setting is used to Write json.
For example my model have a property "SomeData" and the path in the operation is "somedata".

The property can not be found.

My guess is that we should adjust the logic in PropertyProxyCache.FindPropertyInfo to respect the PropertyNameCaseInsensitive setting.

@Havunen
Copy link
Owner

Havunen commented Jan 16, 2025

linked: #38

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants