D19836: Add TeamdriveModifyJob and update teamdrive example to allow renaming a Team Drive.

Daniel Vrátil noreply at phabricator.kde.org
Sun Mar 17 17:43:22 GMT 2019


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

INLINE COMMENTS

> teamdrivecreatejobtest.cpp:2
>  /*
> - * Createright (C) 2019  David Barchiesi <david at barchie.si>
>   *

Oh. I love this! :D

> teamdrivedeletejobtest.cpp:2
>  /*
> - * Deleteright (C) 2019  David Barchiesi <david at barchie.si>
>   *

I love this one even more! :D

> teamdrivemodifyjob.cpp:107
> +        setErrorString(tr("Invalid response content type"));
> +        emitFinished();
> +    }

You should `return` here after calling `emitFinished()`: processing another item after emitting result can lead to some weird states and behavior.

REPOSITORY
  R477 KGAPI Library

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

To: barchiesi, dvratil
Cc: kde-pim, #libkgapi, barchiesi, gennad, dvasin, rodsevich, winterz, vkrause, mlaurent, knauss, dvratil
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-pim/attachments/20190317/ae950e8a/attachment.html>


More information about the kde-pim mailing list