[Kmymoney-devel] Can't remove payee from transaction (3.95)

[Alvaro Soliverez]

On Thu, Feb 18, 2010 at 11:12 AM, Jack <ostroffjh at users.sourceforge.net> wrote:
> On 2010.02.18 01:00, Thomas Baumgart wrote:
>> on Wednesday 17 February 2010 Alvaro Soliverez wrote:
>> > On Wed, Feb 17, 2010 at 2:55 PM, Jack
>> <ostroffjh at users.sourceforge.net wrote:
>>>> For transfer transactions, I prefer not to specify a Payee.  I can
>>>> create such a transaction with no problem.  However, when I
>>>> download an OFX from my bank, a transfer shows up as a payment, and
>>>> I used to (in 1.0) edit the transacion, delete the payee, change
>>>> the transaction type to transfer, and add the other account.  In
>>>> the new version, I do exactly the same thing, but when I save the
>>>> transaction, the payee is still there.
>> > >
>>>> There is another minor difference between 1.0 and 3.95, although
>>>> this is from memory since I don't have a copy of the old version
>>>> installed. In 1.0, I had to explicitly hit ESC or click in another
>>>> field when leaving the Payee field blank, or else it would just
>>>> insert whatever payee was selected in the drop-down.  In 3.95, I
>>>> can just tab out of a blank payee field - which is fine for a new
>>>> transaction, but if I'm deleting a payee when editing a
>>>> transaction, it shows a blank payee field, but it doesn't actually
>>>> delete it, and the original payee shows up after I have saved the
>>>> transaction.
>> >
>> > Hello Jack,
>>> This is probably a problem with the new KMyMoneyPayeeCombo. I'll
>>> look into  it.
>>
>> Looks very similar to https://bugs.kde.org/show_bug.cgi?id=226261
>
> Thomas,
>
> This is not the same as that bug.  I am not trying to completely delete
> the payee.  I am just trying to remove the payee from the transaction.
> I can see this happening if the KMyMoneyPayeeCombo returns "no change"
> rather than "changed to blank" so I think Alvaro is probably on the
> right track.
>

There's a condition in the code where it specifically prevents you
from putting a blank on an existing payee. :D
I have to run some tests and then I'll commit the fix.