> On Sept. 21, 2013, 10:29 a.m., Cristian Oneț wrote: > > I would wait for Thomas's opinion about this. Strictly from a programmer > > POV the patch looks fine, I'm not certain about the functionality, I don't > > know why the equity accounts were not available in the ledger until now.
Equity accounts in KMM are hidden because it's advanced functionality and we try to hide the double-entry accounting as much as possible. Keep in mind that the equity accounts are not assets, or shares in a company, rather the net worth from opening balances in accounts. They are hidden in normal mode, and should be hidden. - Alvaro ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/112847/#review40398 ----------------------------------------------------------- On Sept. 21, 2013, 12:41 p.m., Ralf Habacker wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/112847/ > ----------------------------------------------------------- > > (Updated Sept. 21, 2013, 12:41 p.m.) > > > Review request for KMymoney and Cristian Oneț. > > > Description > ------- > > When enabled display of equity accounts with 'settings->general->filter->show > equity accounts' equity accounts are not displayed in the "Default Account" > tab of payee page and in the unknown payee import dialog. > > The related bug could be triggered with: > 1. create a new file > 2. select the german skr03 standard chart of accounts. > > > Diffs > ----- > > kmymoney/converter/mymoneystatementreader.cpp 5619cc2 > kmymoney/views/kgloballedgerview.cpp 2057b47 > kmymoney/views/kpayeesview.cpp 69697e6 > > Diff: http://git.reviewboard.kde.org/r/112847/diff/ > > > Testing > ------- > > git master > > > Thanks, > > Ralf Habacker > >
_______________________________________________ KMyMoney-devel mailing list KMyMoney-devel@kde.org https://mail.kde.org/mailman/listinfo/kmymoney-devel
