jackrabbit-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Stepan Koltsov (JIRA)" <j...@apache.org>
Subject [jira] Commented: (JCR-2750) MultiStatusResponse should not call resource.getProperties
Date Thu, 23 Sep 2010 12:09:34 GMT

    [ https://issues.apache.org/jira/browse/JCR-2750?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12913990#action_12913990
] 

Stepan Koltsov commented on JCR-2750:
-------------------------------------

Thank you!

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


Mime
View raw message