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

--- Comment #6 from NSLW <lukasz.wojnilow...@gmail.com> ---
(In reply to Jack from comment #5)
> I can't comment on the code itself, but in the initial bug report, you talk
> about csv import, but it looks to me like your proposed changes will apply
> to any import.  How are you checking that the import is for an investment
> account?  If there is actually something in the import, this is great.  If
> it is only because you have to specify that it is an investment import for
> the csv importer, then it should be restricted to that situation.  If the
> proposed changes actually take care of that, you can ignore my comment.

Yes, you're right. It will apply to any import because it was easy to extend
this feature on all imports. 
Type of import is today determined right before "Account selection" so no
problem to identify which kind of import we make. If you want to analyze code
yourself, then look into mymoneystatementreader.cpp, function "import", place
right before "selectOrCreateAccount".

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to