From 88095fdf22c25b4458ed38cf7a1b347467513ec9 Mon Sep 17 00:00:00 2001
From: Iceyer <me@iceyer.net>
Date: Thu, 29 Mar 2018 16:01:21 +0800
Subject: [PATCH] fix: isValid not work in flatpak

Change-Id: Ia1986838b932ef9748a66bdf35cf7be5b8a74a85
---
 debian/rules                 |  5 ++---
 src/widgets/dapplication.cpp | 23 +++++------------------
 2 files changed, 7 insertions(+), 21 deletions(-)

diff --git a/debian/rules b/debian/rules
index af51b5a22..9ff638850 100755
--- a/debian/rules
+++ b/debian/rules
@@ -20,8 +20,7 @@ endif
 %:
 	dh $@ --parallel
 
-#override_dh_auto_configure:
-#	dh_auto_configure -- LIB_INSTALL_DIR=/usr/lib/$(DEB_HOST_MULTIARCH) VERSION=$(CONFIG_VERSION)
-
+override_dh_auto_configure:
+	dh_auto_configure -- LIB_INSTALL_DIR=/usr/lib/$(DEB_HOST_MULTIARCH)
 override_dh_makeshlibs:
 	dh_makeshlibs -V
diff --git a/src/widgets/dapplication.cpp b/src/widgets/dapplication.cpp
index f51c488fe..dac2a93f9 100644
--- a/src/widgets/dapplication.cpp
+++ b/src/widgets/dapplication.cpp
@@ -670,27 +670,14 @@ void DApplication::handleHelpAction()
     QDBusInterface manual("com.deepin.Manual.Open",
                           "/com/deepin/Manual/Open",
                           "com.deepin.Manual.Open");
-    if (manual.isValid()) {
-        manual.asyncCall("ShowManual", appid);
+    QDBusReply<void> reply = manual.call("ShowManual", appid);
+    if (reply.isValid())  {
+        qDebug() << "call com.deepin.Manual.Open success";
         return;
     }
-
+    qDebug() << "call com.deepin.Manual.Open failed" << reply.error();
     // fallback to old interface
-    if (!qgetenv("FLATPAK_APPID").isEmpty()) {
-        appid = qgetenv("FLATPAK_APPID");
-        QDBusInterface legacydman("com.deepin.dman",
-                                  "/com/deepin/dman",
-                                  "com.deepin.dman");
-        if (legacydman.isValid()) {
-            // will block application
-            legacydman.call("ShowManual", appid);
-            return;
-        }
-
-        qWarning() << "can not call dman dbus interface";
-    } else {
-        QProcess::startDetached("dman", QStringList() << appid);
-    }
+    QProcess::startDetached("dman", QStringList() << appid);
 #else
     qWarning() << "not support dman now";
 #endif