<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://svn.reviewboard.kde.org/r/5298/">http://svn.reviewboard.kde.org/r/5298/</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 10th, 2010, 2:57 a.m., <b>Alvaro Soliverez</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="/r/5298/diff/1/?file=35571#file35571line2330" style="color: black; font-weight: bold; text-decoration: underline;">/trunk/extragear/office/kmymoney/kmymoney/reports/pivottable.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; ">int PivotTable::currentDateColumn(void)</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">2330</font></th>
<td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "><span class="kt">bool</span> <span class="n">PivotTable</span><span class="o">::</span><span class="n">periodHasPrice</span><span class="p">(</span><span class="n">ReportAccount</span><span class="o">&</span> <span class="n">account</span><span class="p">,</span> <span class="n">QDate</span><span class="o">&</span> <span class="n">dateStart</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;">if the method in ReportAccount works, you can drop this.</pre>
</blockquote>
<p>On September 23rd, 2010, 7:17 p.m., <b>Bernd Gonsior</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;">There is a problem: method ReportAccount::deepCurrencyPrice is called only once for a period (e.g. a month) from PivotTable::convertToDeepCurrency. To retrieve prices which are not exactly at the last day of the current period (that way the call was done so far) this new method has to iterate over all possible dates from start to end of period.</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;">price reports, which is where this is going to be used, are daily. That's how it works now, and there is no need to change that.</pre>
<br />
<p>- Alvaro</p>
<br />
<p>On September 23rd, 2010, 7:17 p.m., Bernd Gonsior wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://svn.reviewboard.kde.orgrb/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 Bernd Gonsior.</div>
<p style="color: grey;"><i>Updated 2010-09-23 19:17:13</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;">Patch for investment reports only: prices are shown only if at least one price
exists in a requested period.</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>/trunk/extragear/office/kmymoney/kmymoney/dialogs/kreportconfigurationfilterdlg.cpp <span style="color: grey">(1178669)</span></li>
<li>/trunk/extragear/office/kmymoney/kmymoney/mymoney/mymoneyreport.h <span style="color: grey">(1178669)</span></li>
<li>/trunk/extragear/office/kmymoney/kmymoney/mymoney/mymoneyreport.cpp <span style="color: grey">(1178669)</span></li>
<li>/trunk/extragear/office/kmymoney/kmymoney/reports/kreportchartview.cpp <span style="color: grey">(1178669)</span></li>
<li>/trunk/extragear/office/kmymoney/kmymoney/reports/pivottable.cpp <span style="color: grey">(1178669)</span></li>
<li>/trunk/extragear/office/kmymoney/kmymoney/reports/reportaccount.h <span style="color: grey">(1178669)</span></li>
<li>/trunk/extragear/office/kmymoney/kmymoney/reports/reportaccount.cpp <span style="color: grey">(1178669)</span></li>
<li>/trunk/extragear/office/kmymoney/kmymoney/widgets/kmymoneyreportconfigtab1decl.ui <span style="color: grey">(1178669)</span></li>
</ul>
<p><a href="http://svn.reviewboard.kde.org/r/5298/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>