You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@jackrabbit.apache.org by "Jukka Zitting (JIRA)" <ji...@apache.org> on 2010/09/23 13:52:35 UTC

[jira] Resolved: (JCR-2750) MultiStatusResponse should not call resource.getProperties

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

Jukka Zitting resolved JCR-2750.
--------------------------------

         Assignee: Jukka Zitting
    Fix Version/s: 2.2.0
       Resolution: Fixed

Fixed in revision 1000414.

> MultiStatusResponse should not call resource.getProperties
> ----------------------------------------------------------
>
>                 Key: JCR-2750
>                 URL: https://issues.apache.org/jira/browse/JCR-2750
>             Project: Jackrabbit Content Repository
>          Issue Type: Improvement
>            Reporter: Stepan Koltsov
>            Assignee: Jukka Zitting
>             Fix For: 2.2.0
>
>
> current constructor MultiStatusResponse() calls resource.getProperties() even if propFindType == PROPFIND_BY_PROPERTY.
> This is inconvenient, because some properties are expensive to generate if they are not requested. MultiStatusResponse() constructor with parameter PROPFIND_BY_PROPERTY should do:
> ===
> if (propFindType == PROPFIND_BY_PROPERTY) {
>   for (propName : propNameSet) {
>     prop = resource.getProperty(propName);
>     if (prop != null)
>       status200.addContent(prop);
>     else
>       status404.addContent(propName);
>   }
> } else {
>   ...
> }
> ===

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.