Verified Commit ab1cfb6b authored by Daniel Vrátil's avatar Daniel Vrátil 🤖
Browse files

Monitor: enable subscription signals when monitorAll is true

There doesn't seem to be any reason why the Monitor should not
send out notification about collection subscription when 'monitorAll'
is enabled, so drop that and emit it always.
parent 44163cf9
...@@ -1132,11 +1132,9 @@ bool MonitorPrivate::emitCollectionNotification(const Protocol::CollectionChange ...@@ -1132,11 +1132,9 @@ bool MonitorPrivate::emitCollectionNotification(const Protocol::CollectionChange
case Protocol::CollectionChangeNotification::Remove: case Protocol::CollectionChangeNotification::Remove:
return emitToListeners(&Monitor::collectionRemoved, collection); return emitToListeners(&Monitor::collectionRemoved, collection);
case Protocol::CollectionChangeNotification::Subscribe: case Protocol::CollectionChangeNotification::Subscribe:
// ### why?? return emitToListeners(&Monitor::collectionSubscribed, collection, parent);
return !monitorAll && emitToListeners(&Monitor::collectionSubscribed, collection, parent);
case Protocol::CollectionChangeNotification::Unsubscribe: case Protocol::CollectionChangeNotification::Unsubscribe:
// ### why?? return emitToListeners(&Monitor::collectionUnsubscribed, collection);
return !monitorAll && emitToListeners(&Monitor::collectionUnsubscribed, collection);
default: default:
qCDebug(AKONADICORE_LOG) << "Unknown operation type" << msg.operation() << "in collection change notification"; qCDebug(AKONADICORE_LOG) << "Unknown operation type" << msg.operation() << "in collection change notification";
return false; return false;
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment