<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="https://git.reviewboard.kde.org/r/110212/">https://git.reviewboard.kde.org/r/110212/</a>
     </td>
    </tr>
   </table>
   <br />











<div>




<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="https://git.reviewboard.kde.org/r/110212/diff/6/?file=257407#file257407line2" style="color: black; font-weight: bold; text-decoration: underline;">kmymoney/kmymoneyui.rc</a>
    <span style="font-weight: normal;">

     (Diff revision 6)

    </span>
   </th>
  </tr>
 </thead>



 
 

 <tbody>

  <tr>
    <th bgcolor="#f0f0f0" style="border-right: 1px solid #C0C0C0;" align="right"><font size="2">2</font></th>
    <td bgcolor="#ffffff" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "><span class="o"><</span><span class="n">kpartgui</span> <span class="n">version</span><span class="o">=</span><span class="s">"39"</span> <span class="n">name</span><span class="o">=</span><span class="s">"kmymoney"</span> <span class="o">></span></pre></td>
    <th bgcolor="#f0f0f0" style="border-left: 1px solid #C0C0C0; border-right: 1px solid #C0C0C0;" align="right"><font size="2">2</font></th>
    <td bgcolor="#ffffff" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "><span class="o"><</span><span class="n">kpartgui</span> <span class="n">version</span><span class="o">=</span><span class="s">"39"</span> <span class="n">name</span><span class="o">=</span><span class="s">"kmymoney"</span> <span class="o">></span></pre></td>
  </tr>

 </tbody>

</table>

<pre style="margin-left: 2em; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Increment the version here by one to automatically use it. Otherwise, it won't work if you have a local copy in your $HOME/.kde directory tree.</pre>
</div>
<br />



<p>- Thomas Baumgart</p>


<br />
<p>On March 26th, 2014, 6:01 p.m. CET, Felipe Tonello wrote:</p>








<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('https://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 Felipe Tonello.</div>


<p style="color: grey;"><i>Updated March 26, 2014, 6:01 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;">payee: Does not merge when payees are empty

When there is no transactions to be merged we don't need to merge. In fact if
we don't stop there the rest of the code will delete the payees, which is a
behavior for delete payees functionality.

Signed-off-by: Felipe F. Tonello <eu@felipetonello.com>

payee: Show transactions from multiple payees

This patch allow users to have see all transactions that they
will merge.

Signed-off-by: Felipe F. Tonello <eu@felipetonello.com>

payee: Added Payees Merge KButton

Adds a button next to the delete payee and enables it only when multiple
payees are selected.

Signed-off-by: Felipe F. Tonello <eu@felipetonello.com>

payee: Added payee_merge KAction

Added KAction to the context menu and connected to the slotPayeeMerge().

Signed-off-by: Felipe F. Tonello <eu@felipetonello.com>

payee: Added support to Merge Payees

This patch implements the opeartion to merge payees. This is done by reusing
the payee delete slot, modifying it to support merge operation and add support
to it in KPayeeReassignDlg dialog.

The main difference between delete a payee and merging a payee is that when
merging the user can merge to a new payee. So the modification was done to
support this use-case.

Signed-off-by: Felipe F. Tonello <eu@felipetonello.com>

payee: change slotPayeeNew(...) to return boolean value

This is usefull if the user of that function wants to know if the user created
a new payee or not.

Signed-off-by: Felipe F. Tonello <eu@felipetonello.com>

payee: Make PayeeReassignDlg support other operations then delete

This is useful so it can be implemented a merge opration and maybe others in
future as well.

Signed-off-by: Felipe F. Tonello <eu@felipetonello.com>

payee: Better explanation when deleting payee

Signed-off-by: Felipe F. Tonello <eu@felipetonello.com></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;">Normal UI test going trough all the possible options of users input.</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/dialogs/kpayeereassigndlgdecl.ui <span style="color: grey">(2cf563d4ea755e8d89ff2587fa677f14453cbaa5)</span></li>

 <li>kmymoney/dialogs/kpayeereassigndlg.cpp <span style="color: grey">(8bdb25c0b3478c42715fac9f052903ed45b3614c)</span></li>

 <li>kmymoney/dialogs/kpayeereassigndlg.h <span style="color: grey">(a348d2b9fdc68a7c56a616a862a3373ef35aa0ba)</span></li>

 <li>kmymoney/kmymoney.h <span style="color: grey">(56e0bd6a3de42bdaa3fbb2f8586495cbb4d555ca)</span></li>

 <li>kmymoney/kmymoney.cpp <span style="color: grey">(6b96b9790d0ee5b5d34e6cfd382b03e9a103c9e6)</span></li>

 <li>kmymoney/kmymoneyui.rc <span style="color: grey">(65afc920cef7091e3138c9acf75f73ec96929445)</span></li>

 <li>kmymoney/views/kpayeesview.h <span style="color: grey">(c747e691f6509c7bf60201dfb3cc77e9f939a7c5)</span></li>

 <li>kmymoney/views/kpayeesview.cpp <span style="color: grey">(7252241886e05f00555d16dc1388b9d6ecef36a2)</span></li>

 <li>kmymoney/views/kpayeesviewdecl.ui <span style="color: grey">(6efb2ae5f94383851ee2ab15da36187b7079815a)</span></li>

</ul>

<p><a href="https://git.reviewboard.kde.org/r/110212/diff/" style="margin-left: 3em;">View Diff</a></p>







  </td>
 </tr>
</table>








  </div>
 </body>
</html>