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

[Alvaro Soliverez]

On Thu, Feb 18, 2010 at 11:34 AM, Alvaro Soliverez <asoliverez at gmail.com> wrote:
> 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.
>
Jack,
can you try the latest svn? I added a condition to check for that case.

Regards,
Alvaro