[kmymoney] [Bug 401831] Unable to delete an insitution

2019-01-19 Thread Thomas Baumgart
https://bugs.kde.org/show_bug.cgi?id=401831

Thomas Baumgart  changed:

   What|Removed |Added

 Resolution|--- |WORKSFORME
 Status|CONFIRMED   |RESOLVED

-- 
You are receiving this mail because:
You are watching all bug changes.

[kmymoney] [Bug 401831] Unable to delete an insitution

2018-12-07 Thread nigel
https://bugs.kde.org/show_bug.cgi?id=401831

--- Comment #3 from nigel  ---
Thanks for a speedy response.
Workaround is acceptable

-- 
You are receiving this mail because:
You are watching all bug changes.

[kmymoney] [Bug 401831] Unable to delete an insitution

2018-12-06 Thread Thomas Baumgart
https://bugs.kde.org/show_bug.cgi?id=401831

--- Comment #2 from Thomas Baumgart  ---
The source code contains the following comment:

---8<--- accountsproxymodel.cpp:230
If this is an institution that has no children show it only if hide unused
institutions (hide closed accounts for now) is not checked.
---8<---

So the software works as designed. I suggest to go with option 2) as suggested
by Jack for now (5.0.3) and add 1) to a future version (5.1+). Opinions?

-- 
You are receiving this mail because:
You are watching all bug changes.

[kmymoney] [Bug 401831] Unable to delete an insitution

2018-12-06 Thread Jack
https://bugs.kde.org/show_bug.cgi?id=401831

Jack  changed:

   What|Removed |Added

 Status|REPORTED|CONFIRMED
 Ever confirmed|0   |1

--- Comment #1 from Jack  ---
I think I see the problem.  However, not only do institutions with no assigned
accounts show up, neither to institutions where all assigned accounts are
closed.

If you go to Settings/Configure KMyMoney and uncheck "Do not show closed
accounts" then these institutions DO show up in the Institutions View.  At
least that is a workaround that will let you delete those Institutions.

While I can see some logic in including display of institutions in the logic
for displaying accounts, it is certainly not obvious.  I can think of three
options (in the order of my preference):
1) include a note on that view (if display of closed accounts is disabled)
about changing that setting to view institutions with no accounts or with only
closed accounts
2) always display all institutions in the institutions view
3) add a separate "do not show institutions with no assigned or only closed
accounts" setting

The second might be best and simplest to implement, if I'm right that the
institutions view is the only place where this matters.  However, it also means
that the closed accounts would also need to be shown there.

-- 
You are receiving this mail because:
You are watching all bug changes.