Bug 137779

Summary: konversation ignores aliases during connect (called by 'commands' option in network tab)
Product: [Applications] konversation Reporter: Kevin Funk <kfunk>
Component: generalAssignee: Konversation Bugs <konversation-bugs-null>
Status: RESOLVED FIXED    
Severity: normal    
Priority: NOR    
Version First Reported In: unspecified   
Target Milestone: ---   
Platform: unspecified   
OS: Linux   
Latest Commit: Version Fixed/Implemented In:
Sentry Crash Report:

Description Kevin Funk 2006-11-23 17:17:13 UTC 
Version:           1.0.1 (using KDE 3.5.5, Gentoo)
Compiler:          Target: i686-pc-linux-gnu
OS:                Linux (i686) release 2.6.17.13

Konversation does not recognize my aliases i've set up in the options during connect.
I use the alias in the network tab where it should be executed at connect time.
Instead, it drops an "Unknown command" error.
Running the alias manually works fine.
Thanks
Comment 1 Eike Hein 2006-11-23 19:00:28 UTC 
SVN commit 607213 by hein:

Support command aliases in network connect commands.
BUG:137779


 M  +1 -0      ChangeLog  
 M  +2 -1      src/server.cpp  


--- trunk/extragear/network/konversation/ChangeLog #607212:607213
@@ -43,6 +43,7 @@
   position rather than the end of the input line contents.
 - Fixed bug leading to crash upon initiating DCC Chat when "Focus new tabs"
   was enabled.
+- Support command aliases in network connect commands.
 -------------------------------------------------------------------------------
 
 Changes from 1.0 to 1.0.1
--- trunk/extragear/network/konversation/src/server.cpp #607212:607213
@@ -974,7 +974,8 @@
         {
             QString output(*iter);
             output = output.simplifyWhiteSpace();
-            Konversation::OutputFilterResult result = outputFilter->parse(getNickname(),output,QString::null);
+            getOutputFilter()->replaceAliases(output);
+            Konversation::OutputFilterResult result = getOutputFilter()->parse(getNickname(),output,QString::null);
             queue(result.toServer);
         }
     }