[
https://issues.apache.org/jira/browse/SLING-942?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Felix Meschberger updated SLING-942:
------------------------------------
Attachment: SLING-942-fmeschbe.patch
Modified patch, which applies the same logic to the getJSONArray method and
also includes adaptions of the optJSONArray and optJSONObject methods to also
support on-the-fly parsing and conversion. Finally the patch also catches any
problems while parsing the string.
WDYT ?
> Have JSONArray.getJSONObject(int) construct JSONObject if value is String
> -------------------------------------------------------------------------
>
> Key: SLING-942
> URL: https://issues.apache.org/jira/browse/SLING-942
> Project: Sling
> Issue Type: Improvement
> Components: Commons JSON
> Affects Versions: Commons JSON 2.0.2, Commons JSON 2.0.4
> Reporter: Carl Hall
> Assignee: Felix Meschberger
> Attachments: SLING-942-fmeschbe.patch, sling-942.diff
>
> Original Estimate: 1h
> Remaining Estimate: 1h
>
> JSONArray.getJSONObject(int) current checks for JSONObject and throws an
> exception if instanceof fails. I discovered this after excepting to get back
> a JSONObject from this but debugging showed that the value was a JSON String.
> This functionality could be enhanced by checking for String and converting
> to a JSONObject if possible.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
