https://bugs.kde.org/show_bug.cgi?id=395590

Elvis Angelaccio <elvis.angelac...@kde.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |FIXED
   Version Fixed In|                            |1.2.4
             Status|UNCONFIRMED                 |RESOLVED
      Latest Commit|                            |https://commits.kde.org/kio
                   |                            |-gdrive/98eac3003a07a7b785c
                   |                            |916950076d7b0d56d6d61

--- Comment #1 from Elvis Angelaccio <elvis.angelac...@kde.org> ---
Git commit 98eac3003a07a7b785c916950076d7b0d56d6d61 by Elvis Angelaccio.
Committed on 29/06/2018 at 08:44.
Pushed by elvisangelaccio into branch '1.2'.

Use percent encoding when creating urls for UDS entries

Summary:
Otherwise it won't be possible to open gdrive files that contain special
characters (such as `#`) in their path.

This is not handled by libkgapi, which just copies whatever is in the
JSON from Google, and Google does not seem to encode the filenames
either. So we encode the filenames at the kio-gdrive level, where we
actually play with QUrls.

We do so by actually creating a QUrl (which takes care of
percent-encoding in `setPath()`) and then we convert it to a string.

While at it, use faster concatenation rather than arg parsing.
FIXED-IN: 1.2.4

Test Plan:
Try to open a file with `#` in the name.
Check whether files with normal names still work.

Reviewers: dvratil, dfaure

Differential Revision: https://phabricator.kde.org/D13691

M  +23   -6    autotests/urltest.cpp
M  +15   -1    src/kio_gdrive.cpp
M  +1    -0    src/kio_gdrive.h

https://commits.kde.org/kio-gdrive/98eac3003a07a7b785c916950076d7b0d56d6d61

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to