Hello community,
here is the log from the commit of package plasma5-integration for
openSUSE:Factory checked in at 2018-04-22 14:33:26
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/plasma5-integration (Old)
and /work/SRC/openSUSE:Factory/.plasma5-integration.new (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Package is "plasma5-integration"
Sun Apr 22 14:33:26 2018 rev:42 rq:597896 version:5.12.4
Changes:
--------
--- /work/SRC/openSUSE:Factory/plasma5-integration/plasma5-integration.changes
2018-03-30 12:06:08.140526187 +0200
+++
/work/SRC/openSUSE:Factory/.plasma5-integration.new/plasma5-integration.changes
2018-04-22 14:33:31.294251737 +0200
@@ -1,0 +2,7 @@
+Wed Apr 18 08:11:33 UTC 2018 - [email protected]
+
+- Add Fix-initial-directory-selection-for-remote-files.patch to
+ show the proper (remote) directory also for remote files when
+ opening KDE's file dialog (boo#1085364, kde#374913)
+
+-------------------------------------------------------------------
New:
----
Fix-initial-directory-selection-for-remote-files.patch
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Other differences:
------------------
++++++ plasma5-integration.spec ++++++
--- /var/tmp/diff_new_pack.4cUp2M/_old 2018-04-22 14:33:31.878230603 +0200
+++ /var/tmp/diff_new_pack.4cUp2M/_new 2018-04-22 14:33:31.882230458 +0200
@@ -29,6 +29,8 @@
Group: System/GUI/KDE
Url: http://www.kde.org
Source:
http://download.kde.org/stable/plasma/%{version}/plasma-integration-%{version}.tar.xz
+# PATCH-FIX-UPSTREAM
+Patch: Fix-initial-directory-selection-for-remote-files.patch
BuildRequires: extra-cmake-modules >= 5.17.0
BuildRequires: kf5-filesystem
BuildRequires: libQt5Gui-private-headers-devel >= 5.5.0
@@ -76,6 +78,7 @@
%lang_package -n plasma5-integration-plugin
%prep
%setup -q -n plasma-integration-%{version}
+%patch -p1
%build
%cmake_kf5 -d build
++++++ Fix-initial-directory-selection-for-remote-files.patch ++++++
>From bfd41a95530f90ee8d44cbcfd1fa8c62978334a2 Mon Sep 17 00:00:00 2001
From: Alex Richardson <[email protected]>
Date: Sun, 8 Apr 2018 11:06:32 +0100
Subject: KDEPlatformFileDialog: Fix initial directory selection for remote
files
Summary:
Previously KDEPlatformFileDialogHelper::selectFile() would change
options()->initialDirectory() unconditionally even if it was already
set by the QFileDialog code. Since Qt 5.7.1 it is no longer necessary
to derive initialDirectory from the selectFile() call. In fact it is
actuall harmful since it will now override the correct initial directory
that was set by Qt. Without this patch I got the following debug output:
```
KDEPlatformFileDialogHelper::setDirectory
QUrl("sftp://server/home/alr48/cheri/build_sdk.sh";)
KDEPlatformFileDialogHelper::setDirectory
QUrl("sftp://server/home/alr48/cheri/build_sdk.sh";)
KDEPlatformFileDialogHelper::selectFile QUrl("file:///home/alex/build_sdk.sh")
KDEPlatformFileDialogHelper::setDirectory QUrl("file:///home/alex/)
```
The final setDirectory() call is actually a call to
`setDirectory(options->initialDirectory())` which was set in `selectFile()`.
We now depend on Qt 5.9 so we can remove this code without a check for
version >= 5.7.1.
BUG: 374913
Test Plan: Remote directory is now opened correctly (tested with Qt 5.10.0)
Reviewers: #plasma, elvisangelaccio
Reviewed By: elvisangelaccio
Subscribers: ngraham, krzyc, anthonyfieroni, elvisangelaccio, graesslin,
plasma-devel
Tags: #plasma
Differential Revision: https://phabricator.kde.org/D4193
---
src/platformtheme/kdeplatformfiledialoghelper.cpp | 5 -----
1 file changed, 5 deletions(-)
diff --git a/src/platformtheme/kdeplatformfiledialoghelper.cpp
b/src/platformtheme/kdeplatformfiledialoghelper.cpp
index 1a2b1b5..d4ea0cd 100644
--- a/src/platformtheme/kdeplatformfiledialoghelper.cpp
+++ b/src/platformtheme/kdeplatformfiledialoghelper.cpp
@@ -409,11 +409,6 @@ QUrl KDEPlatformFileDialogHelper::directory() const
void KDEPlatformFileDialogHelper::selectFile(const QUrl &filename)
{
m_dialog->selectFile(filename);
-
- // Qt 5 at least <= 5.8.0 does not derive the directory from the passed url
- // and set the initialDirectory option accordingly, also not for known
schemes
- // like file://, so we have to do it ourselves
- options()->setInitialDirectory(m_dialog->directory());
}
void KDEPlatformFileDialogHelper::setDirectory(const QUrl &directory)
--
cgit v0.11.2
