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

Kojo Gambrah commented on OFBIZ-2778:
-------------------------------------

I think this refactoring makes the request definition more intuitive as the 
service called by the event is any ofbiz service type and not specifically 
defined as a JSON service. 

On FTL versus Java for the jsoning, reference the discussions in the thread 
'about using ofbiz as a platform'; there are some developers who would like to 
use parts of the ofbiz platform. I believe they would typically like to 
leverage the data models and services and build new user interfaces fit for 
their applications (adding some entities and services along the way). 

I therefore think that implementing in Java will best serve this interest as 
the platform users will not have to concern themselves with Freemarker even if 
they wanted to modify the framework.

> 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.

Reply via email to