anthonyfieroni added inline comments.

INLINE COMMENTS

> anthonyfieroni wrote in krun.cpp:1319
> You cannot remove it, since it can be a local file, remote file with scheme 
> that differs http and should be redirect then. So you can make it like that
> 
>   if (!job->url().scheme().startsWith("http")) {
>       setUrl(job->url());
>   }
> 
> Now it will not redirect only http and https.

Also you can add Qt::CaseInsensitive as second parameter to startsWith for 
scheme with capital letters.

REPOSITORY
  R241 KIO

REVISION DETAIL
  https://phabricator.kde.org/D15189

To: achauvel, #frameworks, dfaure, cfeck
Cc: anthonyfieroni, ngraham, kde-frameworks-devel, michaelh, bruns

Reply via email to