svn commit: r385576 - /lenya/trunk/src/modules/webdav/java/src/org/apache/lenya/cms/usecases/webdav/

El lun, 13-03-2006 a las 17:45 +0100, Andreas Hartmann
escribió:
> Hi Thorsten,
> 
> thorstenapache.org wrote:
> > Author: thorsten
> > Date: Mon Mar 13 07:46:24 2006
> > New Revision: 385576
> > 
> > URL: http://svn.apache.org/viewcvs?rev=385576&view=rev
> > Log:
> > Bug 38913
> >   WebDav module displays empty directories
(collections)
> >   Reporter:    renaud richardet
> >   Patch:  Jonathan Addison
> > Patch applied, thx Jonathan.
> > 
> > Modified:
> >    
lenya/trunk/src/modules/webdav/java/src/org/apache/lenya/cms
/usecases/webdav/Propfind.java
> > 
> > Modified:
lenya/trunk/src/modules/webdav/java/src/org/apache/lenya/cms
/usecases/webdav/Propfind.java
> > URL: http://svn.apache.org/viewcvs/lenya/trunk/src/modules/webda
v/java/src/org/apache/lenya/cms/usecases/webdav/Propfind.jav
a?rev=385576&r1=385575&r2=385576&view=diff
> >
============================================================
==================
> > ---
lenya/trunk/src/modules/webdav/java/src/org/apache/lenya/cms
/usecases/webdav/Propfind.java (original)
> > +++
lenya/trunk/src/modules/webdav/java/src/org/apache/lenya/cms
/usecases/webdav/Propfind.java Mon Mar 13 07:46:24 2006
> >  -122,26 +122,6 
> >                          checkedOut.add(entry);
> >                      else
> >                          checkedOut.add(null);
> > -                    // get additional language
documents
> > -                    String[] langs =
documents[i].getLanguages();
> > -                    for (int j = 0; j <
langs.length; j++) {
> > -                        if
(!(langs[j].equals(documents[i].getLanguage()))) {
> > -                            String url =
documents[i].getCanonicalWebappURL()
> > -                                   
.replaceFirst(".html$", "_" +
langs[j] + ".html");
> 
> I'm not that familiar with the WebDAV implementation,
but doesn't
> this interfere with custom DocumentBuilders?

No, not as I am aware of it, since it was duplicate code
which produced
a bug in the file listing.

Thx

salu2
-- 
thorsten

"Together we stand, divided we fall!" 
Hey you (Pink Floyd)


------------------------------------------------------------
---------
To unsubscribe, e-mail: dev-unsubscribelenya.apache.org
For additional commands, e-mail: dev-helplenya.apache.org

Thorsten Scherler wrote:
> El lun, 13-03-2006 a las 17:45 +0100, Andreas Hartmann
escribió:
>> Hi Thorsten,
>>
>> thorstenapache.org wrote:
>>> Author: thorsten
>>> Date: Mon Mar 13 07:46:24 2006
>>> New Revision: 385576
>>>
>>> URL: http://svn.apache.org/viewcvs?rev=385576&view=rev
>>> Log:
>>> Bug 38913
>>>   WebDav module displays empty directories
(collections)
>>>   Reporter:    renaud richardet
>>>   Patch:  Jonathan Addison
>>> Patch applied, thx Jonathan.
>>>
>>> Modified:
>>>    
lenya/trunk/src/modules/webdav/java/src/org/apache/lenya/cms
/usecases/webdav/Propfind.java
>>>
>>> Modified:
lenya/trunk/src/modules/webdav/java/src/org/apache/lenya/cms
/usecases/webdav/Propfind.java
>>> URL: http://svn.apache.org/viewcvs/lenya/trunk/src/modules/webda
v/java/src/org/apache/lenya/cms/usecases/webdav/Propfind.jav
a?rev=385576&r1=385575&r2=385576&view=diff
>>>
============================================================
==================
>>> ---
lenya/trunk/src/modules/webdav/java/src/org/apache/lenya/cms
/usecases/webdav/Propfind.java (original)
>>> +++
lenya/trunk/src/modules/webdav/java/src/org/apache/lenya/cms
/usecases/webdav/Propfind.java Mon Mar 13 07:46:24 2006
>>>  -122,26 +122,6 
>>>                          checkedOut.add(entry);
>>>                      else
>>>                          checkedOut.add(null);
>>> -                    // get additional language
documents
>>> -                    String[] langs =
documents[i].getLanguages();
>>> -                    for (int j = 0; j <
langs.length; j++) {
>>> -                        if
(!(langs[j].equals(documents[i].getLanguage()))) {
>>> -                            String url =
documents[i].getCanonicalWebappURL()
>>> -                                   
.replaceFirst(".html$", "_" +
langs[j] + ".html");
>> I'm not that familiar with the WebDAV
implementation, but doesn't
>> this interfere with custom DocumentBuilders?
> 
> No, not as I am aware of it, since it was duplicate
code which produced
> a bug in the file listing.

My bad - sorry, I just saw the lines and not the
"-" signs 

-- Andreas

-- 
Andreas Hartmann
Wyona Inc.  -   Open Source Content Management   -   Apache
Lenya
http://www.wyona.com     
                http://lenya.apache.org
andreas.hartmannwyona.com                     andreasapache.org


------------------------------------------------------------
---------
To unsubscribe, e-mail: dev-unsubscribelenya.apache.org
For additional commands, e-mail: dev-helplenya.apache.org