diff options
| author | Timothy Pearson <kb9vqf@pearsoncomputing.net> | 2013-04-06 01:52:59 -0500 |
|---|---|---|
| committer | Timothy Pearson <kb9vqf@pearsoncomputing.net> | 2013-04-06 01:52:59 -0500 |
| commit | 6fd6724b2f4c1a0bed961eeb267381c7d54746c0 (patch) | |
| tree | a35d6ed6a59bc81a8d1d30f170cdc5f8d6eee8ad /khotkeys/shared | |
| parent | 02d43b793cdfc6e9a5ef09397619654e4d69bbc0 (diff) | |
| download | tdebase-6fd6724b2f4c1a0bed961eeb267381c7d54746c0.tar.gz tdebase-6fd6724b2f4c1a0bed961eeb267381c7d54746c0.zip | |
Add warning messages on hot key command execution failure
Diffstat (limited to 'khotkeys/shared')
| -rw-r--r-- | khotkeys/shared/actions.cpp | 9 |
1 files changed, 7 insertions, 2 deletions
diff --git a/khotkeys/shared/actions.cpp b/khotkeys/shared/actions.cpp index 3174b1dcf..c8590789a 100644 --- a/khotkeys/shared/actions.cpp +++ b/khotkeys/shared/actions.cpp @@ -29,6 +29,7 @@ #include <tdeaccel.h> #include <kservice.h> #include <kprocess.h> +#include <tdemessagebox.h> #include "windows.h" #include "action_data.h" @@ -153,7 +154,10 @@ void Command_url_action::execute() KService::Ptr service = KService::serviceByDesktopName( cmd ); if( service != NULL ) { - KRun::run( *service, KURL::List()); + if (!KRun::run( *service, KURL::List())) + { + KMessageBox::sorry(0, "<qt>" + i18n("KHotKeys was unable to execute") + " '" + cmd + "'<p>" + i18n("Please verify existence of the service") + "</qt>", i18n("Unable to launch service!")); + } break; } } @@ -166,11 +170,12 @@ void Command_url_action::execute() if( !KRun::runCommand( cmd + ( uri.hasArgsAndOptions() ? uri.argsAndOptions() : "" ), cmd, uri.iconName())) { - // CHECKME ? + KMessageBox::sorry(0, "<qt>" + i18n("KHotKeys was unable to execute") + " '" + cmd + "'<p>" + i18n("Please verify existence and permissions of the executable file") + "</qt>", i18n("Unable to launch program")); } break; } default: // error + KMessageBox::sorry(0, "<qt>" + i18n("KHotKeys was unable to execute") + " '" + cmd + "'<p>" + i18n("Please verify existence and permissions of the executable file") + "</qt>", i18n("Unable to launch program")); return; } timeout.start( 1000, true ); // 1sec timeout |