Works for me. Both options are disabled when they should be.
---- Brendan Coupe On Fri, Aug 3, 2018 at 11:10 AM, Thomas Baumgart <t...@net-bembel.de> wrote: > Hi, > > On Freitag, 3. August 2018 06:53:09 CEST Brendan Coupe wrote: > > > I managed to resurrect and old system with KMM 4.8 on it to check on the > > behavior of mapping and closed accounts. > > > > I agree with Thomas on how it should work. Here is how it was in 4.8: > > > > The option to close an account that is mapped is disabled/grayed out. > > > > The option to map an account that is closed is still active and appears > to > > work although I did not complete the process since the account in > question > > is .... closed:-) > > Fixed in https://commits.kde.org/kmymoney/2a08b64b71f6290df7483d1213b37e > 436f004a58 > > Regards > > Thomas > > > > On Thu, Aug 2, 2018 at 1:03 AM, Thomas Baumgart <t...@net-bembel.de> > wrote: > > > > > Hi, > > > > > > On Mittwoch, 1. August 2018 22:20:13 CEST Jack wrote: > > > > > > > Partly due to the recent changes regarding updating multiple > accounts, > > > > I've been reviewing all my mapped accounts. One is a long closed > > > > account. Most account actions cannot be performed on closed > accounts. > > > > You generally need to reopen the account, perform the action, then > > > > close the account again. It turns out it is possible to unmap a > closed > > > > account. It also seems possible to map a closed account (the menu > > > > entry is not disabled.) This seems odd to me - I would expect these > > > > actions to also require an open account. Is this intentional, or > just > > > > an oversight. I'll be glad to open a bug for it to keep track of it, > > > > but only if there is agreement that the current behavior is not how > it > > > > should be. > > > > > > My 0.02 on this: closing should not be possible if the account is still > > > mapped, mapping should not be possible on closed accounts. Could be, > that > > > it was that way, but I don't remember and don't have the time to find > out. > > > Good catch btw. > > -- > > Regards > > Thomas Baumgart > > https://www.signal.org/ Signal, the better WhatsApp > ------------------------------------------------------------- > Embedded Linux: because you can't do control-alt-delete on a pacemaker. > ------------------------------------------------------------- >
