Fernando, Tony,<br>can you verify whether the DB backend is affected by this bug, and review this patch accordingly?<br><br>Regards,<br>Alvaro<br><br><div class="gmail_quote">On Tue, Jun 22, 2010 at 9:43 AM, Vincent Ordy <span dir="ltr"><<a href="mailto:vincent@ordy.net">vincent@ordy.net</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
<div>
<div style="font-family: Verdana,Arial,Helvetica,Sans-Serif;"><div class="im">
<table style="border: 1px solid rgb(201, 195, 153);" bgcolor="#f9f3c9" cellpadding="8" width="100%">
<tbody><tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="http://reviewboard.kde.org/r/4417/" target="_blank">http://reviewboard.kde.org/r/4417/</a>
</td>
</tr>
</tbody></table>
<br>
</div><div class="im"><blockquote style="margin-left: 1em; border-left: 2px solid rgb(208, 208, 208); padding-left: 10px;">
<p style="margin-top: 0pt;">On June 22nd, 2010, 9:27 a.m., <b>Thomas Baumgart</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid rgb(208, 208, 208); padding-left: 10px;">
<pre>I understand and support the changes to the XML backend, but I don't know if we really need a progress bar for the SQL backend. In case we do, please replace QObject::tr with a call to KDE's i18n function.
I noticed two things in the XML backend changes:
int count = atts.value(QString("count")).toUInt();
a) Why do you use an "int" and assign it an "unsigned int"?
b) Using QLatin1String("count") is better than QString("count")
Details for b) can be found on <a href="http://labs.trolltech.com/blogs/2008/04/28/string-theory/" target="_blank">http://labs.trolltech.com/blogs/2008/04/28/string-theory/</a></pre>
</blockquote>
</blockquote>
</div><pre>This line is a copy of lines 167, 174, 181, 188, and 195 that were already there. I should have payed attention though. Thanks for the pointer on QLatin1String.
I'm going to update the patch to fix that.
Alvaro asked me on bugzilla if the database storage was affected by the progress bar bug so I fixed it, but I really don't mind if you drop it (I've never even used the SQL storage before).</pre>
<br><font color="#888888">
<p>- Vincent</p></font><div class="im">
<br>
<p>On June 22nd, 2010, 4:58 a.m., Vincent Ordy wrote:</p>
<table style="background-repeat: repeat-x; border: 1px solid black;" bgcolor="#fefadf" cellpadding="8" cellspacing="0" width="100%">
<tbody><tr>
<td>
<div>Review request for kmymoney.</div>
<div>By Vincent Ordy.</div>
<p style="color: grey;"><i>Updated 2010-06-22 04:58:04</i></p>
<h1 style="color: rgb(87, 80, 18); font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table style="border: 1px solid rgb(184, 181, 160);" bgcolor="#ffffff" cellpadding="10" cellspacing="0" width="100%">
<tbody><tr>
<td>
<pre style="margin: 0pt; padding: 0pt;">- set progressbar value to 0 after a reset (reset() sets it to -1, which is
invalid and cause this bug)
- increment the progressbar value only when appropriate. Some elements like <budgets> do not initialize the progressbar, but the progressbar value was incremented => the value become greater than the maximum => the progressbar is reset by Qt.
- remove useless resets</pre>
</td>
</tr>
</tbody></table>
<h1 style="color: rgb(87, 80, 18); font-size: 10pt; margin-top: 1.5em;">Testing </h1>
<table style="border: 1px solid rgb(184, 181, 160);" bgcolor="#ffffff" cellpadding="10" cellspacing="0" width="100%">
<tbody><tr>
<td>
<pre style="margin: 0pt; padding: 0pt;">build + load 2 files w/ the progressbar timer disabled (kmymoney/kmymoney.cpp:1932) to slow down the progress</pre>
</td>
</tr>
</tbody></table>
<div style="margin-top: 1.5em;">
<b style="color: rgb(87, 80, 18); font-size: 10pt; margin-top: 1.5em;">Bugs: </b>
<a href="https://bugs.kde.org/show_bug.cgi?id=240629" target="_blank">240629</a>
</div>
<h1 style="color: rgb(87, 80, 18); font-size: 10pt; margin-top: 1.5em;">Diffs </h1>
<ul style="margin-left: 3em; padding-left: 0pt;">
<li>/trunk/extragear/office/kmymoney/kmymoney/kmymoney.cpp <span style="color: grey;">(1140640)</span></li>
<li>/trunk/extragear/office/kmymoney/kmymoney/mymoney/storage/mymoneystoragesql.cpp <span style="color: grey;">(1140640)</span></li>
<li>/trunk/extragear/office/kmymoney/kmymoney/mymoney/storage/mymoneystoragexml.cpp <span style="color: grey;">(1140640)</span></li>
</ul>
<p><a href="http://reviewboard.kde.org/r/4417/diff/" style="margin-left: 3em;" target="_blank">View Diff</a></p>
</td>
</tr>
</tbody></table>
</div></div>
</div>
<br>_______________________________________________<br>
KMyMoney-devel mailing list<br>
<a href="mailto:KMyMoney-devel@kde.org">KMyMoney-devel@kde.org</a><br>
<a href="https://mail.kde.org/mailman/listinfo/kmymoney-devel" target="_blank">https://mail.kde.org/mailman/listinfo/kmymoney-devel</a><br>
<br></blockquote></div><br>