[PATCH] (again) webdav search

Best, Jan-Pascal van j.p.vanbest at tbm.tudelft.nl
Tue May 14 08:00:09 BST 2002


> I've applied a modified version of the patch - there was no 
> good reason I 
> could see to have the search stuff outside of davStatList. I 
> also changed the 
> "dav-header" metadata to "davHeader" to be consistent with 
> the others - is 
> that ok?
I noticed ;) And it's OK. It works for me except for a small fix I 
sent you privately: the Content-type and charset stuff should be 
on one line, not one two, like
Content-Type: text/xml; charset=utf-8
The charset is an attribute of the content-type and it belongs together
with it. Patch attached, against the current CVS.

Regards

Jan-Pascal van Best


Index: http.cc
===================================================================
RCS file: /home/kde/kdelibs/kioslave/http/http.cc,v
retrieving revision 1.498
diff -u -3 -p -r1.498 http.cc
--- http.cc	12 May 2002 04:41:22 -0000	1.498
+++ http.cc	13 May 2002 09:17:09 -0000
@@ -2134,7 +2134,7 @@ bool HTTPProtocol::httpOpen()
 
     // Set content type of webdav data
     if (davData)
-      davHeader += "Content-Type: text/xml;\r\ncharset=utf-8\r\n";
+      davHeader += "Content-Type: text/xml; charset=utf-8\r\n";
   
     // add extra header elements for WebDAV
     if ( !davHeader.isNull() )


> 
> I can't test the search functionality, so could you let me 
> know if everything 
> is ok?
> 
> Cheers,
> 
> Hamish.
> 




More information about the kfm-devel mailing list