Better cache handling. Added clear cache button to settings.

[jenirok] / src / daemon / calllistener.cpp

diff --git a/src/daemon/calllistener.cpp b/src/daemon/calllistener.cpp
index 9071713..114a05e 100644 (file)
--- a/src/daemon/calllistener.cpp
+++ b/src/daemon/calllistener.cpp
@@ -18,9 +18,10 @@
 
 #include <QtCore/QDebug>
 #include <QtSql/QSqlQuery>
+#include <QtSql/QSqlError>
 #include "calllistener.h"
 #include "settings.h"
-#include "db.h"
+#include "cache.h"
 
 namespace
 {
@@ -66,6 +67,7 @@ void CallListener::begin()
 
     eniro_->setMaxResults(1);
     eniro_->setFindNumber(false);
+    eniro_->setTimeout(REQUEST_TIMEOUT);
 
     connect(eniro_, SIGNAL(requestFinished(QVector <Eniro::Result> const&,
                                            Eniro::SearchDetails const&, bool)),
@@ -74,7 +76,7 @@ void CallListener::begin()
 
     box_ = new InformationBox();
     label_ = new QLabel("", box_);
-    label_->setMargin(10);
+    label_->setMargin(8);
     box_->setWidget(label_);
 
 }
@@ -105,29 +107,21 @@ void CallListener::end()
 
 void CallListener::search(Eniro::SearchDetails const& details)
 {
-    label_->setText(tr("Searching..."));
-    box_->show();
-
-    DB::connect();
-
-    QSqlQuery query;
-    query.prepare("SELECT name, street, city FROM cache WHERE number = :number");
-    query.bindValue(":number", details.query);
+    Eniro::Result result;
 
-    if(query.exec() && query.next())
+    if(Cache::instance().findItem(details.query, result))
     {
-        showResult(createResult(query.value(0).toString(),
-                                query.value(1).toString(),
-                                query.value(2).toString()));
 
+        showResult(createResult(result.name,
+                                result.street,
+                                result.city));
     }
     else
     {
+        showResult(tr("Searching..."));
         eniro_->search(details);
     }
 
-    DB::disconnect();
-
 }
 
 void CallListener::requestFinished(QVector <Eniro::Result> const& results,
@@ -147,7 +141,7 @@ void CallListener::requestFinished(QVector <Eniro::Result> const& results,
     if(error)
     {
         qDebug() << "Error: " << eniro_->errorString();
-        message = tr("Search failed:") + " " + eniro_->errorString();
+        message = tr("Search failed:") + " " + eniro_->errorString() + ".";
     }
     else if(results.size() == 0)
     {
@@ -156,33 +150,9 @@ void CallListener::requestFinished(QVector <Eniro::Result> const& results,
     else
     {
         message = createResult(results.at(0).name, results.at(0).street, results.at(0).city);
-        QSqlQuery query;
-
-        DB::connect();
-
-        query.prepare("INSERT INTO cache(number, name, street, city) VALUES(:number, :name, :street, :city)");
-        query.bindValue(":number", details.query);
-        query.bindValue(":name", results.at(0).name);
-        query.bindValue(":street", results.at(0).street);
-        query.bindValue(":city", results.at(0).city);
-
-        if(!query.exec())
-        {
-            qDebug() << "Unable to save cache";
-        }
-
-        QString cacheSize = Settings::instance()->get("cache_size");
-
-        // Delete old entries from cache
-        if(cacheSize.toInt() > 0)
-        {
-            if(!query.exec("DELETE c1 FROM cache AS c1 LEFT JOIN (SELECT id FROM cache ORDER BY id DESC LIMIT " + cacheSize + ") AS c2 ON c1.id = c2.id WHERE c2.id IS NULL"))
-            {
-                qDebug() << "Unable to delete old cache entries";
-            }
-        }
-
-        DB::disconnect();
+        Eniro::Result result = results.at(0);
+        result.number = details.query;
+        Cache::instance().addItem(result);
     }
 
     showResult(message);
@@ -210,8 +180,12 @@ QString CallListener::createResult(QString const& name, QString const& street, Q
 
 void CallListener::showResult(QString const& text)
 {
-    label_->setText(text);
-    box_->hide();
+    label_->setText("<font color='black'>" + text + "</font>");
+
+    if(box_->isVisible())
+    {
+        box_->hide();
+    }
     box_->show();
 }