D6507: KMail: configure plugin fixes for Akonadi instances

[Laurent Montel]

mlaurent requested changes to this revision.
mlaurent added inline comments.
This revision now requires changes to proceed.

INLINE COMMENTS

> configurepluginslistwidget.cpp:252
>      for (const Akonadi::AgentType &type : lstAgent) {
> -        if (type.identifier() == service) {
> +        QString id = type.identifier();
> +        if (Akonadi::ServerManager::hasInstanceIdentifier()) {

I don't understand you need to add .instanceIdentifier to list of agent ?
so agent doesn't have .instanceIdentifier() ?

So I don't understand.
Why instance doesn't have .instanceIdentifier() ?

could you verify which name have each instance in your system please ?

REPOSITORY
  R206 KMail

REVISION DETAIL
  https://phabricator.kde.org/D6507

To: winterz, dvratil, mlaurent
Cc: #kde_pim, dvasin, winterz, vkrause, mlaurent, knauss, dvratil
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-pim/attachments/20170705/2ab75b2b/attachment.html>