<div dir="ltr"><div class="gmail_default" style="font-family:monospace">Awesome. Thanks for that.<br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sat, Jul 29, 2023 at 10:07 AM Thomas Baumgart via KMyMoney-devel <<a href="mailto:kmymoney-devel@kde.org">kmymoney-devel@kde.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Montag, 24. Juli 2023 08:07:26 CEST Thomas Baumgart via KMyMoney-devel wrote:<br>
<br>
> On Montag, 24. Juli 2023 01:08:03 CEST Jeremy Whiting wrote:<br>
> <br>
> > I think the problem is that when you click Yes on that dialog to add the<br>
> > payee it shouldn't reset the ledgerview at all. It doesn't when you click<br>
> > No, it just clears the payee box. I think that makes sense, but when you<br>
> > click Yes it shouldn't be doing anything to the ledger. Only adding the<br>
> > payee as a payee behind the scenes and leaving the ledger completely alone.<br>
> > So I can continue to add category, cleared status, etc. Thanks for the<br>
> > pointer though, I'll see why it's calling LedgerView::reset in this case<br>
> > when it shouldn't be IMO.<br>
> <br>
> The reason is simple to explain: adding the payee to the MyMoneyFile object<br>
> creates a MyMoneyFileTransaction. When that is created, the LedgerConcatenate<br>
> model is instructed to stop emitting signals to prevent massive sorting<br>
> operations when ledger objects change. When the MyMoneyFileTransaction<br>
> object is destroyed, signal emission is turned on again and the<br>
> LedgerConcatenate model is reset which is propagated to the LedgerView.<br>
<br>
Fixed with <a href="https://invent.kde.org/office/kmymoney/-/commit/8f98eb55d84a388305da003881d6be8bcef6089e" rel="noreferrer" target="_blank">https://invent.kde.org/office/kmymoney/-/commit/8f98eb55d84a388305da003881d6be8bcef6089e</a><br>
<br>
-- <br>
<br>
Regards<br>
<br>
Thomas Baumgart<br>
<br>
-------------------------------------------------------------<br>
When PRIVACY Is Outlawed...<br>
Only OUTLAWS Will Have Privacy!<br>
-------------------------------------------------------------<br>
</blockquote></div>