[Kmymoney-devel] Review Request 119207: Refactor the contact fetching code into a single class.

[Marko Käning]

> On July 10, 2014, 7:47 a.m., Marko Käning wrote:
> > Hi Cristian, can you supply also a diff for branch frameworks, or is that still too early to ask for? Greets, Marko
> 
> Cristian Oneț wrote:
>     I posted this review request against master since I think that master should also contain this improvement. If it will be pushed then I'll pick it into frameworks without going trough the review process.

Ah, ok.

Things like KF5QGpgme are going to follow later, I figure, right?


- Marko


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/119207/#review62037
-----------------------------------------------------------


On July 10, 2014, 7:24 a.m., Cristian Oneț wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/119207/
> -----------------------------------------------------------
> 
> (Updated July 10, 2014, 7:24 a.m.)
> 
> 
> Review request for KMymoney.
> 
> 
> Repository: kmymoney
> 
> 
> Description
> -------
> 
> Refactor the contact fetching code into a single class.
> 
> This avoids code duplication and makes it easier to transform
> the kdepimlibs dependency into an optional one. If kdepimlibs
> is not detected the user will not be able to load contact data
> from the address book (big deal...).
> 
> It also makes it easier to fetch contact data from other parts of the
> code, like for a payee.
> 
> I'm not sure about putting this into the kmm_mymoney library but since
> it only contains a data source and we don't have another lib for these
> kind of objects it ended up here.
> 
> 
> Diffs
> -----
> 
>   kmymoney/wizards/newuserwizard/knewuserwizard.cpp 7fd250cfc46db505dd8a1b595a2179c91e0183d9 
>   kmymoney/wizards/newuserwizard/knewuserwizard_p.h adb4ca564cdd4b70e8cb4c4f1d053c0bd6335c38 
>   kmymoney/mymoney/CMakeLists.txt 62f29e88017fae38b1da8f2a18cb35c5b627b573 
>   kmymoney/mymoney/mymoneycontact.h PRE-CREATION 
>   kmymoney/mymoney/mymoneycontact.cpp PRE-CREATION 
>   config-kmymoney.h.cmake 5f3845a46413259d60302ba1b921a266ee3bb1f9 
>   CMakeLists.txt dacdf606a9320c1926c4d848d0917708f8cdd44f 
>   kmymoney/dialogs/knewfiledlg.h 92004714c8bedf8b1cf15bfc1e0f20edcab3a66c 
>   kmymoney/dialogs/knewfiledlg.cpp 5b6e8e960184235e3f6bc61439432ea48855a4f4 
> 
> Diff: https://git.reviewboard.kde.org/r/119207/diff/
> 
> 
> Testing
> -------
> 
> Tested the "Load from addressbook" feature, it works. Also compiled the module without finding the kdepimlibs module, it worked.
> 
> 
> Thanks,
> 
> Cristian Oneț
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kmymoney-devel/attachments/20140710/9c58b733/attachment.html>