Skip to content
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

read methods with JsonPath parameter missing #72

Closed
szavrel opened this issue Mar 20, 2015 · 2 comments
Closed

read methods with JsonPath parameter missing #72

szavrel opened this issue Mar 20, 2015 · 2 comments

Comments

@szavrel
Copy link

szavrel commented Mar 20, 2015

Hi!

First of all thanks for the great library! I've noticed that there seem to be some features missing from the release compared to the documentation in the readme. According to the readme doc:

*All read operations are overloaded and also supports compiled JsonPath objects. This can be useful from a performance perspective if the same path is to be executed many times.

`JsonPath compiledPath = JsonPath.compile("$.store.book[1].author");

String author2 = JsonPath.read(document, compiledPath);`*

I've worked with release 1.2.0 and now 2.0.0 and those operations do not exists. Am I missing something? Do I need additional libraries? Otherwise I'd suggest to remove this part from the docs or mark as not yet supported.

Thanks Stephan

@kallestenflo
Copy link
Contributor

Thank you! I will update the documentation.

On Fri, Mar 20, 2015 at 11:30 AM, szavrel notifications@github.com wrote:

Hi!

First of all thanks for the great library! I've noticed that there seem to
be some features missing from the release compared to the documentation in
the readme. According to the readme doc:

*All read operations are overloaded and also supports compiled JsonPath
objects. This can be useful from a performance perspective if the same path
is to be executed many times.

`JsonPath compiledPath = JsonPath.compile("$.store.book[1].author");

String author2 = JsonPath.read(document, compiledPath);`*

I've worked with release 1.2.0 and now 2.0.0 and those operations do not
exists. Am I missing something? So I need additional libraries? Otherwise
I'd suggest to remove this part from the docs or mark as not yet supported.

Thanks Stephan


Reply to this email directly or view it on GitHub
#72.

kallestenflo added a commit that referenced this issue Mar 25, 2015
@kallestenflo
Copy link
Contributor

Invalid documentation removed,

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

No branches or pull requests

2 participants