[ 
https://issues.apache.org/jira/browse/SLING-518?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Carsten Ziegeler updated SLING-518:
-----------------------------------

    Component/s:     (was: Commons)
                 JCR

> JCR-Explorer -- Add support for a "meta" selector to support richer 
> information about node properties
> -----------------------------------------------------------------------------------------------------
>
>                 Key: SLING-518
>                 URL: https://issues.apache.org/jira/browse/SLING-518
>             Project: Sling
>          Issue Type: New Feature
>          Components: JCR
>            Reporter: Craig L. Ching
>            Priority: Minor
>         Attachments: SLING-518-001.patch
>
>
> This is the first patch for the JCR Explorer functionality, I'll continue to 
> prepend "JCR-Explorer" to related JIRA issues.  This implements a new JSON 
> rendering needed by the JCR Explorer.
> I'd originally asked to enhance the existing JSON interfaces to the 
> repository to provide the JCR node properties as JSON objects rather than 
> simple key/value pairs, but Alex suggested [1] that that would break existing 
> operations and that I should create a new "metadata" selector to key the 
> different rendering.
> That said, I'm not entirely thrilled about this patch.  It certainly works 
> and was easy enough to create, however, it is mostly just a copy of the 
> existing JsonItemWriter and maintaining two objects that are so similar is 
> surely less than ideal.  But I didn't see an easy way to enhance the existing 
> JsonItemWriter without fundamentally changing it.  So I'm just "throwing this 
> over the wall" to get some feedback/suggestions on how best to proceed.
> I haven't implemented any tests for this, I'll see what I can whip up.  
> Hopefully there are already test cases for JsonItemWriter and 
> JsonRendererServlet that I can base some tests on.
> Ideally, I'd like to get this functionality sorted out and committed before I 
> submit the rest of the JCR Explorer implementation as the rest is 
> non-intrusive and dependent on this.
> [1] -- 
> http://mail-archives.apache.org/mod_mbox/incubator-sling-dev/200805.mbox/%3c63bea5e623e09f4d92233fb12a9f794301fc8...@emailmn.mqsoftware.com%3e
> NOTE: I'll attach a patch in the next comment.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to