PATCH Re: [Bug 68744] Problems with accented characters in file names
Dawit A.
adwa at cox.net
Mon Dec 15 14:50:49 GMT 2003
On Monday 15 December 2003 08:03, David Faure wrote:
> On Thursday 04 December 2003 00:23, David Faure wrote:
> > I'm for reverting that commit, which will fix the filenames problems,
> > but someone has to fix google queries then, probably in ikws.
>
> Here's the patch I posted to bug #64304 to fix this bug.
> The bug reporter verified it to work.
>
> However I wonder if autoWebSearchQuery() should also be modified the same
> way?
Yes, it should.
> What does that one do? (I wouldn't mind a few comments in that
> code...)
It is the engine used if you have selected a default search engine, i.e.
anything other than 'None', and simply typed a phrase or word that is not a
valid URL into konqueror's location bar. It is called from kuriikwsfilter.cpp
which is the last filter invoked.
Hmm.. I think I see now why Malte attempted to fix the problem in KURIFilter
instead of the plugins in the first place. KURIFilter has a ctor that accepts
a KURL instead of a QString. Which means your patch only partially fixes the
problem since an app can call the ctor that accepts the KURL and run into the
problem described by this report.
--
Regards,
Dawit A.
More information about the kde-core-devel
mailing list