<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/115828/">https://git.reviewboard.kde.org/r/115828/</a>

     </td>

    </tr>

   </table>

   <br />

 <p>Ship it!</p>

 <pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Probably nothing better we can do now...</pre>

 <br />

<p>- Martin Klapetek</p>

<br />

<p>On February 17th, 2014, 1:22 p.m. CET, David Edmundson 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 Telepathy.</div>

<div>By David Edmundson.</div>

<p style="color: grey;"><i>Updated Feb. 17, 2014, 1:22 p.m.</i></p>

<div style="margin-top: 1.5em;">

 <b style="color: #575012; font-size: 10pt;">Repository: </b>

ktp-common-internals

</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;">Add a workaround for broken sorting

There is a bug in QSortFilterProxyModel in which if sort() is called

on a model where all items are filtered it will not start sorting

as it cannot map the proxy column to the source column.

This gets updated whenever there's an insertRows but not from

a dataChanged. With the kpeople cache this means that it's not sorting

as we never do an insertRows.

This bug is fixed in Qt5.2, but not in 4.x

</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>KTp/Models/contacts-filter-model.h <span style="color: grey">(a891d6b)</span></li>

 <li>KTp/Models/contacts-filter-model.cpp <span style="color: grey">(df12112)</span></li>

</ul>

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

  </td>

 </tr>

</table>

  </div>

 </body>

</html>