You are viewing a plain text version of this content. The canonical link for it is here.

Posted to slide-dev@jakarta.apache.org by Yoshiyuki Kumadaki <ku...@webjp.co.jp> on 2001/07/16 06:51:47 UTC

Propfind(path,properties) BUG?

Hi, all
I am testing Slide client ver 1.0.11.
If I call propfindMethod with 2 properties,then return 4 values.
    3 properties,then return 9 values.   n properties,then return n*n values.
It will be BUG.
In WebdavResource, It is not necessary line 2563 and 2567(for loop).


2532    public Enumeration propfindMethod(String path, Vector properties)



2562                // Do loop as many as requested.
2563 //                for (int i = 0; i < properties.size(); i++) {
2564                    // Do not need to be more strict.
2565                    results.addElement
2566                        (DOMUtils.getTextValue(property.getElement()));
2567 //                }


Best Regards
--
Yoshiyuki Kumadaki / 熊懐 善之
  Web Japan Co.,Ltd / （株）ウエブジャパン
<http://www.webjp.co.jp/>

Re: Propfind(path,properties) BUG?

Posted by Remy Maucherat <re...@apache.org>.

> Hi, all
> I am testing Slide client ver 1.0.11.
> If I call propfindMethod with 2 properties,then return 4 values.
>     3 properties,then return 9 values.   n properties,then return n*n
values.
> It will be BUG.
> In WebdavResource, It is not necessary line 2563 and 2567(for loop).
>
>
> 2532    public Enumeration propfindMethod(String path, Vector properties)
>
>
>
> 2562                // Do loop as many as requested.
> 2563 //                for (int i = 0; i < properties.size(); i++) {
> 2564                    // Do not need to be more strict.
> 2565                    results.addElement
> 2566
(DOMUtils.getTextValue(property.getElement()));
> 2567 //                }

Yes, it looks like there's an obvious problem with that code.

Thanks for the patch !

Remy