[kmymoney4] [Bug 360435] CSV Importer doesn't recognize security if its symbol isn't lower case

aga agander93 at gmail.com
Wed Mar 30 17:22:26 UTC 2016


On 28/03/16 13:13, allan via KDE Bugzilla wrote:
> https://bugs.kde.org/show_bug.cgi?id=360435
>
> --- Comment #13 from allan <agander93 at gmail.com> ---
> (In reply to NSLW from comment #12)
>> (In reply to allan from comment #11)
>>> Are you sure about the change to csvwizard.cpp?  As far as I can see, with
>>> "exists = false;" in the while loop, it works correctly.
>>
>> As long as list variable is not empty. If it's empty you wont even enter
>> while loop (thus wont even define exists variable) and it is empty if you
>> have no securities on "securities tab". It's corner case, I'm sure of.
>
> Yes, of course.  You are right.
>
> I think we're all happy with this now, so I'll be going ahead to commit.

Unfortunately, the commit I made (29/03/16 18:29) was incomplete.  I'm 
afraid I'm suffering somewhat at the moment, and wouldn't want to tackle 
the revert and make things worse.  There was no problem with the patch 
supplied by lukasz.wojnilowicz at gmail.com.

Could I ask please for assistance with this, and apologize to Christian 
for messing up his frameworks commit.

Allan


More information about the KMyMoney-devel mailing list