Bug 267029

Summary: CSV Plugin produces confusing message under certain circumstances.
Product: [Applications] kmymoney Reporter: allan <agander93>
Component: generalAssignee: KMyMoney Devel Mailing List <kmymoney-devel>
Status: RESOLVED FIXED    
Severity: normal    
Priority: NOR    
Version: SVN   
Target Milestone: ---   
Platform: openSUSE   
OS: Linux   
Latest Commit: Version Fixed In:

Description allan 2011-02-24 01:53:13 UTC
Version:           SVN trunk (using KDE 4.6.0) 
OS:                Linux

When a user is selecting a column to import, a check is made that that column has not already been selected for a different field, and a dialog is opened to draw attention to the error.  Under certain circumstances, this message is produced even if the user is not entering a column selection, which is quite puzzling.


Reproducible: Sometimes

Steps to Reproduce:
1) Open the plugin, loading the UI with previous parameters from the resource file.
2) Load a new file and start to enter column selections.
3) Switch tabs, say from banking to investment, accidentally or otherwise.
4) Switch back.
5) The resource file is reloaded, but as the UI is updated with these parameters, the check on the reloaded parameters may find they conflict with those entered in step 2, producing the error dialog.  This causes confusion because the user wasn't entering data.

Actual Results:  
Confusing message produced.

Expected Results:  
An avoidance of the situation.

Possibly need to update the resource file as each new column selection is entered, instead of just on saving the file or closing down.
Comment 1 allan 2011-02-26 01:55:42 UTC
(In reply to comment #0)
> Version:           SVN trunk (using KDE 4.6.0) 
> OS:                Linux
> 
> When a user is selecting a column to import, a check is made that that column
> has not already been selected for a different field, and a dialog is opened to
> draw attention to the error.  Under certain circumstances, this message is
> produced even if the user is not entering a column selection, which is quite
> puzzling.
> 
> 
> Reproducible: Sometimes
> 
> Steps to Reproduce:
> 1) Open the plugin, loading the UI with previous parameters from the resource
> file.
> 2) Load a new file and start to enter column selections.
> 3) Switch tabs, say from banking to investment, accidentally or otherwise.
> 4) Switch back.
> 5) The resource file is reloaded, but as the UI is updated with these
> parameters, the check on the reloaded parameters may find they conflict with
> those entered in step 2, producing the error dialog.  This causes confusion
> because the user wasn't entering data.
> 
> Actual Results:  
> Confusing message produced.
> 
> Expected Results:  
> An avoidance of the situation.
> 
> Possibly need to update the resource file as each new column selection is
> entered, instead of just on saving the file or closing down.

Rather than this solution, I now instead save settings on changing tab, as well as previously done on saving or closing.

Committed SVN revision 1222786.
Comment 2 Cristian OneČ› 2011-05-12 12:30:50 UTC
Allan, when committing a bugfix p'ease use the "BUG: id" marker in your commit message so that the bug you fix gets automatically marked as closed. See http://techbase.kde.org/Policies/SVN_Commit_Policy#Special_keywords_in_SVN_log_messages for more details. Thanks :).