1 From 7b54f3db98e96231fc9f16429f325cdb1f37498c Mon Sep 17 00:00:00 2001
2 From: Michael Weghorn <m.weghorn@posteo.de>
3 Date: Thu, 31 Jan 2019 09:52:51 +0100
4 Subject: tdf#123077 gtk3_kde5: Set KFileWidget's custom widget only once
6 Since the event filter is only used to set the custom
7 widget in the KFileWidget, it can and needs to be removed
8 again once this has been done; which also avoids crashes.
10 (s. https://gerrit.libreoffice.org/#/c/67185/ for more
11 infos, where the same thing is done for kde5)
13 Change-Id: I5c719fb17510916b4730ed5c00bb638df2f183e3
14 Reviewed-on: https://gerrit.libreoffice.org/67184
16 Reviewed-by: Michael Weghorn <m.weghorn@posteo.de>
17 (cherry picked from commit 30cc54a4532a732a0cf6dfe9943521978ff7292f)
18 Reviewed-on: https://gerrit.libreoffice.org/67203
19 Reviewed-by: Katarina Behrens <Katarina.Behrens@cib.de>
21 vcl/unx/gtk3_kde5/kde5_filepicker.cxx | 4 ++++
22 1 file changed, 4 insertions(+)
24 diff --git a/vcl/unx/gtk3_kde5/kde5_filepicker.cxx b/vcl/unx/gtk3_kde5/kde5_filepicker.cxx
25 index 33f64ad0..9914869 100644
26 --- a/vcl/unx/gtk3_kde5/kde5_filepicker.cxx
27 +++ b/vcl/unx/gtk3_kde5/kde5_filepicker.cxx
28 @@ -265,7 +265,11 @@ bool KDE5FilePicker::eventFilter(QObject* o, QEvent* e)
30 KWindowSystem::setMainWindow(w, _winId);
31 if (auto* fileWidget = w->findChild<KFileWidget*>({}, Qt::FindDirectChildrenOnly))
33 fileWidget->setCustomWidget(_extraControls);
34 + // remove event filter again; the only purpose was to set the custom widget here
35 + qApp->removeEventFilter(this);
39 return QObject::eventFilter(o, e);