<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/111256/">http://git.reviewboard.kde.org/r/111256/</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="http://git.reviewboard.kde.org/r/111256/diff/1/?file=165888#file165888line55" style="color: black; font-weight: bold; text-decoration: underline;">KTp/Models/kpeopletranslationproxy.cpp</a>
<span style="font-weight: normal;">
(Diff revision 1)
</span>
</th>
</tr>
</thead>
<tbody>
<tr>
<th bgcolor="#e9eaa8" style="border-right: 1px solid #C0C0C0;" align="right"><font size="2">52</font></th>
<td bgcolor="#fdfebc" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "> <span class="n">IMPersonsDataSource</span> <span class="o">*</span><span class="n">imPlugin</span> <span class="o">=</span> <span class="n">qobject_cast</span><span class="o"><</span><span class="n">IMPersonsDataSource</span><span class="o">*></span><span class="p">(</span><span class="n">PersonPluginManager</span><span class="o">::</span><span class="n">presencePlugin</span><span class="p">());</span></pre></td>
<th bgcolor="#e9eaa8" style="border-left: 1px solid #C0C0C0; border-right: 1px solid #C0C0C0;" align="right"><font size="2">52</font></th>
<td bgcolor="#fdfebc" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "><span class="c1">// qDebug() << "requesting" << proxyIndex.row() << role;</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;">why adding commented qDebug?</pre>
</div>
<br />
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Yes, I think that it belongs to ktp-common-internals :)</pre>
<p>- Aleix</p>
<br />
<p>On June 26th, 2013, 2:19 p.m. UTC, David Edmundson wrote:</p>
<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 Telepathy.</div>
<div>By David Edmundson.</div>
<p style="color: grey;"><i>Updated June 26, 2013, 2:19 p.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;">Import the KPeople KTP integration plugins
There are two plugins, one for the datasource which provides
IM data in real time.
The other is an actions plugin which provides actions that can be started on a PersonsData.
We need to provide KTp integration into libkpeople in a plugin, but we also want to access this
singleton of accounts/contacts from the other plugin and from the KTp Models. To do this a static library
ktpkpeople is created and everything links against that.
KTp Models the actions plugin and the im data source all link against this.
</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>kpeople/actionsplugin/implugin.h <span style="color: grey">(PRE-CREATION)</span></li>
<li>kpeople/actionsplugin/CMakeLists.txt <span style="color: grey">(PRE-CREATION)</span></li>
<li>KTp/Models/kpeopletranslationproxy.cpp <span style="color: grey">(PRE-CREATION)</span></li>
<li>KTp/Models/CMakeLists.txt <span style="color: grey">(720a5b02465b825b08d12d7c43342ad1ba69c9d1)</span></li>
<li>CMakeLists.txt <span style="color: grey">(a83069d59fed945d04560662e3a307ef33b28332)</span></li>
<li>kpeople/actionsplugin/implugin.cpp <span style="color: grey">(PRE-CREATION)</span></li>
<li>kpeople/actionsplugin/ktp_kpeople_plugin.desktop <span style="color: grey">(PRE-CREATION)</span></li>
<li>kpeople/common/CMakeLists.txt <span style="color: grey">(PRE-CREATION)</span></li>
<li>kpeople/common/impersonsdatasource.h <span style="color: grey">(PRE-CREATION)</span></li>
<li>kpeople/common/impersonsdatasource.cpp <span style="color: grey">(PRE-CREATION)</span></li>
<li>kpeople/datasourceplugin/CMakeLists.txt <span style="color: grey">(PRE-CREATION)</span></li>
<li>kpeople/datasourceplugin/im_persons_data_source_plugin.desktop <span style="color: grey">(PRE-CREATION)</span></li>
<li>kpeople/datasourceplugin/impersonsdatasourceplugin.cpp <span style="color: grey">(PRE-CREATION)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/111256/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>