oddity with closed account

[Brendan Coupe]

Works for me. Both options are disabled when they should be.


----
Brendan Coupe

On Fri, Aug 3, 2018 at 11:10 AM, Thomas Baumgart <thb at 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 <thb at 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.
> -------------------------------------------------------------
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kmymoney-devel/attachments/20180803/277a718f/attachment.html>