> On May 6, 2014, 2:51 p.m., David Faure wrote: > > kurifilter-plugins/shorturi/kshorturifilter.cpp, line 374 > > <https://git.reviewboard.kde.org/r/118017/diff/1/?file=271779#file271779line374> > > > > Better use QUrl u = QUrl::fromLocalFile(path) then. > > Alexander Richardson wrote: > Doesn't pass before > > Alexander Richardson wrote: > isMalformed is found by calling !uri().isValid(), which is > !QUrl::fromUserInput("~").isValid(). > > Without removing the isMalformed check, it will never attempt to > determine whether the file exists and therefore exists will always be false > and the result is an empty URL then.
This is the test I am using in KIO: http://paste.kde.org/ptyzmefz6 - Alexander ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118017/#review57402 ----------------------------------------------------------- On May 6, 2014, 2:42 p.m., Alexander Richardson wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/118017/ > ----------------------------------------------------------- > > (Updated May 6, 2014, 2:42 p.m.) > > > Review request for Plasma and David Faure. > > > Repository: kio-extras > > > Description > ------- > > With QUrl we have to set the scheme manually, seems like KUrl defaulted to > file://. > > Also removed the !isMalformed in the codition, QUrl("~") is malformed, > whereas KUrl("~") was not > > > Diffs > ----- > > kurifilter-plugins/shorturi/kshorturifilter.cpp > 58f3435f7599acf692a082d216a0c58684bcf8b6 > > Diff: https://git.reviewboard.kde.org/r/118017/diff/ > > > Testing > ------- > > Added unit test in KIO, works now > > > Thanks, > > Alexander Richardson > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel