Summary: | Crash while editing a split | ||
---|---|---|---|
Product: | [Applications] kmymoney | Reporter: | allan <agander93> |
Component: | general | Assignee: | KMyMoney Devel Mailing List <kmymoney-devel> |
Status: | RESOLVED FIXED | ||
Severity: | crash | CC: | onet.cristian |
Priority: | NOR | Keywords: | drkonqi |
Version: | unspecified | ||
Target Milestone: | --- | ||
Platform: | Compiled Sources | ||
OS: | Linux | ||
Latest Commit: | http://commits.kde.org/kmymoney/c6639a214066388a6c718b5a9a42db2e7cf9fd5b | Version Fixed In: |
Description
allan
2014-06-15 12:23:42 UTC
Difficulty while editing - Because I cannot edit the existing splits, I try to enter a new one. When I click Edit to open a new one, I get "You need to assign a category to this split before it can be entered." , twice. When I enter the first character, this same message again appears twice. I select the category from the drop-down, but when I try to enter into the memo field, the category has been cleared and again two messages. When I try to enter the amount, same again, so use the calculator. Ho hum... > this split before it can be entered." , twice. When I enter the
> first character, this same message again appears twice.
Got the same problem about a year ago. Recompiling from a clean source
tree did help in my case. I somehow messed up my repo.
Hope this helps. Greetz.
Martin
Thanks Martin, but sadly, no here. I had even created a new sandbox and clone, but no help. Now I've done a make clean on that and still have the problem.
Manage to reproduce this using a lot of random double clicks in the split table, after obtaining two transaction editors, close each of them with 'Enter' and the crash occurs. Git commit 8e7898fbd7de4cc7557d5de9eaae32386d2859b1 by Cristian Oneț. Committed on 31/08/2014 at 15:49. Pushed by conet into branch 'master'. Fix a nasty split editor bug. Because of the way the endEdit method was written the endEdit operation could be executed without actually ending the edit (missing category). But code that was calling endEdit expected that it allways would end the edit thus sometimes edit widgets could be obtained on two rows. Once in this state accepting both editors would cause a crash. Also removed a workaround which would end the edit on a resize. Instead don't do the operation that was resizing the rows while in edit. M +24 -21 kmymoney/dialogs/kmymoneysplittable.cpp M +9 -1 kmymoney/dialogs/kmymoneysplittable.h http://commits.kde.org/kmymoney/8e7898fbd7de4cc7557d5de9eaae32386d2859b1 Git commit f3d666d93dbcb2f0b51a344ab2d78f7c68c92c67 by Cristian Oneț. Committed on 31/08/2014 at 15:49. Pushed by conet into branch 'frameworks'. Fix a nasty split editor bug. Because of the way the endEdit method was written the endEdit operation could be executed without actually ending the edit (missing category). But code that was calling endEdit expected that it allways would end the edit thus sometimes edit widgets could be obtained on two rows. Once in this state accepting both editors would cause a crash. Also removed a workaround which would end the edit on a resize. Instead don't do the operation that was resizing the rows while in edit. (cherry picked from commit 8e7898fbd7de4cc7557d5de9eaae32386d2859b1) Conflicts: kmymoney/dialogs/kmymoneysplittable.cpp M +24 -21 kmymoney/dialogs/kmymoneysplittable.cpp M +9 -1 kmymoney/dialogs/kmymoneysplittable.h http://commits.kde.org/kmymoney/f3d666d93dbcb2f0b51a344ab2d78f7c68c92c67 Git commit c6639a214066388a6c718b5a9a42db2e7cf9fd5b by Christian Dávid, on behalf of Cristian Oneț. Committed on 31/08/2014 at 15:49. Pushed by christiand into branch 'add-onlinebanking'. Fix a nasty split editor bug. Because of the way the endEdit method was written the endEdit operation could be executed without actually ending the edit (missing category). But code that was calling endEdit expected that it allways would end the edit thus sometimes edit widgets could be obtained on two rows. Once in this state accepting both editors would cause a crash. Also removed a workaround which would end the edit on a resize. Instead don't do the operation that was resizing the rows while in edit. M +24 -21 kmymoney/dialogs/kmymoneysplittable.cpp M +9 -1 kmymoney/dialogs/kmymoneysplittable.h http://commits.kde.org/kmymoney/c6639a214066388a6c718b5a9a42db2e7cf9fd5b |