You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@pivot.apache.org by "Greg Brown (JIRA)" <ji...@apache.org> on 2010/06/25 14:54:51 UTC

[jira] Commented: (PIVOT-547) Bug in JSON.java prevents the use of Element as a destination for data binding

    [ https://issues.apache.org/jira/browse/PIVOT-547?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12882558#action_12882558 ] 

Greg Brown commented on PIVOT-547:
----------------------------------

This may or may not be a bug. The JSON accessor methods aren't really intended to be used with XML elements - they are meant to manipulate JSON-like structures, which XML elements are not. You can use the org.apache.pivot.xml.XML class to access nested XML data.

In any case, I'd like to look into this a bit more. Can you provide some sample code that reproduces the issue? Thanks.


> Bug in JSON.java prevents the use of Element as a destination for data binding
> ------------------------------------------------------------------------------
>
>                 Key: PIVOT-547
>                 URL: https://issues.apache.org/jira/browse/PIVOT-547
>             Project: Pivot
>          Issue Type: Bug
>          Components: core-xml
>    Affects Versions: 1.5
>            Reporter: Michael Allman
>            Priority: Minor
>         Attachments: xml_binding_fix.patch
>
>
> A bug in the JSON.put method prevents using an Element instance as a destination for data binding.  This issue arises because Element implements both the Dictionary and Sequence interfaces.
> I've attached a proposed fix.  The approach I've taken is to test if the key can be parsed into an int.  If not, it's assumed to be a Dictionary key.  This fixes the issue with Element and should fix the issue for the more general case of any other class that implement Dictionary and Sequence.
> Given that this is a bug, I've generated the patch against the 1.5.x branch.
> Cheers.

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