diff options
-rw-r--r-- | src/opensearch/opensearchengine.cpp | 6 | ||||
-rw-r--r-- | src/opensearch/opensearchengine.h | 5 | ||||
-rw-r--r-- | src/opensearch/opensearchmanager.cpp | 2 | ||||
-rw-r--r-- | src/opensearch/opensearchmanager.h | 6 | ||||
-rw-r--r-- | src/opensearch/opensearchreader.cpp | 15 | ||||
-rw-r--r-- | src/opensearch/opensearchwriter.cpp | 4 | ||||
-rw-r--r-- | src/opensearch/suggestionparser.cpp | 18 | ||||
-rw-r--r-- | src/opensearch/suggestionparser.h | 22 |
8 files changed, 45 insertions, 33 deletions
diff --git a/src/opensearch/opensearchengine.cpp b/src/opensearch/opensearchengine.cpp index f100bb60..9d5e866f 100644 --- a/src/opensearch/opensearchengine.cpp +++ b/src/opensearch/opensearchengine.cpp @@ -35,8 +35,10 @@ // Qt Includes #include <QtCore/QRegExp> -OpenSearchEngine::OpenSearchEngine(QObject *) - : m_parser(0) + +OpenSearchEngine::OpenSearchEngine(QObject *parent) + : QObject(parent) + , m_parser(0) { } diff --git a/src/opensearch/opensearchengine.h b/src/opensearch/opensearchengine.h index 03be1212..b3965ecb 100644 --- a/src/opensearch/opensearchengine.h +++ b/src/opensearch/opensearchengine.h @@ -47,11 +47,12 @@ class QNetworkAccessManager; class QNetworkReply; -class OpenSearchEngine + +class OpenSearchEngine : public QObject { public: typedef QPair<QString, QString> Parameter; - + OpenSearchEngine(QObject *parent = 0); ~OpenSearchEngine(); diff --git a/src/opensearch/opensearchmanager.cpp b/src/opensearch/opensearchmanager.cpp index ed8c5fc5..e5afc144 100644 --- a/src/opensearch/opensearchmanager.cpp +++ b/src/opensearch/opensearchmanager.cpp @@ -168,7 +168,7 @@ void OpenSearchManager::jobFinished(KJob *job) { const ResponseList suggestionsList = m_activeEngine->parseSuggestion(m_jobData); kDebug() << "Received suggestions in "<< _typedText << " from " << m_activeEngine->name() << ": "; - foreach(Response r, suggestionsList) + Q_FOREACH(const Response &r, suggestionsList) { kDebug() << r.title; } diff --git a/src/opensearch/opensearchmanager.h b/src/opensearch/opensearchmanager.h index 43f11640..f935a6d7 100644 --- a/src/opensearch/opensearchmanager.h +++ b/src/opensearch/opensearchmanager.h @@ -79,7 +79,7 @@ public: void addOpenSearchEngine(const KUrl &url, const QString &title); -public slots: +public Q_SLOTS: /** * Ask the specific suggestion engine to request for suggestion for the search text * @@ -87,11 +87,11 @@ public slots: */ void requestSuggestion(const QString &searchText); -private slots: +private Q_SLOTS: void dataReceived(KIO::Job *job, const QByteArray &data); void jobFinished(KJob *job); -signals: +Q_SIGNALS: void suggestionReceived(const QString &text, const ResponseList &suggestion); void openSearchEngineAdded(const QString &name, const QString &searchUrl, const QString &fileName); diff --git a/src/opensearch/opensearchreader.cpp b/src/opensearch/opensearchreader.cpp index ebca7877..1c106afa 100644 --- a/src/opensearch/opensearchreader.cpp +++ b/src/opensearch/opensearchreader.cpp @@ -34,6 +34,9 @@ #include "opensearchengine.h" #include "suggestionparser.h" +// KDE Includes +#include <KLocalizedString> + // Qt Includes #include <QtCore/QIODevice> @@ -79,7 +82,7 @@ OpenSearchEngine *OpenSearchReader::read() || namespaceUri() != QL1S("http://a9.com/-/spec/opensearch/1.1/") ) { - raiseError(QObject::tr("The file is not an OpenSearch 1.1 file.")); + raiseError(i18n("The file is not an OpenSearch 1.1 file.")); return engine; } @@ -119,7 +122,10 @@ OpenSearchEngine *OpenSearchReader::read() while (!(isEndElement() && name() == QL1S("Url"))) { - if (!isStartElement() || (name() != QL1S("Param") && name() != QL1S("Parameter"))) { + if (!isStartElement() + || (name() != QL1S("Param") + && name() != QL1S("Parameter"))) + { readNext(); continue; } @@ -138,14 +144,15 @@ OpenSearchEngine *OpenSearchReader::read() } } - if (type == QLatin1String("text/html")) + if (type == QL1S("text/html")) { engine->setSearchUrlTemplate(url); engine->setSearchParameters(parameters); } else { - if (engine->suggestionsUrlTemplate().isEmpty() && type == QL1S("application/x-suggestions+json")) //note: xml is prefered + if (engine->suggestionsUrlTemplate().isEmpty() + && type == QL1S("application/x-suggestions+json")) //note: xml is prefered { engine->setSuggestionsUrlTemplate(url); engine->setSuggestionsParameters(parameters); diff --git a/src/opensearch/opensearchwriter.cpp b/src/opensearch/opensearchwriter.cpp index 00a2f0cd..1b34939f 100644 --- a/src/opensearch/opensearchwriter.cpp +++ b/src/opensearch/opensearchwriter.cpp @@ -32,14 +32,10 @@ // Local Includes #include "opensearchengine.h" -// KDE Includes -#include <KDebug> - // Qt Includes #include <QtCore/QIODevice> - OpenSearchWriter::OpenSearchWriter() : QXmlStreamWriter() { diff --git a/src/opensearch/suggestionparser.cpp b/src/opensearch/suggestionparser.cpp index 3b656a3f..98bb8a69 100644 --- a/src/opensearch/suggestionparser.cpp +++ b/src/opensearch/suggestionparser.cpp @@ -24,18 +24,15 @@ * * ============================================================ */ -#define QL1C(x) QLatin1Char(x) -#define QL1S(x) QLatin1String(x) // Self Includes #include "suggestionparser.h" -// Local Includes -#include<KDebug> // Qt Includes #include<QByteArray> #include<QStringList> + ResponseList SuggestionParser::parse(const QByteArray &) { return ResponseList(); @@ -71,19 +68,22 @@ ResponseList XMLParser::parse(const QByteArray &resp) m_reader.readNext(); - while(!(m_reader.isEndElement() && m_reader.name() == "Item")) + while( !(m_reader.isEndElement() && m_reader.name() == QL1S("Item")) ) { if(m_reader.isStartElement()) { - if (m_reader.name() == QL1S("Text")) title = m_reader.readElementText(); - if (m_reader.name() == QL1S("Url")) url = m_reader.readElementText(); + if (m_reader.name() == QL1S("Text")) + title = m_reader.readElementText(); + if (m_reader.name() == QL1S("Url")) + url = m_reader.readElementText(); if (m_reader.name() == QL1S("Image")) { image = m_reader.attributes().value("source").toString(); image_width = m_reader.attributes().value("width").toString().toInt(); image_height = m_reader.attributes().value("height").toString().toInt(); } - if (m_reader.name() == QL1S("Description")) description = m_reader.readElementText(); + if (m_reader.name() == QL1S("Description")) + description = m_reader.readElementText(); } m_reader.readNext(); @@ -132,7 +132,7 @@ ResponseList JSONParser::parse(const QByteArray &resp) QStringList responsePartsList; qScriptValueToSequence(responseParts.property(1), responsePartsList); - foreach(QString s, responsePartsList) + Q_FOREACH(const QString &s, responsePartsList) { rlist << Response(s); } diff --git a/src/opensearch/suggestionparser.h b/src/opensearch/suggestionparser.h index 3b72c5d3..158da51f 100644 --- a/src/opensearch/suggestionparser.h +++ b/src/opensearch/suggestionparser.h @@ -24,19 +24,20 @@ * * ============================================================ */ + #ifndef SUGGESTIONPARSER_H #define SUGGESTIONPARSER_H -// Self Includes - -// Local Includes +// Rekonq Includes +#include "rekonq_defines.h" // Qt Includes -#include <QList> -#include <QtScript/QScriptEngine> +#include <QtCore/QList> #include <QtCore/QXmlStreamReader> +#include <QtScript/QScriptEngine> + class Response { @@ -80,6 +81,10 @@ public: {}; }; + +// ----------------------------------------------------------------- + + typedef QList <Response> ResponseList; @@ -88,7 +93,7 @@ class SuggestionParser public: virtual ~SuggestionParser(); virtual ResponseList parse(const QByteArray &resp); - virtual QString type(){ return ""; } + virtual QString type() = 0; }; @@ -99,9 +104,10 @@ protected: public: ResponseList parse(const QByteArray &resp); - QString type(){ return "application/x-suggestions+xml"; } + inline QString type() { return QL1S("application/x-suggestions+xml"); } }; + class JSONParser : public SuggestionParser { private: @@ -109,7 +115,7 @@ private: public: ResponseList parse(const QByteArray &resp); - QString type(){ return "application/x-suggestions+json"; } + inline QString type() { return QL1S("application/x-suggestions+json"); } }; #endif //SUGGESTIONPARSER_H |