Commit 883d991d authored by David Faure's avatar David Faure

NotificationSubscriber: fix regression introduced by my commit 61ae4984

Instead of resolving all aliases in this hot code path, only add
a special case for text/directory (alias for text/vcard).

Thanks to Konrad Czapla for the investigation of the issue.
parent ce907f4d
Pipeline #45097 failed with stage
in 19 minutes and 40 seconds
......@@ -350,6 +350,11 @@ bool NotificationSubscriber::isCollectionMonitored(Entity::Id id) const
bool NotificationSubscriber::isMimeTypeMonitored(const QString &mimeType) const
{
// Assumes mLock being locked by caller
// KContacts::Addressee::mimeType() unfortunately uses an alias
if (mimeType == QLatin1String("text/directory")) {
return mMonitoredMimeTypes.contains(QStringLiteral("text/vcard"));
}
return mMonitoredMimeTypes.contains(mimeType);
}
......
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