{api/base,gui/options}-dialog: prevent closing without signal

Use hide() to avoid emitting idempotent events. There's isVisible() but
no isClosed() or equivalent. Worse yet, close() can return true twice in
a row, despite what docs for `bool QWidget::close()' say.

2 files changed, 11 insertions, 1 deletions

diff --git a/api/plugin-api.cpp b/api/plugin-api.cpp
index 1bf56200..100902eb 100644
--- a/api/plugin-api.cpp
+++ b/api/plugin-api.cpp
@@ -11,7 +11,13 @@ ITracker::~ITracker() {}
 ITrackerDialog::~ITrackerDialog() {}
 
 plugin_api::detail::BaseDialog::BaseDialog() {}
-void plugin_api::detail::BaseDialog::closeEvent(QCloseEvent*) { emit closing(); }
+
+void plugin_api::detail::BaseDialog::closeEvent(QCloseEvent*)
+{
+    if (isVisible())
+        emit closing();
+}
+
 Metadata::Metadata() {}
 IFilter::IFilter() {}
 IFilterDialog::IFilterDialog() {}
@@ -19,3 +25,5 @@ IProtocol::IProtocol() {}
 IProtocolDialog::IProtocolDialog() {}
 ITracker::ITracker() {}
 ITrackerDialog::ITrackerDialog() {}
+
+void plugin_api::detail::BaseDialog::done(int) { close(); hide(); }
diff --git a/api/plugin-api.hpp b/api/plugin-api.hpp
index e9de9dad..4c9dee2f 100644
--- a/api/plugin-api.hpp
+++ b/api/plugin-api.hpp
@@ -48,6 +48,8 @@ public:
     void closeEvent(QCloseEvent *) override;
 signals:
     void closing();
+private slots:
+    void done(int) override;
 };
 
 } // ns