D12386: LdapUrl: remove ctor code which made the QUrl invalid.

David Faure noreply at phabricator.kde.org
Fri Apr 20 18:31:56 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
  dn() already takes care of removing the leading slash from the path
  anyway.
  
  This only showed up during debugging of a new unittest, because actual
  kioslave code (which is where LdapUrl is built from QUrl) only queries
  for components out of the LdapUrl, not for the full string
  representation.

TEST PLAN
  unittest in next commit

REPOSITORY
  R178 PIM: KLDAP

BRANCH
  ctor

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

AFFECTED FILES
  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/14c8a98b/attachment.html>


More information about the kde-pim mailing list