Help: Remove duplicate results

Workaround for QTBUG-108131

Change-Id: If3de18249fe11b753323c5375559d5ffd0ef0673
Reviewed-by: Eike Ziller <eike.ziller@qt.io>

src/plugins/help/helpindexfilter.cpp

@@ -101,10 +101,7 @@ void HelpIndexFilter::accept(const LocatorFilterEntry &selection,
     Q_UNUSED(selectionStart)
     Q_UNUSED(selectionLength)
     const QString &key = selection.displayName;
-    QMultiMap<QString, QUrl> links;
+    const QMultiMap<QString, QUrl> links = LocalHelpManager::linksForKeyword(key);
-    const QList<QHelpLink> docs = LocalHelpManager::helpEngine().documentsForKeyword(key, QString());
-    for (const auto &doc : docs)
-        links.insert(doc.title, doc.url);
     emit linksActivated(links, key);
 }
 

src/plugins/help/helpmanager.cpp

@@ -195,17 +195,32 @@ QSet<QString> HelpManager::userDocumentationPaths()
     return d->m_userRegisteredFiles;
 }
 
-// This should go into Qt 4.8 once we start using it for Qt Creator
+QMultiMap<QString, QUrl> HelpManager::linksForKeyword(QHelpEngineCore *engine,
+                                                      const QString &key,
+                                                      std::optional<QString> filterName)
+{
+    QMultiMap<QString, QUrl> links;
+    const QList<QHelpLink> docs = filterName.has_value()
+                                      ? engine->documentsForKeyword(key, filterName.value())
+                                      : engine->documentsForKeyword(key);
+
+    for (const auto &doc : docs)
+        links.insert(doc.title, doc.url);
+
+    // Remove duplicates (workaround for QTBUG-108131)
+    links.removeIf([&links](const QMultiMap<QString, QUrl>::iterator it) {
+        return links.find(it.key(), it.value()) != it;
+    });
+
+    return links;
+}
+
 QMultiMap<QString, QUrl> HelpManager::linksForKeyword(const QString &key)
 {
     QTC_ASSERT(!d->m_needsSetup, return {});
     if (key.isEmpty())
         return {};
-    QMultiMap<QString, QUrl> links;
+    return HelpManager::linksForKeyword(d->m_helpEngine, key, QString());
-    const QList<QHelpLink> docs = d->m_helpEngine->documentsForKeyword(key, QString());
-    for (const auto &doc : docs)
-        links.insert(doc.title, doc.url);
-    return links;
 }
 
 QMultiMap<QString, QUrl> HelpManager::linksForIdentifier(const QString &id)

src/plugins/help/helpmanager.h

@@ -8,6 +8,7 @@
 QT_FORWARD_DECLARE_CLASS(QUrl)
 
 #include <QFutureInterface>
+#include <QHelpEngineCore>
 #include <QVariant>
 
 namespace Help {
@@ -34,6 +35,9 @@ public:
 
     QMultiMap<QString, QUrl> linksForIdentifier(const QString &id) override;
     QMultiMap<QString, QUrl> linksForKeyword(const QString &key) override;
+    static QMultiMap<QString, QUrl> linksForKeyword(QHelpEngineCore *engine,
+                                                    const QString &key,
+                                                    std::optional<QString> filterName);
 
     static QUrl findFile(const QUrl &url);
     QByteArray fileData(const QUrl &url) override;

src/plugins/help/localhelpmanager.cpp

@@ -33,6 +33,7 @@
 #include <QDesktopServices>
 #include <QFontDatabase>
 #include <QHelpEngine>
+#include <QHelpLink>
 #include <QMutexLocker>
 
 #include <optional>
@@ -521,3 +522,8 @@ bool LocalHelpManager::openOnlineHelp(const QUrl &url)
     }
     return false;
 }
+
+QMultiMap<QString, QUrl> LocalHelpManager::linksForKeyword(const QString &keyword)
+{
+    return HelpManager::linksForKeyword(&LocalHelpManager::helpEngine(), keyword, std::nullopt);
+}

src/plugins/help/localhelpmanager.h

@@ -99,6 +99,8 @@ public:
     static bool canOpenOnlineHelp(const QUrl &url);
     static bool openOnlineHelp(const QUrl &url);
 
+    static QMultiMap<QString, QUrl> linksForKeyword(const QString &keyword);
+
 signals:
     void fallbackFontChanged(const QFont &font);
     void fontZoomChanged(int percentage);

src/shared/help/indexwindow.cpp

@@ -175,10 +175,7 @@ void IndexWindow::disableSearchLineEdit()
 void IndexWindow::open(const QModelIndex &index, bool newPage)
 {
     const QString keyword = m_filteredIndexModel->data(index, Qt::DisplayRole).toString();
-    QMultiMap<QString, QUrl> links;
+    const QMultiMap<QString, QUrl> links = LocalHelpManager::linksForKeyword(keyword);
-    const QList<QHelpLink> docs = LocalHelpManager::helpEngine().documentsForKeyword(keyword);
-    for (const auto &doc : docs)
-        links.insert(doc.title, doc.url);
 
     emit linksActivated(links, keyword, newPage);
 }