Bug 342926 - "Next" button may get disabled on Banking page
Summary: "Next" button may get disabled on Banking page
Status: RESOLVED FIXED
Alias: None
Product: kmymoney
Classification: Applications
Component: importer (show other bugs)
Version: git (master)
Platform: Mint (Ubuntu based) Linux
: NOR normal
Target Milestone: ---
Assignee: KMyMoney Devel Mailing List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-01-16 18:25 UTC by allan
Modified: 2015-01-18 11:57 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In: 4.7.2


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description allan 2015-01-16 18:25:03 UTC
On reaching the Banking page, if the category combo shows that a column has been selected, and if the user decides to use that column as a memo instead, then, as intended, the user is warned of this clash.  If he/she accepts the change, then, again as expected, the category and memo combos get reset.

However, this also disables the "Next" button, even though the required columns still show as selected.  It is necessary to reselect one of the required fields in order to re-enable the "Next" button.

Patch to follow.

Reproducible: Always

Steps to Reproduce:
1.Import a CSV file and select a column to use as the category.
2.Change your mind and instead select that field to be a memo.
3.A message appears "The 'category' field already has this column selected. 
   Please reselect both entries as necessary."
4. Click OK to the message.
5. The clash results in both combos being reset, and also the "Next" button.
6. Now select that field to be a memo.
7. The Next" button is still disabled.
8. To remedy this, reselect one of the required fields - Date, Payee or Amount.

Actual Results:  
As above.

Expected Results:  
The "Next" button should not get reset if all required fields are still selected.
Comment 1 allan 2015-01-17 12:39:20 UTC
Git commit fef9efb4ec87520ba976726b641b57b30d6f91a7 by Allan Anderson.
Committed on 17/01/2015 at 12:07.
Pushed by allananderson into branch 'master'.
- Fix incorrect disabling of Next button in CSV importer.

M  +0    -1    kmymoney/plugins/csvimport/csvdialog.cpp

http://commits.kde.org/kmymoney/fef9efb4ec87520ba976726b641b57b30d6f91a7
Comment 2 allan 2015-01-18 11:41:48 UTC
Git commit cf87967b565c35019ee5709dd7b9781e3fef288c by Allan Anderson.
Committed on 17/01/2015 at 12:07.
Pushed by allananderson into branch '4.7'.
- Fix incorrect disabling of Next button in CSV importer.

(cherry picked from commit fef9efb4ec87520ba976726b641b57b30d6f91a7)

M  +0    -1    kmymoney/plugins/csvimport/csvdialog.cpp

http://commits.kde.org/kmymoney/cf87967b565c35019ee5709dd7b9781e3fef288c