oddity with closed account

[Thomas Baumgart]

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/2a08b64b71f6290df7483d1213b37e436f004a58

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 --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 846 bytes
Desc: This is a digitally signed message part.
URL: <http://mail.kde.org/pipermail/kmymoney-devel/attachments/20180803/425c63e5/attachment.sig>