You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@jackrabbit.apache.org by "angela (JIRA)" <ji...@apache.org> on 2008/02/07 10:55:11 UTC

[jira] Created: (JCR-1364) DirListingExportHandler: Should not implement PropertyHandler

DirListingExportHandler: Should not implement PropertyHandler
-------------------------------------------------------------

                 Key: JCR-1364
                 URL: https://issues.apache.org/jira/browse/JCR-1364
             Project: Jackrabbit
          Issue Type: Bug
          Components: jackrabbit-jcr-server
            Reporter: angela
            Assignee: angela
            Priority: Minor


issue found by Roland Porath:

if the DirListingExportHandler is used with some other collection nodetype that nt:folder (that may allow other properties) the list of dav properties obtained upon PROPFIND (being delegated to PropertyHandler) results in an imcomplete list.

since the only benefit of the DirListingExportHandler is to display something nice(?) upon a GET to a folder, i'd suggest to remove the implementation of PropertyHandler from the DirListingExportHandler.

angela

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


[jira] Updated: (JCR-1364) DirListingExportHandler: Should not implement PropertyHandler

Posted by "angela (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/JCR-1364?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

angela updated JCR-1364:
------------------------

    Component/s: jackrabbit-webapp

> DirListingExportHandler: Should not implement PropertyHandler
> -------------------------------------------------------------
>
>                 Key: JCR-1364
>                 URL: https://issues.apache.org/jira/browse/JCR-1364
>             Project: Jackrabbit
>          Issue Type: Bug
>          Components: jackrabbit-jcr-server, jackrabbit-webapp
>            Reporter: angela
>            Assignee: angela
>            Priority: Minor
>
> issue found by Roland Porath:
> if the DirListingExportHandler is used with some other collection nodetype that nt:folder (that may allow other properties) the list of dav properties obtained upon PROPFIND (being delegated to PropertyHandler) results in an imcomplete list.
> since the only benefit of the DirListingExportHandler is to display something nice(?) upon a GET to a folder, i'd suggest to remove the implementation of PropertyHandler from the DirListingExportHandler.
> angela

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


[jira] Resolved: (JCR-1364) DirListingExportHandler: Should not implement PropertyHandler

Posted by "angela (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/JCR-1364?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

angela resolved JCR-1364.
-------------------------

    Resolution: Fixed

fixed as suggested (not tested)

> DirListingExportHandler: Should not implement PropertyHandler
> -------------------------------------------------------------
>
>                 Key: JCR-1364
>                 URL: https://issues.apache.org/jira/browse/JCR-1364
>             Project: Jackrabbit
>          Issue Type: Bug
>          Components: jackrabbit-jcr-server, jackrabbit-webapp
>            Reporter: angela
>            Assignee: angela
>            Priority: Minor
>
> issue found by Roland Porath:
> if the DirListingExportHandler is used with some other collection nodetype that nt:folder (that may allow other properties) the list of dav properties obtained upon PROPFIND (being delegated to PropertyHandler) results in an imcomplete list.
> since the only benefit of the DirListingExportHandler is to display something nice(?) upon a GET to a folder, i'd suggest to remove the implementation of PropertyHandler from the DirListingExportHandler.
> angela

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


[jira] Updated: (JCR-1364) DirListingExportHandler: Should not implement PropertyHandler

Posted by "Jukka Zitting (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/JCR-1364?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Jukka Zitting updated JCR-1364:
-------------------------------

    Fix Version/s: 1.5

> DirListingExportHandler: Should not implement PropertyHandler
> -------------------------------------------------------------
>
>                 Key: JCR-1364
>                 URL: https://issues.apache.org/jira/browse/JCR-1364
>             Project: Jackrabbit
>          Issue Type: Bug
>          Components: jackrabbit-jcr-server, jackrabbit-webapp
>            Reporter: angela
>            Assignee: angela
>            Priority: Minor
>             Fix For: 1.5
>
>
> issue found by Roland Porath:
> if the DirListingExportHandler is used with some other collection nodetype that nt:folder (that may allow other properties) the list of dav properties obtained upon PROPFIND (being delegated to PropertyHandler) results in an imcomplete list.
> since the only benefit of the DirListingExportHandler is to display something nice(?) upon a GET to a folder, i'd suggest to remove the implementation of PropertyHandler from the DirListingExportHandler.
> angela

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


[jira] Commented: (JCR-1364) DirListingExportHandler: Should not implement PropertyHandler

Posted by "angela (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/JCR-1364?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12566534#action_12566534 ] 

angela commented on JCR-1364:
-----------------------------

suggested fix:

- remove DirListingExportHandler from PropertyHander list in WEBINF/config.xml
- remove DirListingExportHandler from default population (PropertyManagerImpl.getDefault)
- for backwards compat. keep DirListingExportHandler implementing PropertyHandler but 
  always return 'false' upon PropertyHandler.canExport

> DirListingExportHandler: Should not implement PropertyHandler
> -------------------------------------------------------------
>
>                 Key: JCR-1364
>                 URL: https://issues.apache.org/jira/browse/JCR-1364
>             Project: Jackrabbit
>          Issue Type: Bug
>          Components: jackrabbit-jcr-server
>            Reporter: angela
>            Assignee: angela
>            Priority: Minor
>
> issue found by Roland Porath:
> if the DirListingExportHandler is used with some other collection nodetype that nt:folder (that may allow other properties) the list of dav properties obtained upon PROPFIND (being delegated to PropertyHandler) results in an imcomplete list.
> since the only benefit of the DirListingExportHandler is to display something nice(?) upon a GET to a folder, i'd suggest to remove the implementation of PropertyHandler from the DirListingExportHandler.
> angela

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