D11510: [KPropertiesDialog] Support Checksum tab for any local path
Kai Uwe Broulik
noreply at phabricator.kde.org
Tue Mar 20 15:06:38 UTC 2018
broulik created this revision.
broulik added reviewers: Frameworks, dfaure, elvisangelaccio.
Restricted Application added a project: Frameworks.
broulik requested review of this revision.
REVISION SUMMARY
`computeChecksum` was already called with `localPath()` so check for it not being empty rather than `isLocalFile()`.
This allows to calculate checksums also from e.g. `desktop:/`
BUG: 392100
TEST PLAN
Verified that "Checksum" tab shows up for files on the desktop and verified that checksums for `desktop:/foo.txt` are the same as `file:///home/foo/Desktop/foo.txt`
REPOSITORY
R241 KIO
REVISION DETAIL
https://phabricator.kde.org/D11510
AFFECTED FILES
src/widgets/kpropertiesdialog.cpp
To: broulik, #frameworks, dfaure, elvisangelaccio
Cc: michaelh, ngraham
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20180320/684ac761/attachment-0001.html>
More information about the Kde-frameworks-devel
mailing list