Bug 249438 - In investment ledger, transfer transaction popup shows all accounts grayed out, but transfer works
Summary: In investment ledger, transfer transaction popup shows all accounts grayed ou...
Status: RESOLVED FIXED
Alias: None
Product: kmymoney
Classification: Applications
Component: general (show other bugs)
Version: unspecified
Platform: Compiled Sources Linux
: NOR normal
Target Milestone: ---
Assignee: KMyMoney Devel Mailing List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-08-29 19:22 UTC by Jack
Modified: 2011-08-15 13:27 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments
popup for transaction move from my brokerage account (22.48 KB, image/png)
2010-08-31 01:08 UTC, Jack
Details
popup for transaction move from my investment account (19.57 KB, image/png)
2010-08-31 01:09 UTC, Jack
Details
popup for transaction move from sample brokerage account (15.35 KB, image/png)
2010-08-31 01:10 UTC, Jack
Details
popup for transaction move from sample investment account (10.26 KB, image/png)
2010-08-31 01:10 UTC, Jack
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Jack 2010-08-29 19:22:31 UTC
Version:           unspecified (using KDE 4.4.5) 
OS:                Linux

Note: this is a minor display issue only.  The actual behavior is correct.

With an investment account shown in the ledger, selecting a transaction, right clicking ans selecting "move transaction to" pops up a list of all my transaction accounts.  However, they are all very faint - as if they were disabled.  However, clicking on one successfully moves the transaction.

Reproducible: Didn't try

Steps to Reproduce:
select a transaction in the ledger in an investment account.  move the transaction to another account

Actual Results:  
the list of "to" accounts appears all grayed out, as if they were disabled.

Expected Results:  
the list of "to" accounts would appear more bold (normal)

I'd call it minor - but I have no idea how easy it will be to find and fix.  I assume it is in the code that displays the list of "to" accounts.
Comment 1 Jack 2010-08-29 19:27:58 UTC
KMyMoney 4.5.0
Comment 2 Alvaro Soliverez 2010-08-30 02:26:33 UTC
I see them correctly. May this have something to do with your theme?
Comment 3 Jack 2010-08-31 01:08:52 UTC
Created attachment 51125 [details]
popup for transaction move from my brokerage account
Comment 4 Jack 2010-08-31 01:09:43 UTC
Created attachment 51126 [details]
popup for transaction move from my investment account
Comment 5 Jack 2010-08-31 01:10:17 UTC
Created attachment 51127 [details]
popup for transaction move from sample brokerage account
Comment 6 Jack 2010-08-31 01:10:56 UTC
Created attachment 51128 [details]
popup for transaction move from sample investment account
Comment 7 Jack 2010-08-31 01:13:51 UTC
I just attached four images - the transaction move popup for a brokerage account and an investment account for my own kmy file and for the sample kmy file I used for the manual.  As far as I can tell, all four should be with the same theme, but my investment popup looks different.  I also tried changing themes, and didn't notice any difference - perhaps I'll try some other themes.

Also - in the sample investment popup - why does a brokerage account appear on the list?  That doesn't make any sense, does it?
Comment 8 Alvaro Soliverez 2010-08-31 01:23:20 UTC
In the case of your investment account, it seems to have the KMM theming applied, but I can't tell why. Does it happen every time on that specific account? Do you have another investment account or can you create a test account and check whether that happens too?
Regarding the brokerage account, I don't know the specifics regarding the rules to include it or not.
Comment 9 Jack 2010-08-31 01:59:18 UTC
I have three investment accounts, and they all look the same.  I also just created a new one, and it also looks the same.  I guess my question is why the investment accounts look different from the other asset/liability accounts.
Comment 10 Alvaro Soliverez 2010-08-31 04:54:16 UTC
I tried with several investment accounts and in all cases I see the account selector in normal colors.

How about your color settings in KMyMoney? Did you select to use system colors?
Comment 11 Jack 2010-08-31 18:16:35 UTC
I've tried with complete default settings for color and with system colors (which claims just to be for background) and no change.

One question I have is why is the popup different for investment account and brokerage account.  If both showed gray text, I would think of color settings, but I can't figure out why they would be different.
Comment 12 Jack 2011-08-13 19:25:24 UTC
Just to update that this behavior is still present with latest svn.  I'm still curious why the popup is different for investment vs. any other account type.
Comment 13 Cristian Oneț 2011-08-13 22:36:19 UTC
I can't reproduce this with my investment accounts. Try to create an anonymous file from you data and send it to me directly if you would not attach it here.
Comment 14 Cristian Oneț 2011-08-15 11:07:25 UTC
SVN commit 1247346 by conet:

BUG: 249438
Instead of disabling a parent item which does not matches the filter but has children that do match the filter mark it as not selectable.
If we would disable the item then all it's children will be drawn as disabled even if they would be selectable. Disabling and selectability are two aspects and it seems that disabling applies to a whole subtree.

 M  +4 -4      kmymoneyaccountselector.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=1247346
Comment 15 Cristian Oneț 2011-08-15 11:08:29 UTC
SVN commit 1247347 by conet:

BUG: 249438
Instead of disabling a parent item which does not matches the filter but has children that do match the filter mark it as not selectable.
If we would disable the item then all it's children will be drawn as disabled even if they would be selectable. Disabling and selectability are two aspects and it seems that disabling applies to a whole subtree.
Backported to the stable branch.

 M  +4 -4      kmymoneyaccountselector.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=1247347
Comment 16 Jack 2011-08-15 13:27:20 UTC
Thanks, Cristian.  I confirm this fixes the problem for me.