Try to fix race condition causing us to loose initial updates.

The maillist sometimes misses updates, and this might be the root cause
of it. Because it's difficult to test we'll just try this fix for now.

1 files changed, 2 insertions, 1 deletions

diff --git a/common/queryrunner.cpp b/common/queryrunner.cpp
index 2062828..73ccf6d 100644
--- a/common/queryrunner.cpp
+++ b/common/queryrunner.cpp
@@ -77,9 +77,10 @@ QueryRunner<DomainType>::QueryRunner(const Sink::Query &query, const Sink::Resou
         // Incremental updates are always loaded directly, leaving it up to the result to discard the changes if they are not interesting
         setQuery([=]() { return incrementalFetch(query, bufferType); });
         // Ensure the connection is open, if it wasn't already opened
-        // TODO If we are not connected already, we have to check for the latest revision once connected, otherwise we could miss some updates
         mResourceAccess->open();
         QObject::connect(mResourceAccess.data(), &Sink::ResourceAccess::revisionChanged, this, &QueryRunner::revisionChanged);
+        // open is not synchronous, so from the time when the initial query is started until we have started and connected to the resource, it's possible to miss updates. We therefore unconditionally try to fetch new entities once we are connected.
+        QObject::connect(mResourceAccess.data(), &Sink::ResourceAccess::ready, this, &QueryRunner::revisionChanged);
     }
     mResultProvider->onDone([this]() {
         delete this;