Review Request 127944: KDE Platform Theme: set file dialog overwrite option appropriately for saving

Jonathan Marten jjm at keelhaul.me.uk
Fri May 20 09:22:52 BST 2016


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/127944/
-----------------------------------------------------------

(Updated May 20, 2016, 9:22 a.m.)


Review request for kde-workspace and Plasma.


Changes
-------

Use a temporary file for the overwrite check, instead of assuming a name that should exist.


Bugs: 360666
    https://bugs.kde.org/show_bug.cgi?id=360666


Repository: plasma-integration


Description
-------

The referenced bug says that, by default, there is no file overwrite check when using QFileDialog to save a file.  Indeed, on closer investigating it appears that there is no way to even explictly force an overwrite check when using the KDE platform theme, because of this code in plasma-integration/src/platformtheme/kdeplatformfiledialoghelper.cpp:

    // overwrite option
    if (options()->testOption(QFileDialogOptions::FileDialogOption::DontConfirmOverwrite)) {
        dialog->m_fileWidget->setConfirmOverwrite(false);
    }

The default for KFileWidget is already for no overwrite check (as set in kio/src/filewidgets/kfilewidget.cpp which initialises KFileWidgetPrivate::confirmOverwrite to false).  There is no way to override this from the calling application through the platform plugin.

Suggest that the default option should be the same as that defined by Qt for QFileDialog:  always perform an ovewrwrite check on saving, unless the caller has set the QFileDialog::DontConfirmOverwrite option.  This is also a sensible default to have from the user's point of view.  This change does that in the platform theme plugin, for all saving operations.


Diffs (updated)
-----

  autotests/kfiledialog_unittest.cpp 59915da 
  src/platformtheme/kdeplatformfiledialoghelper.cpp 139c35d 
  tests/qfiledialogtest.cpp 1d69ea1 

Diff: https://git.reviewboard.kde.org/r/127944/diff/


Testing
-------

Built plasma-intergration with this change, confirmed correct operation of file dialogues and that confirmation is requested when overwriting an existing file, unless the QFileDialog::DontConfirmOverwrite option is specified.


Thanks,

Jonathan Marten

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20160520/bc98e3ca/attachment.htm>


More information about the kde-core-devel mailing list