Review Request: Allows umlauts and other non-ASCII symbols in the locations runner for URLs + work again with mailto: etc.

[Matthias Fuchs]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://svn.reviewboard.kde.org/r/6042/
-----------------------------------------------------------

(Updated 2010-12-04 14:24:09.010483)


Review request for Plasma and Aaron Seigo.


Changes
-------

Adds modifications to fix another bug.
The fixes would be commited seperately though are combined here for an easier review-process.


Summary (updated)
-------

Allows a wider range of urls without a protocol to be accepted, like those containing umlauts etc., queries, or IPV4-Adresses.
If no protocol has be defined, then ports aren't supported.
BUG:227380

Correctly handle protocols that are not a NetorkLocation or http, ftp.
BUG:228017


This addresses bugs 227380 and 228017.
    https://bugs.kde.org/show_bug.cgi?id=227380
    https://bugs.kde.org/show_bug.cgi?id=228017


Diffs (updated)
-----

  /trunk/KDE/kdebase/workspace/plasma/generic/runners/locations/locationrunner.cpp 1203489 

Diff: http://svn.reviewboard.kde.org/r/6042/diff


Testing (updated)
-------

====More Urls:====
Tried with different urls, including the one of the bug report www.hörstreich.de, google.at, 74.125.39.99/search?hl=en&biw=1280&bih=891&q=kde&aq=f&aqi=g10&aql=&oq=&gs_rfai= etc.

Problem:
There is one problem though that is unrelated to this patch and did _not_ exist in KDE 4.5.4
If I enter "google.at" it will display the result that I can go to http://google.at, yet if I add ":\d" e.g. google.at:8 the result won't be updated as nothing matches, yet the old result still stays.
In KDE 4.5.4 the old result would also disappear after a short time.

====Correctly handle Protocls:====
Tried mailto:chris at example.com, mailto:infobot at example.com?subject=current-issue and ~/kde.

Problem:
Again this problem existed before this patch.
In BUG:167862 there is mentioned, that simply entering a folder name in your $HOME-dir displays the folder, so instead of "~/kde" also "kde" should work. Though this is not the case. Instead when the "Nepomuk Desktop Search Runner" is activated just inputing "kde" works, unless it is excluded from indexing.


Thanks,

Matthias

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/plasma-devel/attachments/20101204/ab55a315/attachment.htm