D12387: LdapUrl: fix LDAP queries broken with non-ascii search strings.

David Faure noreply at phabricator.kde.org
Fri Apr 20 18:32:51 BST 2018


dfaure created this revision.
dfaure added reviewers: dvratil, mlaurent.
Restricted Application added a project: KDE PIM.
dfaure requested review of this revision.

REVISION SUMMARY
  Qt5 made query() return a PrettyDecoded query string, while the code
  was calling fromPercentEncoding for the filter, i.e. assuming it to be FullyEncoded.
  
  With unittest

TEST PLAN
  Typing Valérie in kmail composer now finds LDAP completion correctly.

REPOSITORY
  R178 PIM: KLDAP

BRANCH
  non-ascii-fix

REVISION DETAIL
  https://phabricator.kde.org/D12387

AFFECTED FILES
  autotests/testkldap.cpp
  src/ldapurl.cpp

To: dfaure, dvratil, mlaurent
Cc: #kde_pim, dvasin, winterz, vkrause, mlaurent, knauss, dvratil
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-pim/attachments/20180420/520b6ef9/attachment.html>


More information about the kde-pim mailing list