[jira] [Commented] (OFBIZ-5790) Json string parameters as a service input are not recognized by OFBiz ServiceEventHandler.

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

[jira] [Commented] (OFBIZ-5790) Json string parameters as a service input are not recognized by OFBiz ServiceEventHandler.

Nicolas Malin (Jira)

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

Adrian Crum commented on OFBIZ-5790:
------------------------------------

#1 looks like an ugly kludge to me, but I don't have an alternative approach. It would be nice if you could specify the content type in the request map instead of in the view map - then the controller could select the correct representation handler for the response.

#3 - I have used the JSON converters in client projects.

My recommendation would be to finish the work Adam started (#3) and have various parts of the framework convert things to/from JSON using the converter framework, then switch the converter framework over to a JSON library. I don't think we need any additional JSON code.




> Json string parameters as a service input are not recognized by OFBiz ServiceEventHandler.
> ------------------------------------------------------------------------------------------
>
>                 Key: OFBIZ-5790
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-5790
>             Project: OFBiz
>          Issue Type: Improvement
>    Affects Versions: Trunk
>            Reporter: Amardeep Singh Jhajj
>            Priority: Minor
>         Attachments: OFBIZ-5790.patch, jackson-annotations-2.4.0.jar, jackson-core-2.4.2.jar, jackson-databind-2.4.2.jar
>
>
> I was trying to pass the Json string as a input to a service, but is not recognized by ServiceEventHandler.
> Example Json String- {"faciltyId": "WebStoreWarehouse"}
> I worked on this issue and attached a patch here. Here I have used Jackson Json library for parsing.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)