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

     </td>

    </tr>

   </table>

   <br />

<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">

 <p style="margin-top: 0;">On December 30th, 2013, 2:23 p.m. UTC, <b>David Faure</b> wrote:</p>

 <blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">

  <pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">This looks wrong to me. This code was in KFileWidget in kde4 already. OTOH the port of KEncodingFileDialog to KFileWidget is what's new in frameworks.

So I'd be quite tempted to say that the bug is in the way KEncodingFileDialog uses KFileWidget, unless you can prove otherwise :)

I'm afraid that changing this will break KFileWidget/KFileDialog.</pre>

 </blockquote>

</blockquote>

<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">If you have a look at KEncodinFileDialog, then it's just a dummy around the KFileWidget. Only signals that it's processing are terminal one. So IMHO the problem is in the KFileWidget. My bad, I should have posted example with KFileWidget directly. What has changed since KDE4 is also the KUrl (magic) -> QUrl (dummy). KFileDialog is in kde4support now, but yes, breaking it would be bad anyway. Will try to do some diffing between version, maybe something will pop out.</pre>

<br />

<p>- Michal</p>

<br />

<p>On December 28th, 2013, 11:33 a.m. UTC, Michal Humpula 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 KDE Frameworks.</div>

<div>By Michal Humpula.</div>

<p style="color: grey;"><i>Updated Dec. 28, 2013, 11:33 a.m.</i></p>

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

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

kio

</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;">If I understand correctly documentation of KFileWidget, it should be perfectly ok to do something like this:

KEncodingFileDialog::getOpenUrlsAndEncoding(QString(), 

QUrl("file:///etc/passwd"));

But that doesn't display the thing I'm expecting. Tracing it down I came up with the fix. I'm not claiming that it's the correct one, but at least in my situation the KFileWidget behaves as expected in all tested situations. 

Surprisingly the

void KFileWidgetPrivate::setLocationText(const QList<QUrl> &urlList)

doesn't call any setUrl, which hints that it could actually be correct.</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>src/filewidgets/kfilewidget.cpp <span style="color: grey">(11597b3)</span></li>

</ul>

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

  </td>

 </tr>

</table>

  </div>

 </body>

</html>