[ 
https://issues.apache.org/jira/browse/SLING-303?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Felix Meschberger updated SLING-303:
------------------------------------

    Attachment: SLING-303_fmeschbe_2.patch

Updated patch keeping the rootPath in the UjaxPostProcessor and renaming 
currentPath to path.

Looking at the code, I think, the path property is never null as it is set in 
the initCurrentPath method called first in the run() method. So all checks 
along the lines

   String path = htmlResponse.getPath();
   if (path == null) {
       path = rootPath;
   }

is unneeded as path will never be null.

Is this correct.

> make UjaxHtmlResponse public usable
> -----------------------------------
>
>                 Key: SLING-303
>                 URL: https://issues.apache.org/jira/browse/SLING-303
>             Project: Sling
>          Issue Type: Improvement
>          Components: microsling
>            Reporter: Tobias Bocanegra
>            Assignee: Felix Meschberger
>            Priority: Minor
>         Attachments: SLING-303_fmeschbe.patch, SLING-303_fmeschbe_2.patch, 
> ujax_status_provider.r633155.patch
>
>
> the current ujax html response can only by used by the UjaxPostServlet. It 
> would be useful if it can also be used by other classes.

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