[Konsole-devel] [Bug 113515] Field 'Execute' field is considered mandatory in Konsole 3.5.0-beta1 configuration, tab 'Session'

[Kurt V.Hindenburg]

------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
         
http://bugs.kde.org/show_bug.cgi?id=113515         
kurt.hindenburg kdemail net changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
         Resolution|                            |FIXED



------- Additional Comments From kurt.hindenburg kdemail net  2005-09-28 18:15 -------
SVN commit 464922 by hindenburg:

If Execute field empty, don't display warning about invalid Execute entry.

BUG: 113515


 M  +11 -11    sessioneditor.cpp  


--- branches/KDE/3.5/kdebase/kcontrol/konsole/sessioneditor.cpp #464921:464922
 @ -290,22 +290,22  @
   // Verify Execute entry is valid; otherwise Konsole will ignore it.
   // This code is take from konsole.cpp; if you change one, change both.
   QString exec = executeLine->text();
-  if ( exec.startsWith( "su -c \'" ) ) {
-     exec = exec.mid( 7, exec.length() - 8 );
-  }
-  exec = KRun::binaryName( exec, false );
-  exec = KShell::tildeExpand( exec );
-  QString pexec = KGlobal::dirs()->findExe( exec );
+  if ( !exec.isEmpty() )  // If Execute field is empty, default shell is used.
+  {
+    if ( exec.startsWith( "su -c \'" ) )
+      exec = exec.mid( 7, exec.length() - 8 );
+    exec = KRun::binaryName( exec, false );
+    exec = KShell::tildeExpand( exec );
+    QString pexec = KGlobal::dirs()->findExe( exec );
 
-  if ( pexec.isEmpty() )
-  {
-    int result = KMessageBox::warningContinueCancel( this,
+    if ( pexec.isEmpty() )
+    {
+      int result = KMessageBox::warningContinueCancel( this,
             i18n( "The Execute entry is not a valid command.\n"
 			"You can still save this session, but it will not show up in Konsole's Session list." ),
 			i18n( "Invalid Execute Entry" ),
 			KStdGuiItem::save() );
-    if ( result != KMessageBox::Continue )
-    {
+      if ( result != KMessageBox::Continue )
         return;
     }