Bug 236410

Summary:	Crash in Ledger View when changing payee
Product:	[Applications] kmymoney	Reporter:	allan <agander93>
Component:	general	Assignee:	KMyMoney Devel Mailing List <kmymoney-devel>
Status:	RESOLVED FIXED
Severity:	crash	Keywords:	usability
Priority:	NOR
Version:	unspecified
Target Milestone:	---
Platform:	Compiled Sources
OS:	Linux
Latest Commit:		Version Fixed In:
Attachments:	anon KMM file import qif file

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.