D23979: Add missing parameters to Permission related jobs and fix url typo.
Daniel Vrátil
noreply at phabricator.kde.org
Tue Sep 17 13:04:52 BST 2019
dvratil requested changes to this revision.
dvratil added a comment.
This revision now requires changes to proceed.
Thanks for the fix. Could you also modify the constructors so that the class members are initialized using the `fooDefault` constants as well, so that we really only have a single place in the entire file where the default value is defined?
REPOSITORY
R477 KGAPI Library
REVISION DETAIL
https://phabricator.kde.org/D23979
To: barchiesi, dvratil
Cc: #libkgapi, kde-pim, barchiesi, fbampaloukas, dvasin, rodsevich, winterz, vkrause, mlaurent, knauss, dvratil
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-pim/attachments/20190917/e3548413/attachment.html>
More information about the kde-pim
mailing list