Review Request: Use KFileItem::targetUrl() instead of KFileItem::url() in PreviewJob

[Sebastian Trueg]

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

Review request for kdelibs.


Summary
-------

Currently the PreviewJob uses KFileItem::url() to generate the preview for files. However, today targetUrl() is the correct one to use since other hierarchies than the file:/ one can contain local files.
To be precise one would even have to check for UDS_LOCAL_FILE but using targetUrl() is a good start.
Since PreviewJob is a rather big class I would like to have someone look at the patch to check if I missed any url() -> targetUrl() replacement.


This addresses bug 248234.
    https://bugs.kde.org/show_bug.cgi?id=248234


Diffs
-----

  trunk/KDE/kdelibs/kio/kio/previewjob.cpp 1171417 

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


Testing
-------


Thanks,

Sebastian

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20100903/0ff3ab74/attachment.htm>