[ 
https://jira.nuxeo.com/browse/NXP-7380?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Thomas Roger resolved NXP-7380.
-------------------------------

    Resolution: Fixed

In the {{RestHelper#initcontextFromRequest}} method, if the outcome is 
{{null}}, we use the DocumentView#getViewId() as outcome if we have a non-null 
{{docView}}.

http://hg.nuxeo.org/nuxeo/nuxeo-jsf/rev/4028c30956de

> Fix propagation of conversation on custom URLs
> ----------------------------------------------
>
>                 Key: NXP-7380
>                 URL: https://jira.nuxeo.com/browse/NXP-7380
>             Project: Nuxeo Enterprise Platform
>          Issue Type: Task
>          Components: Web UI
>    Affects Versions: 5.4.2
>            Reporter: Thomas Roger
>            Assignee: Thomas Roger
>             Fix For: 5.4.3
>
>
> When accessing URLs that defines a repository location but no document ref, 
> the conversation used is not set to be a long running conversation.
> In this case, {{RestHelper#initcontextFromRequest}} returns an outcome null, 
> that's why the conversation is not a long running conversation.
> The code in {{ConversationInterceptor#beginConversationIfNecessary}} tests 
> the {{result}} of the {{initcontextFromRequest}} method and begins a 
> conversation only if the result is not {{null}}:
> {code}
> if ( result!=null || method.getReturnType().equals(void.class) )
> {
>     boolean nested = false;
>     if ( method.isAnnotationPresent(Begin.class) )
>     {
>         nested = method.getAnnotation(Begin.class).nested();
>     }
>     beginConversation( nested, getProcessDefinitionName(method) );
>     setFlushMode(method); //TODO: what if conversation already exists? Or a 
> nested conversation?
> }
> {code}
> In the case where we have only a repository location, the returned value is 
> always {{null}}.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

        
_______________________________________________
ECM-tickets mailing list
[email protected]
http://lists.nuxeo.com/mailman/listinfo/ecm-tickets

Reply via email to