-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Change JsonNode.with(String)
and withArray(String)
to consider argument as JsonPointer
if valid expression
#3568
Comments
JsonNode.with(String)
and withArray(String)
to consider argument as JsonPointer
(if valid)JsonNode.with(String)
and withArray(String)
to consider argument as JsonPointer
if valid expression
For the corner case where a user is creating/fetching keys that start with a forward slash this change is unequivocally not backwards compatible. Given that (I'm not personally effected by this change, was just wondering what the closest replacement for the deprecated |
With #1980 there are new methods/overloads for
withArray()
andwith()
, which takeJsonPointer
.The old methods that take
String
should be retrofitted in 2.x so that:JsonPointer
(that is, starts with leading '/` or is empty String ""), it should be used as suchJsonPointer
, should it check existingproperty
.For Jackson 3.0 we can just drop the check and expect valid
JsonPointer
String representation.The reason for change is to make expected main use case work, while retaining reasonable backwards-compatibility.
The text was updated successfully, but these errors were encountered: