[
https://issues.apache.org/jira/browse/OLINGO-856?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15149415#comment-15149415
]
Michael Bolz commented on OLINGO-856:
-------------------------------------
Hi [~rareddy],
Yes, you are right that the two {{register(..)}} methods in the
{{ODataHttpHandler}} are not necessary.
However if someone has implemented the interface and the methods are deleted he
has to implement the delegation from the new generic {{register(OlingoExtension
e)}} to his own/old method (or implement it another way).
And a single method:
{code}
void register(Processor p);
{code}
in combination with extend of the {{CustomContentTypeSupport}} and
{{CustomETagSupport}} with {{Processor}} do not fit
to the idea that the {{Processor}} interface should be used for core processing
classes and all other are more like /Support/ or /Extension/ classes.
Actual I would prefer the committed solution, however I’am open for some more
opinions and discussions ;o)
Kind Regards,
Michael
> ODataHandler like interface in core API
> ---------------------------------------
>
> Key: OLINGO-856
> URL: https://issues.apache.org/jira/browse/OLINGO-856
> Project: Olingo
> Issue Type: New Feature
> Components: odata4-server
> Affects Versions: (Java) V4 4.1.0
> Reporter: Michael Bolz
> Assignee: Michael Bolz
> Priority: Minor
>
> If Olingo will be used in a none Java Servlet environment it is complicated
> to do this because of the {{void process(HttpServletRequest request,
> HttpServletResponse response);}} entry point which is dependent on the
> servlet request/response objects.
> To avoid this a interface with a {{process}} method like in the
> {{org.apache.olingo.server.core.ODataHandler}} with the {{ODataRequest}} and
> {{ODataResponse}} objects would be better.
> This interface also should be the result of the
> {{OData.createHandler(ServiceMetadata serviceMetadata):ODataHttpHandler}}
> method.
> This would help a lot to use Olingo in other web frameworks (beside of the
> Java Servlet).
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
