Clean out the refresh-on-demand code from EventsModel.

[pkanalytics] / events.cpp

diff --git a/events.cpp b/events.cpp
index 5752d82ca4fb5c8d7cfed40ec705e1d17f698258..4ce3e86c67604f3692ea68bc8a29c7e992feb754 100644 (file)
--- a/events.cpp
+++ b/events.cpp
@@ -10,6 +10,11 @@ using namespace std;
 
 string format_timestamp(uint64_t pos);
 
+EventsModel::EventsModel(sqlite3 *db) : db(db)
+{
+       load_data();
+}
+
 QVariant EventsModel::headerData(int section, Qt::Orientation orientation, int role) const
 {
        if (role != Qt::DisplayRole) {
@@ -33,13 +38,13 @@ QVariant EventsModel::data(const QModelIndex &index, int role) const
        if (role != Qt::DisplayRole) {
                return QVariant();
        }
-       refresh_if_needed();
        if (index.column() == 0) {
                return QString::fromUtf8(format_timestamp(events[index.row()].t));
        } else if (index.column() == 1) {
                optional<int> player_id = events[index.row()].player_id;
                if (player_id) {
-                       const Player &p = players[*player_id];
+                       auto p_it = players.find(*player_id);
+                       const Player &p = p_it->second;
                        return QString::fromUtf8(p.name + " (" + p.number + ")");
                } else {
                        return QVariant();
@@ -50,15 +55,10 @@ QVariant EventsModel::data(const QModelIndex &index, int role) const
        return QVariant();
 }
 
-void EventsModel::refresh_if_needed() const
+void EventsModel::load_data()
 {
-       if (!stale) {
-               return;
-       }
-
        players.clear();
        events.clear();
-       stale = false;
 
        // Read the players.
        sqlite3_stmt *stmt;
@@ -115,9 +115,6 @@ void EventsModel::refresh_if_needed() const
                fprintf(stderr, "SELECT finalize: %s\n", sqlite3_errmsg(db));
                abort();
        }
-
-       // TODO what if data changes externally?
-       //emit dataChanged(QModelIndex(
 }
 
 unsigned EventsModel::insert_event(uint64_t t, int player_id)