Bug 198371 - Inconsistent display of channel modes when option to use raw modes is disabled
Summary: Inconsistent display of channel modes when option to use raw modes is disabled
Status: RESOLVED FIXED
Alias: None
Product: konversation
Classification: Applications
Component: protocol (show other bugs)
Version: Git
Platform: Unlisted Binaries Linux
: NOR normal
Target Milestone: ---
Assignee: Konversation Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-06-30 00:34 UTC by Eike Hein
Modified: 2009-07-22 15:32 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Eike Hein 2009-06-30 00:34:37 UTC
Konversation has a "Use raw modes for mode changes" option. When this option is disabled (which it defaults to), the server response to "/mode <channel>" will be displayed in such a way that channel mode chars are replaced by human-readable strings, e.g. "moderated" for the "m" mode char. The same is true for the server response to a "/mode channel +/-<mode>" command (or someone else setting/removing a mode), however the list of human-readable strings used in those two cases is inconsistent. For example, while the "r" channel mode is shown as "server reop" in the response to "/mode <channel>", in the response to "/mode <channel> +/-<mode>" it is left as "r".

Also, the Channel Settings Dialog ignores the setting and always uses mode chars in its Advanced Modes table.

And finally, as the setting affects more than only mode changes, it should have a more generic name covering all of its effects.
Comment 1 Eike Hein 2009-06-30 00:44:44 UTC
SVN commit 989313 by hein:

Use a better name for the raw modes option.
CCBUG:198371


 M  +1 -1      chatwindowbehaviour_config.ui  


WebSVN link: http://websvn.kde.org/?view=rev&revision=989313
Comment 2 Travis McHenry 2009-07-22 14:11:19 UTC
SVN commit 1000978 by tjmchenry:

Move raw mode -> mode strings to a qhash in common.cpp
Use this in channeloptionsdialog's 'show advanced modes' list
CCBUG:198371
Add CHANMODES from RPL_ISUPPORT to the allowedChannelModes()
BUG:198365



 M  +26 -0     common.cpp  
 M  +8 -0      common.h  
 M  +9 -3      irc/channeloptionsdialog.cpp  
 M  +36 -20    irc/inputfilter.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=1000978
Comment 3 Travis McHenry 2009-07-22 15:32:27 UTC
SVN commit 1001026 by tjmchenry:

Move the get for the channelmodeshash to a smarter location
Remove a no longer needed spell check setting on-close
Make it so modes are translated if known with /mode #channel +/-<mode>
BUG:198371

 M  +23 -8     channel.cpp  
 M  +1 -1      inputfilter.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=1001026