[Kmymoney-devel] [Bug 269074] csv importer tests crash

allan agander93 at gmail.com
Tue Mar 22 02:09:51 CET 2011


https://bugs.kde.org/show_bug.cgi?id=269074





--- Comment #4 from allan <agander93 gmail com>  2011-03-22 02:09:50 ---
I use KGlobal::locale()->decimalSymbol() in several places in the plugin,
partly why I implemented those unittests.  You may remember a week or two ago,
I queried the support for no decimal places, and you responded that the
qifimporter didn't support that?  Well, because the plugin can be used to
produce qif files, I now test for absence of decimals, and if none found, add
them with the locale decimal separator.

To help in closing in on the issue, am I right in thinking that you have run
only the tests, and have not 'used' the plugin itself, so this is the only
point where you encounter the problem?

I've made an assumption that the local decimal separator is pretty fundamental
and is bound to be present, but it seems that perhaps that is not a good idea.

As I can't reproduce the problem here, might it help to test whether having a
KGlobal::locale()->decimalSymbol().isEmpty() test?  Can you do that, or should
I implement it anyway and produce a patch, which I have no means of testing?

-- 
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.


More information about the KMyMoney-devel mailing list