[frameworks-kio] [Bug 352073] "http://ksnapshot/" is opened in web browser if KSnapshot is not installed

[Nate Graham]

https://bugs.kde.org/show_bug.cgi?id=352073

Nate Graham  changed:

   What|Removed |Added

 CC||n...@kde.org
 Status|UNCONFIRMED |RESOLVED
 Resolution|--- |FIXED

--- Comment #7 from Nate Graham  ---
No longer an issue with Spectacle, and ssues with stale files would not be
KIO's fault.

-- 
You are receiving this mail because:
You are watching all bug changes.

[frameworks-kio] [Bug 352073] "http://ksnapshot/" is opened in web browser if KSnapshot is not installed

[Thomas Lübking via KDE Bugzilla]

https://bugs.kde.org/show_bug.cgi?id=352073

--- Comment #6 from Thomas Lübking  ---
https://quickgit.kde.org/?p=khotkeys.git=blob=f107826df7ceccb81f37d13aee2760cf9803c1a4=331336dfbdf7e6cc0bfc236913eea4a079681c65=libkhotkeysprivate%2Factions%2Fcommand_url_action.cpp

Looks like it should be

if( !KRun::runCommand(
cmd + ( uri.hasArgsAndOptions() ? uri.argsAndOptions() : "" ),
cmd, uri.iconName(), NULL ))

but could as well end up being

KUriFilterData::LocalFile =>  new KRun( uri.uri(), 0L);

I'd have to test myself (was just looking at some khotkeys bug while fixing
some others ;-)

-- 
You are receiving this mail because:
You are watching all bug changes.

[frameworks-kio] [Bug 352073] "http://ksnapshot/" is opened in web browser if KSnapshot is not installed

[David Faure via KDE Bugzilla]

https://bugs.kde.org/show_bug.cgi?id=352073

--- Comment #5 from David Faure  ---
new KRun(QUrl("random_binary")) doesn't open http://random_binary, it just
outputs "KRun::scanFile:  NO SUPPORT FOR READING!"

Do you have a more info about how exactly KRun is being called?

In general http:// being prepended means there's a QUrl::fromUserInput(str)
call somewhere.

-- 
You are receiving this mail because:
You are watching all bug changes.

[frameworks-kio] [Bug 352073] "http://ksnapshot/" is opened in web browser if KSnapshot is not installed

[Thomas Lübking via KDE Bugzilla]

https://bugs.kde.org/show_bug.cgi?id=352073

Thomas Lübking  changed:

   What|Removed |Added

Product|khotkeys|frameworks-kio
 CC||kdelibs-b...@kde.org,
   ||thomas.luebk...@gmail.com
  Component|general |general
Version|5.4.0   |unspecified
   Assignee|k...@michael-jansen.biz  |fa...@kde.org

--- Comment #4 from Thomas Lübking  ---
We'd probably need some kind of kconf_update script to remove the ksnapshot
bits when installing spectacle (though it's still possible to have both) - or
in khotkeys (shipping default actions with khotkeys was probably a mistake ...
15 years ago ;-)

At least there's no ksnapshot (printscreen) default action anymore.

-> *Maybe* krun should not interpret "random_binary" as protocol/url.
Asking kio group.

-- 
You are receiving this mail because:
You are watching all bug changes.