From 038facd4fcca60aa92fe29bbb47e9669029d0fe9 Mon Sep 17 00:00:00 2001 From: Andrea Diamantini Date: Fri, 27 Apr 2012 01:05:20 +0200 Subject: Revert "Fix opensearch add shortcut handling" This reverts commit d09b430f9835fd19f9a0088a5620af20bdf36b0e. --- src/opensearch/opensearchmanager.cpp | 100 +++++++++++++++++------------------ src/opensearch/opensearchmanager.h | 4 +- src/opensearch/opensearchreader.cpp | 1 - src/urlbar/urlbar.cpp | 3 +- src/urlbar/webshortcutwidget.cpp | 9 ++-- src/urlbar/webshortcutwidget.h | 14 +---- src/webtab.cpp | 2 +- src/webtab.h | 12 ++--- 8 files changed, 66 insertions(+), 79 deletions(-) (limited to 'src') diff --git a/src/opensearch/opensearchmanager.cpp b/src/opensearch/opensearchmanager.cpp index 120957fa..480e9a26 100644 --- a/src/opensearch/opensearchmanager.cpp +++ b/src/opensearch/opensearchmanager.cpp @@ -115,8 +115,9 @@ bool OpenSearchManager::isSuggestionAvailable() void OpenSearchManager::addOpenSearchEngine(const KUrl &url, const QString &title, const QString &shortcut) { + Q_UNUSED(title); + m_shortcut = shortcut; - m_title = trimmedEngineName(title); if (m_state != IDLE) { @@ -169,49 +170,67 @@ void OpenSearchManager::dataReceived(KIO::Job *job, const QByteArray &data) void OpenSearchManager::jobFinished(KJob *job) { - if (!job->error() && m_state == REQ_DESCRIPTION) + // Do NOT parse if job had same errors or the typed string is empty + if (job->error() || _typedText.isEmpty()) + { + emit suggestionsReceived(_typedText, ResponseList()); + m_state = IDLE; + return; // just silently return + } + + if (m_state == REQ_SUGGESTION) + { + ResponseList suggestionsList; + if (isSuggestionAvailable()) + { + suggestionsList = m_activeEngine->parseSuggestion(_typedText, m_jobData); + } + emit suggestionsReceived(_typedText, suggestionsList); + idleJob(); + return; + } + + if (m_state == REQ_DESCRIPTION) { OpenSearchReader reader; OpenSearchEngine *engine = reader.read(m_jobData); if (engine) { - m_engineCache.insert(m_title, engine); - m_engines.insert(m_jobUrl, m_shortcut); + m_engineCache.insert(engine->name(), engine); + m_engines.insert(m_jobUrl, trimmedEngineName(engine->name())); saveEngines(); QString path; - if (engine->providesSuggestions()) + if (engine->providesSuggestions()) //save opensearch description only if it provides suggestions { - // save opensearch description only if it provides suggestions OpenSearchWriter writer; path = KGlobal::dirs()->findResource("data", "rekonq/opensearch/"); QFile file(path + trimmedEngineName(engine->name()) + ".xml"); writer.write(&file, engine); - - // save desktop file here - QString searchUrl = OpenSearchEngine::parseTemplate("\\{@}", engine->searchUrlTemplate()); - m_currentJob = NULL; - - path = KGlobal::mainComponent().dirs()->saveLocation("services", "searchproviders/"); - KConfig _service(path + m_title + ".desktop", KConfig::SimpleConfig); - KConfigGroup service(&_service, "Desktop Entry"); - service.writeEntry("Type", "Service"); - service.writeEntry("ServiceTypes", "SearchProvider"); - service.writeEntry("Name", m_title); - service.writeEntry("Query", searchUrl); - service.writeEntry("Keys", m_shortcut); - // TODO charset - service.writeEntry("Charset", "" /* provider->charset() */); - // we might be overwriting a hidden entry - service.writeEntry("Hidden", false); - service.sync(); - - // Update filters in running applications... - QDBusMessage msg = QDBusMessage::createSignal("/", "org.kde.KUriFilterPlugin", "configure"); - QDBusConnection::sessionBus().send(msg); - - emit openSearchEngineAdded(engine->name()); } + + QString searchUrl = OpenSearchEngine::parseTemplate("\\{@}", engine->searchUrlTemplate()); + m_currentJob = NULL; + + path = KGlobal::mainComponent().dirs()->saveLocation("services", "searchproviders/"); + KConfig _service(path + trimmedEngineName(engine->name()) + ".desktop", KConfig::SimpleConfig); + KConfigGroup service(&_service, "Desktop Entry"); + service.writeEntry("Type", "Service"); + service.writeEntry("ServiceTypes", "SearchProvider"); + service.writeEntry("Name", engine->name()); + service.writeEntry("Query", searchUrl); + service.writeEntry("Keys", m_shortcut); + // TODO charset + service.writeEntry("Charset", "" /* provider->charset() */); + // we might be overwriting a hidden entry + service.writeEntry("Hidden", false); + service.sync(); + + // Update filters in running applications... + QDBusMessage msg = QDBusMessage::createSignal("/", "org.kde.KUriFilterPlugin", "configure"); + QDBusConnection::sessionBus().send(msg); + + emit openSearchEngineAdded(engine->name(), searchUrl, m_shortcut); } else { @@ -219,27 +238,6 @@ void OpenSearchManager::jobFinished(KJob *job) } idleJob(); - return; - } - - // Do NOT parse if job had same errors or the typed string is empty - if (job->error() || _typedText.isEmpty()) - { - emit suggestionsReceived(_typedText, ResponseList()); - m_state = IDLE; - return; // just silently return - } - - if (m_state == REQ_SUGGESTION) - { - ResponseList suggestionsList; - if (isSuggestionAvailable()) - { - suggestionsList = m_activeEngine->parseSuggestion(_typedText, m_jobData); - } - emit suggestionsReceived(_typedText, suggestionsList); - idleJob(); - return; } } diff --git a/src/opensearch/opensearchmanager.h b/src/opensearch/opensearchmanager.h index 38179318..e698e0ee 100644 --- a/src/opensearch/opensearchmanager.h +++ b/src/opensearch/opensearchmanager.h @@ -96,7 +96,7 @@ private Q_SLOTS: Q_SIGNALS: void suggestionsReceived(const QString &text, const ResponseList &suggestion); - void openSearchEngineAdded(const QString &name); + void openSearchEngineAdded(const QString &name, const QString &searchUrl, const QString &fileName); private: QString trimmedEngineName(const QString &engineName) const; @@ -116,9 +116,7 @@ private: KUrl m_jobUrl; QString _typedText; - QString m_shortcut; - QString m_title; }; #endif // OPENSEARCHMANAGER_H diff --git a/src/opensearch/opensearchreader.cpp b/src/opensearch/opensearchreader.cpp index a4f43b7a..897fca29 100644 --- a/src/opensearch/opensearchreader.cpp +++ b/src/opensearch/opensearchreader.cpp @@ -83,7 +83,6 @@ OpenSearchEngine *OpenSearchReader::read() || namespaceUri() != QL1S("http://a9.com/-/spec/opensearch/1.1/") ) { - kDebug() << "The file is not an OpenSearch 1.1 file: " << name(); raiseError(i18n("The file is not an OpenSearch 1.1 file.")); return engine; } diff --git a/src/urlbar/urlbar.cpp b/src/urlbar/urlbar.cpp index 97a3c1c7..2b4b7bf0 100644 --- a/src/urlbar/urlbar.cpp +++ b/src/urlbar/urlbar.cpp @@ -134,7 +134,8 @@ UrlBar::UrlBar(QWidget *parent) connect(_tab->view(), SIGNAL(iconChanged()), this, SLOT(refreshFavicon())); // search icon - connect(rApp->opensearchManager(), SIGNAL(openSearchEngineAdded(QString)), this, SLOT(updateRightIcons())); + connect(rApp->opensearchManager(), SIGNAL(openSearchEngineAdded(QString,QString,QString)), + this, SLOT(updateRightIcons())); // bookmark icon connect(rApp->bookmarkManager(), SIGNAL(bookmarksUpdated()), this, SLOT(updateRightIcons())); diff --git a/src/urlbar/webshortcutwidget.cpp b/src/urlbar/webshortcutwidget.cpp index c6dc3b79..409d0b32 100644 --- a/src/urlbar/webshortcutwidget.cpp +++ b/src/urlbar/webshortcutwidget.cpp @@ -4,7 +4,6 @@ * * Copyright (C) 2009 by Fredy Yanardi * Copyright (C) 2010-2011 by Lionel Chauvin -* Copyright (C) 2012 by Andrea Diamantini * * * This program is free software; you can redistribute it and/or @@ -28,7 +27,7 @@ // Self Includes #include "webshortcutwidget.h" -#include "webshortcutwidget.moc" +#include "rekonq_defines.h" // KDE Includes #include @@ -114,8 +113,6 @@ void WebShortcutWidget::show(const KUrl &url, const QString &openSearchName, con void WebShortcutWidget::accept() { emit webShortcutSet(m_url, m_nameLineEdit->text(), m_wsLineEdit->text()); - - // close widget (and destroy it) close(); } @@ -161,3 +158,7 @@ void WebShortcutWidget::shortcutsChanged(const QString& newShorthands) } } } + +#include "webshortcutwidget.moc" + + diff --git a/src/urlbar/webshortcutwidget.h b/src/urlbar/webshortcutwidget.h index d1291714..bc072f47 100644 --- a/src/urlbar/webshortcutwidget.h +++ b/src/urlbar/webshortcutwidget.h @@ -4,7 +4,6 @@ * * Copyright (C) 2009 by Fredy Yanardi * Copyright (C) 2010-2011 by Lionel Chauvin -* Copyright (C) 2012 by Andrea Diamantini * * * This program is free software; you can redistribute it and/or @@ -29,24 +28,15 @@ #ifndef WEBSHORTCUTWIDGET_H #define WEBSHORTCUTWIDGET_H - -// Rekonq Includes -#include "rekonq_defines.h" - -// KDE Includes +#include #include #include -// Qt Includes -#include - -// Forward Declarations class QLabel; class QLineEdit; class QPushButton; - -class REKONQ_TESTS_EXPORT WebShortcutWidget : public QMenu +class WebShortcutWidget : public QMenu { Q_OBJECT public: diff --git a/src/webtab.cpp b/src/webtab.cpp index 09c64054..57f46ea4 100644 --- a/src/webtab.cpp +++ b/src/webtab.cpp @@ -318,7 +318,7 @@ void WebTab::showSearchEngine(const QPoint &pos) connect(widget, SIGNAL(webShortcutSet(KUrl,QString,QString)), rApp->opensearchManager(), SLOT(addOpenSearchEngine(KUrl,QString,QString))); - connect(rApp->opensearchManager(), SIGNAL(openSearchEngineAdded(QString)), + connect(rApp->opensearchManager(), SIGNAL(openSearchEngineAdded(QString,QString,QString)), this, SLOT(openSearchEngineAdded())); widget->show(extractOpensearchUrl(e), title, pos); diff --git a/src/webtab.h b/src/webtab.h index 39bffdb7..5c04b683 100644 --- a/src/webtab.h +++ b/src/webtab.h @@ -33,21 +33,21 @@ #include "rekonq_defines.h" // Local Includes +#include "webpage.h" #include "webview.h" // KDE Includes #include // Qt Includes -#include +#include // Forward Declarations -class NotificationBar; -class PreviewSelectorBar; -class QPoint; class UrlBar; +class PreviewSelectorBar; class WalletBar; -class WebPage; +class NotificationBar; +class QPoint; class REKONQ_TESTS_EXPORT WebTab : public QWidget @@ -97,7 +97,7 @@ private Q_SLOTS: void showRSSInfo(const QPoint &pos); void showSearchEngine(const QPoint &pos); void openSearchEngineAdded(); - + void showMessageBar(); private: -- cgit v1.2.1