From 294e7ee7e918cf4d85ddf4eca1f5bcedabf561eb Mon Sep 17 00:00:00 2001 From: Shane Synan Date: Sun, 28 Feb 2016 18:03:37 -0600 Subject: [PATCH] Don't update IrcUser modes with channel modes Remove IrcUser mode updates in IrcChannel. IrcUsers are shared between channels, while channel user modes are not. Showing user modes in the Quassel tooltips need fixed client-side to check the current channel. --- src/common/ircchannel.cpp | 7 ------- 1 file changed, 7 deletions(-) diff --git a/src/common/ircchannel.cpp b/src/common/ircchannel.cpp index 6673a799..de7b8309 100644 --- a/src/common/ircchannel.cpp +++ b/src/common/ircchannel.cpp @@ -198,9 +198,6 @@ void IrcChannel::joinIrcUsers(const QList &users, const QStringList & // If you wonder why there is no counterpart to ircUserJoined: // the joins are propagated by the ircuser. The signal ircUserJoined is only for convenience - // Also update the IRC user's record of modes; this allows easier tracking - ircuser->addUserModes(modes[i]); - newNicks << ircuser->nick(); newModes << modes[i]; newUsers << ircuser; @@ -291,8 +288,6 @@ void IrcChannel::addUserMode(IrcUser *ircuser, const QString &mode) if (!_userModes[ircuser].contains(mode)) { _userModes[ircuser] += mode; - // Also update the IRC user's record of modes; this allows easier tracking - ircuser->addUserModes(mode); QString nick = ircuser->nick(); SYNC_OTHER(addUserMode, ARG(nick), ARG(mode)) emit ircUserModeAdded(ircuser, mode); @@ -314,8 +309,6 @@ void IrcChannel::removeUserMode(IrcUser *ircuser, const QString &mode) if (_userModes[ircuser].contains(mode)) { _userModes[ircuser].remove(mode); - // Also update the IRC user's record of modes; this allows easier tracking - ircuser->removeUserModes(mode); QString nick = ircuser->nick(); SYNC_OTHER(removeUserMode, ARG(nick), ARG(mode)); emit ircUserModeRemoved(ircuser, mode); -- 2.20.1