[Differential] [Request, 25 lines] D3796: Fix Plasma-QPA filedialog to show wrong directory with QFileDialog::selectUrl()

kossebau (Friedrich W. H. Kossebau) noreply at phabricator.kde.org
Thu Dec 22 19:09:06 UTC 2016


kossebau created this revision.
kossebau added reviewers: Plasma, Frameworks, dfaure.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.

REVISION SUMMARY
  QFileDialog does not set the initialDirectory option in codepaths from
  QFileDialog::selectUrl or QFileDialog::selectFile when passing the task
  on to the native widget. So KDEPlatformFileDialogHelper has to make sure
  itself that info is kept.
  Because KDEPlatformFileDialogHelper::initializeDialog() calls
  setDirectory(options()->initialDirectory()); as intended, usually during
  the show event (and thus resets any otherwise wanted state of the filewidget
  it has from previous setup calls via QFileDialog), to whatever value the
  initialDirectory option had before, usually the working directory as set
  during initialization.

TEST PLAN
  New unit test kfiledialog_unittest testSelectUrl no longer fails with added
  code, also behaves now as expected with code e.g. from KUrlRequester.

REPOSITORY
  R135 Integration for Qt applications in Plasma

BRANCH
  fixFileDialogDirectoryWithSelectUrl

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

AFFECTED FILES
  autotests/kfiledialog_unittest.cpp
  src/platformtheme/kdeplatformfiledialoghelper.cpp

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: kossebau, #plasma, #frameworks, dfaure
Cc: plasma-devel, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20161222/aeab53e9/attachment.html>


More information about the Plasma-devel mailing list