[Kmymoney-devel] csvimport crash on multiple import

[Allan]

On 24/04/14 02:18, Chris wrote:
>
> Ok. You got me!
>
> Will need to remove the "select file" button from the intermediate pages.
>

There weren't any intermediate pages, but it is now sorted.  On 
completion of each import, control returns to the into page, allowing 
access to the Skip Setup button, if needed, and Next File and Finish 
buttons.

> When stepping through the settings, the last pages buttons are a bit
> confusing you are presented with a finish button and import csv button.
> I think you should have one or the other not both. A couple of times I
> have automatically clicked finish expecting it to import the selected file.

The reason I have them both is that if, after doing an import, the user 
might realise/get told that a parameter is incorrect.  So, he can 
backtrack a bit, correct the error, then import again without having to 
exit or backtrack right to the intro page.  Or, he/she might just have 
imported into the wrong account and wants to try again.  Alternatively, 
he may Finish, which saves his settings, or cancel, which doesn't.

Cristian made a change that I don't like, in that now, the main KMM app 
is disabled until the plugin is closed.  I prefer the original, which 
leaves the plugin still available, while allowing the user to check the 
import result/whatever, without having to close and reopen the plugin 
and navigate through again.

> I also have noted clicking on the cancel button brings up a dialogue
> that looks like it wants to shutdown kmymoney rather than just close the
> wizard. I don't think you need a warning to close the wizard and it is
> confusing. No data is lost when closing it, so pressing the cancel
> button should simply close the wizard.

I did already think about clarifying the message, but as you say, it can 
go without any problem.

> Chris

Allan