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