[ 
https://issues.apache.org/jira/browse/TAP5-2747?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17681741#comment-17681741
 ] 

Ben Weidig commented on TAP5-2747:
----------------------------------

I've checked out JsonPath, and IMO the syntax is overkill for the small subset 
of features that it would be support.

My motivation behind this feature was to provide a simple keypath-based lookup, 
akin to property expressions, not a full-blown query syntax like JsonPath.
That's why it only contains the dereference and safe dereference operator, and 
a way to access arrays.

Even reduced to just lookups, I believe it could provide value to tapestry-json 
and is small enough not to introduce a complicated long-term maintenance 
commitment.
However, the feature is also motivated by wanting something like it this week 
to simplify JSON-related code.
So I might just make a utility class for my projects, if it won't pass the bar 
of relevancy/additional value for Tapestry itself. 

> KeyPath-based traversal of JSONCollection
> -----------------------------------------
>
>                 Key: TAP5-2747
>                 URL: https://issues.apache.org/jira/browse/TAP5-2747
>             Project: Tapestry 5
>          Issue Type: New Feature
>          Components: tapestry-json
>    Affects Versions: 5.8.2
>            Reporter: Ben Weidig
>            Assignee: Ben Weidig
>            Priority: Minor
>
> As navigation nested {{JSONObject}} structures can be tedious, so I propose 
> adding a String-based KeyPath traversal system.
> For example:
> {code:java}
> JSONObject json = ...
> // {
> //   outer: { inner: "here" },
> //   array: [
> //     1,
> //     { innerArray: ["A", "B", "C"] },
> //     ["Z", "Y", "X"]
> //   ]
> // }
> var result = json.getByKeyPath("array[1].innerArray[5]?");
> // Equivalent to
> var result = json.getByKeyPath("array.[1].innerArray.[5]?");{code}
>  
> The {{getByKeyPath}} method returns an Object, but the other types are 
> available, too.
> The following types of keypaths should be supported:
>  * {{JSONObject}} navigation, e.g. {{outer.inner -> "here"}}
>  * {{JSONArray}} indices, e.g. {{array[0] -> 1}}
>  * {{JSONArray}} indices without key, e.g. {{array.[0] -> 1}}
>  * Optionality, e.g, {{outer.nonExistant? -> null}}
> The different types are combinable as needed.
> By default, the KeyPath is delimited by "." but there should also be a 
> {{getByKeyPath(String getKeyPath, String delimiter)}} to support "." in JSON 
> keys.
> I already have a proof-of-concept working. However, it requires some cleanup 
> and a lot of testing before being ready.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to