<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="12" style="border: 1px #c9c399 solid; border-radius: 6px; -moz-border-radius: 6px; -webkit-border-radius: 6px;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="https://git.reviewboard.kde.org/r/112847/">https://git.reviewboard.kde.org/r/112847/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On September 21st, 2013, 2:24 p.m. UTC, <b>Alvaro Soliverez</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">I fail to see why you would want equity accounts displayed for imports. Can you explain the use case?
Keep in mind that equity accounts are not stock or share accounts of some kind.</pre>
</blockquote>
<p>On October 2nd, 2013, 5:36 a.m. UTC, <b>Ralf Habacker</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">To be able to import opening balances for example when using german template "all accounts", which have an account "Equity"->"Opening balances". Otherwise I would have to manual enter the opening balances for all related accounts by hand.</pre>
</blockquote>
<p>On February 14th, 2014, 1:27 p.m. UTC, <b>Alvaro Soliverez</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">These opening balances are exactly that, opening balances, and should not have a payee assigned.
That account is to balance the entry when you start using KMyMoney and you enter the opening balances for your asset and liability accounts. As per double-entry accounting, all transactions have to be balanced, and money can't just pop out of the blue.
eg. you start using KMM on Jan 1, but you already had a bank account by that date. You create the account and enter the balance on that date. To balance that transaction, a split is assigned to "Equity/Opening balances". No payees are involved.
Sorry for the delay, this went under my radar.
</pre>
</blockquote>
<p>On March 2nd, 2015, 5:30 a.m. UTC, <b>Cristian Oneț</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">I understand what Alvaro is saying but is there a way to import opening balances of accounts if one has many accounts? If there is not then I would suggest shipping this patch since the visibility of the equity accounts is controlled by the expert mode flag anyway. I really don't like limiting the functionality in valid use cases just for the sake of simplicity.</p></pre>
</blockquote>
<p>On March 2nd, 2015, 11:04 a.m. UTC, <b>Alvaro Soliverez</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">You can import opening balances, but you can't assign them a payee, because it doesn't make sense. I'm opposing to this because so far I fail to see the grounds to let that happen. Opening balances just don't have a payee.</p></pre>
</blockquote>
</blockquote>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">OK, I understand now, I mistakingly thought that Ralf's purpose is to import the opening balances and not to assign them to a payee. I also see no reason to assign opening balance transactions to payess, but it seems that this is what the patch does. Ralf, how does the above patch help you import opening balances?</p></pre>
<br />
<p>- Cristian</p>
<br />
<p>On September 21st, 2013, 12:41 p.m. UTC, Ralf Habacker wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="12" style="border: 1px #888a85 solid; border-radius: 6px; -moz-border-radius: 6px; -webkit-border-radius: 6px;">
<tr>
<td>
<div>Review request for KMymoney and Cristian Oneț.</div>
<div>By Ralf Habacker.</div>
<p style="color: grey;"><i>Updated Sept. 21, 2013, 12:41 p.m.</i></p>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt;">Repository: </b>
kmymoney
</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">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.
</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">git master</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>kmymoney/converter/mymoneystatementreader.cpp <span style="color: grey">(5619cc2)</span></li>
<li>kmymoney/views/kgloballedgerview.cpp <span style="color: grey">(2057b47)</span></li>
<li>kmymoney/views/kpayeesview.cpp <span style="color: grey">(69697e6)</span></li>
</ul>
<p><a href="https://git.reviewboard.kde.org/r/112847/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>