[Differential] [Commented On] D2632: Don't append unnecessary slashes in Fetch Project dialog

[elvisangelaccio (Elvis Angelaccio)]

elvisangelaccio added a comment.


  In https://phabricator.kde.org/D2632#49247, @mwolff wrote:
  
  > wait, before pushing this: if I cannot type "/foo/bar" into the location bar and then set a project name to get "/foo/bar/asdf", then something is broken. The duplicate "/" are just stylistically annoying, but a change in behavior as apparently introduced by this patch is imo not acceptable
  
  
  As far as I can see, this was already broken before this patch. `/foo/bar` would just become `/foo//asdf`, dropping `bar`.

REPOSITORY
  rKDEVPLATFORM KDevPlatform

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

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: elvisangelaccio, #kdevelop, brauch
Cc: mwolff, brauch, kdevelop-devel, #kdevelop, Pilzschaf, akshaydeo, surgenight, arrowdodger
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20160904/8afe7507/attachment.html>