Review Request 127251: [KUnitConversion] Fix downloading currency exchange rates
Ben Cooksley
bcooksley at kde.org
Wed Mar 2 09:00:08 UTC 2016
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/127251/#review93058
-----------------------------------------------------------
As a warning (because you'll be using QNAM here) please ensure you handle redirects properly and completely. Any software which fails to do so is considered unsupported even before it is released in regards to accessing KDE.org infrastructure. Workarounds will not be implemented on our systems in response to incorrect usage of QNAM.
- Ben Cooksley
On March 2, 2016, 12:48 a.m., Kai Uwe Broulik wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/127251/
> -----------------------------------------------------------
>
> (Updated March 2, 2016, 12:48 a.m.)
>
>
> Review request for KDE Frameworks and Vishesh Handa.
>
>
> Bugs: 345750
> https://bugs.kde.org/show_bug.cgi?id=345750
>
>
> Repository: kunitconversion
>
>
> Description
> -------
>
> QNetworkReply does not implement waitForReadyRead
>
> Also, the code never actually created the cache dir it was trying to create a file in.
>
>
> Diffs
> -----
>
> src/currency.cpp 3b99644
>
> Diff: https://git.reviewboard.kde.org/r/127251/diff/
>
>
> Testing
> -------
>
> Works now. It's downloaded once and then taken from cache file in ~/.local/share/libkunitconversion/currency.xml
>
> Given it's a Tier 2 framework doesn't make sense to add KIO now, also failed to reproduce the crashes mentioned in the code.
>
> Tests pass (only if I run them on English locale btw)
>
> Obviously not happy with this being sync but alas that's how the API works.
>
> Not sure if this is a KRunner bug or KUnitConverison but if I enter "5 USD" it converts fine, if I enter "5 usd" it returns zero for all the currencies it converted to.
>
>
> Thanks,
>
> Kai Uwe Broulik
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20160302/c87c85e5/attachment.html>
More information about the Kde-frameworks-devel
mailing list