On Montag, 24. Juli 2023 08:07:26 CEST Thomas Baumgart via KMyMoney-devel wrote:
> On Montag, 24. Juli 2023 01:08:03 CEST Jeremy Whiting wrote: > > > I think the problem is that when you click Yes on that dialog to add the > > payee it shouldn't reset the ledgerview at all. It doesn't when you click > > No, it just clears the payee box. I think that makes sense, but when you > > click Yes it shouldn't be doing anything to the ledger. Only adding the > > payee as a payee behind the scenes and leaving the ledger completely alone. > > So I can continue to add category, cleared status, etc. Thanks for the > > pointer though, I'll see why it's calling LedgerView::reset in this case > > when it shouldn't be IMO. > > The reason is simple to explain: adding the payee to the MyMoneyFile object > creates a MyMoneyFileTransaction. When that is created, the LedgerConcatenate > model is instructed to stop emitting signals to prevent massive sorting > operations when ledger objects change. When the MyMoneyFileTransaction > object is destroyed, signal emission is turned on again and the > LedgerConcatenate model is reset which is propagated to the LedgerView. Fixed with https://invent.kde.org/office/kmymoney/-/commit/8f98eb55d84a388305da003881d6be8bcef6089e -- Regards Thomas Baumgart ------------------------------------------------------------- When PRIVACY Is Outlawed... Only OUTLAWS Will Have Privacy! -------------------------------------------------------------
signature.asc
Description: This is a digitally signed message part.