diff --git a/applets/kicker/plugin/kastatsfavoritesmodel.cpp b/applets/kicker/plugin/kastatsfavoritesmodel.cpp --- a/applets/kicker/plugin/kastatsfavoritesmodel.cpp +++ b/applets/kicker/plugin/kastatsfavoritesmodel.cpp @@ -470,8 +470,10 @@ this, [&] (const QString ¤tActivity) { DEBUG << "Activity just got changed to" << currentActivity; Q_UNUSED(currentActivity); - auto clientId = d->m_clientId; - initForClient(clientId); + if (d) { + auto clientId = d->m_clientId; + initForClient(clientId); + } }); } @@ -501,7 +503,7 @@ bool KAStatsFavoritesModel::trigger(int row, const QString &actionId, const QVariant &argument) { - return d->trigger(row, actionId, argument); + return d && d->trigger(row, actionId, argument); } bool KAStatsFavoritesModel::enabled() const @@ -547,6 +549,7 @@ void KAStatsFavoritesModel::portOldFavorites(const QStringList &ids) { + if (!d) return; DEBUG << "portOldFavorites" << ids; const auto activityId = ":global"; @@ -617,6 +620,8 @@ void KAStatsFavoritesModel::removeFavoriteFrom(const QString &id, const Activity &activity) { + if (!d || id.isEmpty()) return; + const auto url = d->normalizedId(id).value(); Q_ASSERT(!activity.values.isEmpty()); @@ -631,6 +636,8 @@ void KAStatsFavoritesModel::setFavoriteOn(const QString &id, const QString &activityId) { + if (!d || id.isEmpty()) return; + const auto url = d->normalizedId(id).value(); DEBUG << "setFavoriteOn" << id << activityId << url << " (actual)"; @@ -650,6 +657,8 @@ void KAStatsFavoritesModel::moveRow(int from, int to) { + if (!d) return; + d->move(from, to); }