Review Request 128478: Only import OFX requested date range
Thomas Baumgart
tbaumgart at kde.org
Wed Jul 20 05:55:27 UTC 2016
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/128478/#review97637
-----------------------------------------------------------
Your patch in general seems OK if you only have OFX accounts. It will interfere though with other online transaction downloads e.g. HBCI.
Since the problem and the solution are very OFX specific, I suggest to move the logic into the OFX plugin. I would move the calculation of the **startDate** into **bool OfxImporterPlugin::import(const QString& filename)** and the filter logic at the end of **int OfxImporterPlugin::ofxTransactionCallback(struct OfxTransactionData data, void * pv)** where unhandled transaction types are already eliminated.
"a/kmymoney/kmymoney/converter/mymoneystatementreader-1b81833-left.cpp" (line 402)
<https://git.reviewboard.kde.org/r/128478/#comment65813>
QDate(1900,1,1) is more light weight here
- Thomas Baumgart
On Juli 18, 2016, 7:44 nachm., Jeff Lundblad wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/128478/
> -----------------------------------------------------------
>
> (Updated Juli 18, 2016, 7:44 nachm.)
>
>
> Review request for KMymoney.
>
>
> Bugs: 365818
> http://bugs.kde.org/show_bug.cgi?id=365818
>
>
> Repository: kmymoney
>
>
> Description
> -------
>
> Only import transactions that are newer than or equal to the "Start date of import" in the account's online settings
>
>
> Diffs
> -----
>
> "a/kmymoney/kmymoney/converter/mymoneystatementreader-1b81833-left.cpp" 0b31377
>
> Diff: https://git.reviewboard.kde.org/r/128478/diff/
>
>
> Testing
> -------
>
> Tested on Citi credit card OFX downloads which nearly always download 2 years worth of transactions regardless of the date range that the OFX request requests.
>
>
> Thanks,
>
> Jeff Lundblad
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kmymoney-devel/attachments/20160720/4b246d6a/attachment.html>
More information about the KMyMoney-devel
mailing list