D8208: Add Duplicate feature

[David Faure]

dfaure added a comment.


  Hmm I don't think I should change the behaviour of QMimeDatabase after all.
  
  https://lxr.kde.org/source/kde/kdeutils/ark/kerfuffle/createdialog.cpp#0106 does
  
    const QString detectedSuffix = QMimeDatabase().suffixForFileName(fileName);
    if (currentMimeType().suffixes().contains(detectedSuffix)) {
  
  which would break if case was preserved.
  
  fileName.right(extension.size()) is exactly the way I almost-fixed this in Qt, so it's the right thing to do here instead :)
  http://www.davidfaure.fr/kde/qt_preserve_case.diff for the record.

REPOSITORY
  R318 Dolphin

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

To: ngraham, #dolphin, #kde_applications, elvisangelaccio, dfaure, broulik, davidedmundson
Cc: fvogt, mthw, kfm-devel, meven, markg, fazevedo, cfeck, #dolphin, pberestov, iasensio, fprice, MrPepe, fbampaloukas, alexde, Codezela, feverfew, spoorun, navarromorales, firef, ngraham, andrebarros, emmanuelp, mikesomov
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20200111/4e007917/attachment.htm>