You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ofbiz.apache.org by "Scott Gray (JIRA)" <ji...@apache.org> on 2009/08/14 16:21:14 UTC
[jira] Issue Comment Edited: (OFBIZ-2778) Alter Json controller
requests to move json behaviour to response (from event)
[ https://issues.apache.org/jira/browse/OFBIZ-2778?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12743208#action_12743208 ]
Scott Gray edited comment on OFBIZ-2778 at 8/14/09 7:19 AM:
------------------------------------------------------------
I like the concept but I'm not sure if we need another request-response type to handle json, couldn't we achieve the same result using the existing view/screen/freemarker model? After all json is just another formatted text response not really any different from html, xml, csv, etc.
Anyone else have an opinion?
EDIT: couldn't acheive -> couldn't we achieve (sounded kind of rude with the missing we)
was (Author: lektran):
I like the concept but I'm not sure if we need another request-response type to handle json, couldn't achieve the same result using the existing view/screen/freemarker model? After all json is just another formatted text response not really any different from html, xml, csv, etc.
Anyone else have an opinion?
> Alter Json controller requests to move json behaviour to response (from event)
> ------------------------------------------------------------------------------
>
> Key: OFBIZ-2778
> URL: https://issues.apache.org/jira/browse/OFBIZ-2778
> Project: OFBiz
> Issue Type: Improvement
> Components: framework
> Reporter: Bob Morley
> Attachments: json.diff
>
>
> I think it would be better to apply the Json behaviour of a controller request to the response and not by having multiple Json event types. These event types always seem to be a wrapped call to another event type that only adds identical Json behaviour (ie. JsonServiceEventHandler wraps ServiceEventHandler and simply adds a Json string to the response).
> Attached is a patch with this change implemented on the "Mark Viewed" function in Order Manager.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.