<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="http://git.reviewboard.kde.org/r/107619/">http://git.reviewboard.kde.org/r/107619/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On December 7th, 2012, 6:50 p.m., <b>Cristian Oneț</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<table width="100%" border="0" bgcolor="white" style="border: 1px solid #C0C0C0; border-collapse: collapse; margin: 2px padding: 2px;">
<thead>
<tr>
<th colspan="4" bgcolor="#F0F0F0" style="border-bottom: 1px solid #C0C0C0; font-size: 9pt; padding: 4px 8px; text-align: left;">
<a href="http://git.reviewboard.kde.org/r/107619/diff/1/?file=97424#file97424line865" style="color: black; font-weight: bold; text-decoration: underline;">kmymoney/views/khomeview.cpp</a>
<span style="font-weight: normal;">
(Diff revision 1)
</span>
</th>
</tr>
</thead>
<tbody style="background-color: #e4d9cb; padding: 4px 8px; text-align: center;">
<tr>
<td colspan="4"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">void KHomeView::showAccountEntry(const MyMoneyAccount& acc, const MyMoneyMoney& value, const MyMoneyMoney& valueToMinBal, const bool showMinBal)</pre></td>
</tr>
</tbody>
<tbody>
<tr>
<th bgcolor="#b1ebb0" style="border-right: 1px solid #C0C0C0;" align="right"><font size="2"></font></th>
<td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "></pre></td>
<th bgcolor="#b1ebb0" style="border-left: 1px solid #C0C0C0; border-right: 1px solid #C0C0C0;" align="right"><font size="2">865</font></th>
<td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "> <span class="k">if</span> <span class="p">(</span><span class="n">QDate</span><span class="o">::</span><span class="n">fromString</span><span class="p">(</span><span class="n">acc</span><span class="p">.</span><span class="n">value</span><span class="p">(</span><span class="s">"lastStatementDate"</span><span class="p">),</span> <span class="n">Qt</span><span class="o">::</span><span class="n">ISODate</span><span class="p">)</span> <span class="o">></span> <span class="n">QDate</span><span class="o">::</span><span class="n">fromString</span><span class="p">(</span><span class="n">acc</span><span class="p">.</span><span class="n">value</span><span class="p">(</span><span class="s">"lastImportedTransactionDate"</span><span class="p">),</span> <span class="n">Qt</span><span class="o">::</span><span class="n">ISODate</span><span class="p">))</span></pre></td>
</tr>
</tbody>
</table>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">You are comparing the last reconciliation date with the last imported transaction date, it that what you really want?</pre>
</blockquote>
<p>On December 8th, 2012, 8:15 p.m., <b>Marko Käning</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 thought that "lastStatementDate" is the last transaction in the ledger. Now you tell me that it is the last reconciled one... OK, that explains it.
How do I find the date of the last transaction in the ledger?</pre>
</blockquote>
<p>On December 10th, 2012, 4:04 p.m., <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;">I'm not sure that you can obtain that date without fetching the transactions list and looking at the last transaction. Thomas, could you give some hints about this?</pre>
</blockquote>
</blockquote>
<pre style="margin-left: 1em; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Oh, I was afraid of that... It's probably not a very quick thing to do, I mean to determine in such a way the latest transaction for every single account displayed.
It would be good to know where these date values like "lastStatementDate", "lastImportedTransactionDate" and alike are documented in the code. Could you give me a hint for future reference?</pre>
<br />
<p>- Marko</p>
<br />
<p>On December 6th, 2012, 9:41 p.m., Marko Käning wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.org/media/rb/images/review_request_box_top_bg.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for KMymoney.</div>
<div>By Marko Käning.</div>
<p style="color: grey;"><i>Updated Dec. 6, 2012, 9:41 p.m.</i></p>
<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;">The goal is to show another icon in homepage accounts tables if there are transactions for an account after the last online transaction.
Unfortunately this is not yet functioning as expected! :-(
Obviously my date comparison in line 865 doesn't work the way I thought it would do.
What am I doing wrong here?</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;">Does not yet show the desired icon. :-(</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/views/khomeview.cpp <span style="color: grey">(c79337176b7265cabe15cad4972bc719e797af7c)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/107619/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>