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


<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.orgrb/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 kdelibs.</div>
<div>By Dawit Alemayehu.</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;">The attached patch fixes the problem reported in the bug reports listed above. Specifically, clicking on ftp files seems to not work on more occasions than not for me and reporters of those bugs. In my own local tests, clicking on a PDF file simply results in okular being opened and nothing displayed. Actually a copy completed notification with an "open" button is shown and clicking on this button opens the PDF file in another instance of okular. For text files kate gives a warning message about not being able to read the file and to check permission. In this case too a status notification is shown however and clicking on it results in the same behavior as the PDF case.

The problem seems to stem from the fact that in case of Ftp::copy, which is used in both of the examples I gave above, the finished signal is emitted before calling the ftpCloseCommand. Reversing that order as this patch does fixes the problem completely here. I suspect socket io buffering to be the culprit, but I did not dig enough to say definitively.</pre>
  </td>
 </tr>
</table>




<div style="margin-top: 1.5em;">
 <b style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Bugs: </b>


 <a href="http://bugs.kde.org/show_bug.cgi?id=181393">181393</a>, 

 <a href="http://bugs.kde.org/show_bug.cgi?id=203445">203445</a>


</div>


<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">

 <li>kioslave/ftp/ftp.cpp <span style="color: grey">(7414588)</span></li>

</ul>

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




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




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