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

     </td>

    </tr>

   </table>

   <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;">hi, thanks for the patch.

however, i'm not sure i understand the issue it is addressing.

the applicationjobs plasma dataengine isn't meant to have an extensive dbus interface at all really. that's kuiserver's job, as it's the actual host of it.

additionally, since the applicationjobs::JobView implements the org.kde.JobViewV2 interface, and methods like ::terminate already exist, that gives us bidirectional communication automatically. (you can test this by...when some long job is running, go to qdbusviewer, org.kde.plasma-desktop, DataEngine/applicationjobs/JobView_%1/org.kde.JobViewV2...and you will see all the methods implemented anyways, and the calls of those methods are propagated upwards to kuiserver).</pre>

 <br />

<p>- Shaun</p>

<br />

<p>On January 19th, 2012, 7:55 a.m., Bellegarde Cédric wrote:</p>

<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.org/media/rb/images/review_request_box_top_bg.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">

 <tr>

  <td>

<div>Review request for KDE Base Apps and kdelibs.</div>

<div>By Bellegarde Cédric.</div>

<p style="color: grey;"><i>Updated Jan. 19, 2012, 7:55 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;">Record job dbus id in Plasma::DataEngine object.

This will give a simple way to do things like this:

            QString path = "/JobViewServer/JobView_" + QString(data["jobDbusId"]);

            QDBusMessage m = QDBusMessage::createMethodCall(

                                   "org.kde.JobViewServer",

                                   path,

                                   "org.kde.JobViewV2",

                                   "terminate");

            bus.call(m);

when you want to terminate a job.</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>plasma/generic/dataengines/applicationjobs/kuiserverengine.cpp <span style="color: grey">(59a4de7)</span></li>

</ul>

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

  </td>

 </tr>

</table>

  </div>

 </body>

</html>