I'm having odd behavior when using CSV import resulting in a kmymoney crash. User is able to close the CSV Wizard window by pressing escape key without closing CSV Import window. As a result, when finally closing CSV Import window, application is crashing. Moreover a user is able to partially close CSV Wizard dialog (see attached bug_csv_wizard_start_after_pressing_escape.png) Unfortunately I'm on Archlinux so unable to attach proper backtrace, sorry! Let me know if you have trouble reproducing it I will try to compile with debug symbols. Reproducible: Always Steps to Reproduce: 1. Start CSV Import plugin 2. On main window (see attached bug_csv_wizard_start.png) press "escape" key 3. Part of window is lost (this probably is also a bug) - see attached bug_csv_wizard_start_after_pressing_escape.png 5. Pressing escape again, will not do anything to window. 5. Now switch focus to any other window, and go back to CSV Wizard window and press "escape" key again 6. CSV Wizard window is gone, switch to CSV Import window and close it 7. Crash Actual Results: Kmymoney has crashed. Expected Results: CSV Wizard windows should ignore escape key and shouldn't allow user to "partialy" close widget. When closing CSV Wizard window, CSV Import windows should also close. On the other hand, CSV Import windows probably shouldn't be even visible on the start "page" of wizard - it is useless and contains only acres of white empty space. Vanilla Archlinux packages for KDE4 and Kmymoney
Created attachment 93166 [details] Start page of CSV Wizard which is reacting on escape key.
Created attachment 93167 [details] Partially closed CSV Wizard window after pressing escape
I can reproduce this bug, so confirming.
Git commit c0073108f83af30218a05a568c74c5efac20728e by Łukasz Wojniłowicz. Committed on 10/09/2016 at 16:18. Pushed by wojnilowicz into branch 'master'. Handle CSV Importer exits the right way 1) restore eventFilter only this time it's working right, 2) after pressing cancel button, CSV Wizard should close without any save, 3) CSV Wizard window should be modal and closed before main window. REVIEW:128876 FIXED-IN:5.0 M +23 -9 kmymoney/plugins/csvimport/csvwizard.cpp M +9 -8 kmymoney/plugins/csvimport/csvwizard.h M +7 -1 kmymoney/plugins/csvimport/csvwizard.ui http://commits.kde.org/kmymoney/c0073108f83af30218a05a568c74c5efac20728e