[Konversation-devel] [Bug 214258] New: Commands like /op and /deop cause an invalid MODE command to be send preceding a correct one

[Eike Hein]

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

           Summary: Commands like /op and /deop cause an invalid MODE
                    command to be send preceding a correct one
           Product: konversation
           Version: SVN
          Platform: Unlisted Binaries
        OS/Version: Linux
            Status: NEW
          Severity: normal
          Priority: NOR
         Component: protocol
        AssignedTo: konversation-devel at kde.org
        ReportedBy: hein at kde.org


"/op <target>" will cause Konvi to send "MODE #currentchan +o" before "MODE
#currentchan +o <target>", and similar for "/deop" and various other commands
that make use of OutputFilter::changeMode(). Here's some background bandied
about on IRC:

[13:35] <Sho_> psn: In case you want to take a stab at this later when I'm
likely offline: There's a for loop in OutputFilter::changeMode() to produce
things like "+ooo nick nick nick", and the if block in that loop block there
seems to be stupid and the cause for Elmaron's problem of Konvi sending "MODE
#chan +o" before "MODE #chan +o nick" on /op (also happens on /deop etc), but
I'm too fever-addled atm to think through the ramifications or deduce why the
original author 
[13:35] <Sho_> added it.
[13:35] <Sho_> psn: Change dates back to
http://websvn.kde.org/tags/konversation/0.15/kdeextragear-2/konversation/konversation/outputfilter.cpp?r1=239559&r2=253633
[13:36] <Sho_> psn: I tried 1.1 and the same bug is in there, too

-- 
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.