[Okular-devel] Review Request 127042: Fix QUrl string encoding

Aleix Pol Gonzalez aleixpol at kde.org
Fri Feb 26 16:11:15 UTC 2016


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/127042/
-----------------------------------------------------------

(Updated Feb. 26, 2016, 4:11 p.m.)


Status
------

This change has been marked as submitted.


Review request for Okular and Alex Richardson.


Changes
-------

Submitted with commit 154c98fdaab2e593091fe644cfaeed96dd9fefaf by Aleix Pol to branch frameworks.


Repository: okular


Description
-------

This Url is converted in to a string to be parsed back using `QUrl( urlString, QUrl::StrictMode )`.
By default, it uses `PrettyDecoded`, which makes `QUrl::StrictMode` choke.

An alternative patch would be to overload openDocument to also take QUrl and skip the back-and-forth conversion.


Diffs
-----

  shell/okular_main.cpp 9bdc1fe 
  shell/shell.h c7c155b 
  shell/shell.cpp c97054a 

Diff: https://git.reviewboard.kde.org/r/127042/diff/


Testing
-------

Now I can open files with a space in it.


Thanks,

Aleix Pol Gonzalez

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/okular-devel/attachments/20160226/1247732e/attachment.html>


More information about the Okular-devel mailing list