[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kfm-devel
Subject:    Re: [PATCH] Webdav bug
From:       Hamish Rodda <meddie () yoyo ! cc ! monash ! edu ! au>
Date:       2002-03-22 2:53:39
[Download RAW message or body]

>> One question though, with a woring KURL called "url", shouldn't
>> url.setPath(url.path(+1)) not interfere with the encoded/decoded status?
>> I'm finding that /~test/ becomes /%7Etest/ when output from
>> encodedPathAndQuery(), but without it, it remains as /~test/...?
>
>I added KURL::adjustPath(...) for that, since David also needed it.
>
>I have committed the basic parts of your patch to CVS, making use of ths new
>adjustPath function.

Waldo,

Thanks for this. I have tested it and the bugs are fixed.

I've noticed that a call to adjustPath() still causes the output of 
encodedPathAndQuery() to switch from a decoded path to an encoded path (ie 
/~test/ to /%7Etest/). I don't know why. Now that I look at it again, the 
name encodedPathAndQuery suggests that the behaviour before calling 
adjustPath() is the incorrect one. Sending encoded paths now in PROPFIND 
requests doesn't seem to be running into problems however so it's not an 
issue to me.

Cheers,

Hamish.
[prev in list] [next in list] [prev in thread] [next in thread]