Bug 236410 - Crash in Ledger View when changing payee
Summary: Crash in Ledger View when changing payee
Status: RESOLVED FIXED
Alias: None
Product: kmymoney
Classification: Applications
Component: general (show other bugs)
Version: unspecified
Platform: Compiled Sources Linux
: NOR crash
Target Milestone: ---
Assignee: KMyMoney Devel Mailing List
URL:
Keywords: usability
Depends on:
Blocks:
 
Reported: 2010-05-05 11:15 UTC by allan
Modified: 2010-05-06 15:01 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:


Attachments
anon KMM file (26.77 KB, application/xml)
2010-05-06 01:03 UTC, allan
Details
import qif file (60 bytes, application/x-qw)
2010-05-06 01:05 UTC, allan
Details

Note You need to log in before you can comment on or make changes to this bug.
Description allan 2010-05-05 11:15:53 UTC
Version:           3.97.2 svn 1121075 (using KDE 4.3.5)
OS:                Linux
Installed from:    Compiled From Sources

Attempting to modify the payee of an imported transaction to something more meaningful.

From within the transaction, I copied the account name, cleared the payee and entered the word 'from '. When I right clicked to paste, I was asked if I wanted to add 'from ' as a new payee.  I clicked 'no' and the payee box cleared.  So I started it again, but instead of right click, I did ctl-v.  I was now asked if I wanted to add 'from payee' as a new payee, I clicked yes and KMM crashed.

The BT contained only
Application: KMyMoney (kmymoney), signal: Segmentation fault
[KCrash Handler]
#6  0x00000000 in ?? ()
#7  0x00000000 in ?? ()
Comment 1 Cristian Oneț 2010-05-05 20:21:46 UTC
This seems to work fine here. Preparing an anon file where this can be used + instructions would be useful. Thanks.
Comment 2 allan 2010-05-06 01:03:39 UTC
Created attachment 43283 [details]
anon KMM file
Comment 3 allan 2010-05-06 01:05:14 UTC
Created attachment 43284 [details]
import qif file
Comment 4 allan 2010-05-06 01:17:07 UTC
(In reply to comment #1)
> This seems to work fine here. Preparing an anon file where this can be used +
> instructions would be useful. Thanks.

It didn't 100% fail, but these steps seem to do the trick.

Load anon file attached.
Import qif file into account A000001
Select account A000001
Edit the single transaction
Enter A000006 as the account
Copy the A000006 just entered
Clear the payee
Start to enter 'from' into payee
You'll be asked if 'f' should be a new payee
Click NO
Enter 'from ' then right click to paste
You'll be asked if 'from ' should be a new payee
Click NO
Enter 'from ' then type ctrl+v to paste
Click Yes to accept payee
KMM crashes

Useless BT
Comment 5 Alvaro Soliverez 2010-05-06 05:23:21 UTC
The problem seems to be when selecting a payee and then pressing any letter. The popup glimpses for a second (which calls a focusOutEvent), then vanishes.
Weirdly, it doesn't happen if selecting a payee and pasting the value, only when there is a key press involved.

The reason of the QFocusEvent is too general to be of any use in this case.
Comment 6 Alvaro Soliverez 2010-05-06 05:26:20 UTC
BTW, I think this particular bug is solved and what we are seeing is the behaviour reported in BUG:235618
Comment 7 Cristian Oneț 2010-05-06 08:36:07 UTC
(In reply to comment #6)
> BTW, I think this particular bug is solved and what we are seeing is the
> behaviour reported in BUG:235618

So the crash is solved by this commit?

SVN commit 1123261 by conet:

Allow using the context menu inside the widget.

 M  +6 -0      kmymoneymvccombo.cpp
Comment 8 allan 2010-05-06 14:51:14 UTC
(In reply to comment #7)
> (In reply to comment #6)
> > BTW, I think this particular bug is solved and what we are seeing is the
> > behaviour reported in BUG:235618
> 
> So the crash is solved by this commit?
> 
> SVN commit 1123261 by conet:
> 
> Allow using the context menu inside the widget.
> 
>  M  +6 -0      kmymoneymvccombo.cpp

Now on svn1123617 and all seems well.  Repeated the above steps a few times with no crashes.  Many thanks.
Comment 9 Cristian Oneț 2010-05-06 15:01:39 UTC
(In reply to comment #8)
> (In reply to comment #7)
> > (In reply to comment #6)
> > > BTW, I think this particular bug is solved and what we are seeing is the
> > > behaviour reported in BUG:235618
> > 
> > So the crash is solved by this commit?
> > 
> > SVN commit 1123261 by conet:
> > 
> > Allow using the context menu inside the widget.
> > 
> >  M  +6 -0      kmymoneymvccombo.cpp
> 
> Now on svn1123617 and all seems well.  Repeated the above steps a few times
> with no crashes.  Many thanks.

Hey, thanks a lot for testing :) keep up the good work. I'll close this one then.