Question on KUrlCompletion::allMatches()

[David Faure]

On Sunday 15 February 2009, Christian Ehrlicher wrote:
> Christian Ehrlicher schrieb:
> > Hi,
> > 
> > what should KUrlCompletion::allMatches() return? An encoded url or a 
> > path? At least on windows it returns the path and not the url which 
> > makes KUrlCompletionTest::testLocalURL() fail.
> > 
> ok, I found the problem:
> 
> in DirectoryListThread::run() m_prepend is not percent encoded when
> m_complete_url is true. kurlcompletiontest works on linux because you
> normally don't have a space in your temp path location.
> I tried to add the percent encoded path but this doesn't work as 
> KCompletion::allMatches() doesn't find the added paths...

Unit test extended to use a directory with space and '#',
unit test fixed to compare the urls properly. No kurlcompletion bug after all.

-- 
David Faure, faure at kde.org, sponsored by Qt Software @ Nokia to work on KDE,
Konqueror (http://www.konqueror.org), and KOffice (http://www.koffice.org).