<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/110587/">http://git.reviewboard.kde.org/r/110587/</a>
</td>
</tr>
</table>
<br />
<table bgcolor="#e0e0e0" width="100%" cellpadding="8" style="border: 1px gray solid;">
<tr>
<td>
<h1 style="margin-right: 0.2em; padding: 0; font-size: 10pt;">This change has been marked as submitted.</h1>
</td>
</tr>
</table>
<br />
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.org/static/rb/images/review_request_box_top_bg.ab6f3b1072c9.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for KMymoney.</div>
<div>By Allan Anderson.</div>
<p style="color: grey;"><i>Updated May 24, 2013, 11:30 a.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;">When I found recently that I couldn't edit multiple investment transactions, and started to investigate, I discovered this old bug, which had resulted in the disabling of the feature. I fixed the reported problem and re-enabled that code. That fix was committed as 3f45cfc6996f9c2cd2fa01dd0232051c50075651 on 2 April. When I investigated further, I discovered there were some additional issues, which Thomas felt should be left in abeyance pending a re-write of the Ledger code.
I then found that there was a potential danger of a user editing unlike investment types and not noticing that one of the types could get changed inadvertently without his noticing, particularly if a number of transactions were being edited at once. This has therefore now been fixed and editing of unlike investment types is no longer possible.
It was also noticed that the tooltips in this part of /kmymoney/views/kgloballedgerview.cpp did not work as I expected, in that there was no tooltip visible on the transaction in question, although it was found that the tooltips were attached to the edit button. It was felt that this wasn't very helpful, as the user might have attempted the edit from the context menu or by double-click. So, I have added code to display error information at the transaction position. I haven't done this for every error message, where I was unable to determine how to produce the error condition. However, I have covered the mixed investment types situation, multiple splits, and flagging any attempt to edit where a closed account was involved. The original code for tooltips has been left in place, however.</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;">Ensured that attempts to edit unlike items is prevented, and also to create or edit transactions where a closed account is involved. Unit test run without error.</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/kmymoney.cpp <span style="color: grey">(063f8b5)</span></li>
<li>kmymoney/views/kgloballedgerview.h <span style="color: grey">(0041ea2)</span></li>
<li>kmymoney/views/kgloballedgerview.cpp <span style="color: grey">(8a2c592)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/110587/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>