continue;
if(queryItem->bufferName().toLower() == ircUser->nick().toLower()) {
- queryItem->attachIrcUser(ircUser);
+ queryItem->setIrcUser(ircUser);
break;
}
}
return;
IrcUser *ircUser = net->ircUser(bufferInfo.bufferName());
- if(ircUser)
- attachIrcUser(ircUser);
+ setIrcUser(ircUser);
}
QVariant QueryBufferItem::data(int column, int role) const {
}
}
+void QueryBufferItem::setBufferName(const QString &name) {
+ BufferItem::setBufferName(name);
+ NetworkId netId = data(0, NetworkModel::NetworkIdRole).value<NetworkId>();
+ const Network *net = Client::network(netId);
+ if(net)
+ setIrcUser(net->ircUser(name));
+}
+
QString QueryBufferItem::toolTip(int column) const {
// pretty much code duplication of IrcUserItem::toolTip() but inheritance won't solve this...
Q_UNUSED(column);
return QString("<p> %1 </p>").arg(toolTip.join("<br />"));
}
-void QueryBufferItem::attachIrcUser(IrcUser *ircUser) {
+void QueryBufferItem::setIrcUser(IrcUser *ircUser) {
+ if(_ircUser == ircUser)
+ return;
+
+ if(_ircUser) {
+ disconnect(_ircUser, 0, this, 0);
+ }
+
+ if(ircUser) {
+ connect(ircUser, SIGNAL(quited()), this, SLOT(removeIrcUser()));
+ connect(ircUser, SIGNAL(awaySet(bool)), this, SIGNAL(dataChanged()));
+ }
+
_ircUser = ircUser;
- connect(_ircUser, SIGNAL(quited()), this, SLOT(ircUserQuited()));
- connect(_ircUser, SIGNAL(awaySet(bool)), this, SIGNAL(dataChanged()));
emit dataChanged();
}
-void QueryBufferItem::ircUserQuited() {
+void QueryBufferItem::removeIrcUser() {
_ircUser = 0;
emit dataChanged();
}
inline BufferId bufferId() const { return _bufferInfo.bufferId(); }
inline BufferInfo::Type bufferType() const { return _bufferInfo.type(); }
- void setBufferName(const QString &name);
+ virtual void setBufferName(const QString &name);
virtual inline QString bufferName() const { return _bufferInfo.bufferName(); }
virtual inline QString topic() const { return QString(); }
virtual inline int nickCount() const { return 0; }
virtual QVariant data(int column, int role) const;
virtual bool setData(int column, const QVariant &value, int role);
+
virtual inline bool isActive() const { return (bool)_ircUser; }
virtual QString toolTip(int column) const;
+ virtual void setBufferName(const QString &name);
+
public slots:
- void attachIrcUser(IrcUser *ircUser);
- void ircUserQuited();
+ void setIrcUser(IrcUser *ircUser);
+ void removeIrcUser();
private:
IrcUser *_ircUser;
? newnick
: prefix;
+
+ // the order is cruicial
+ // otherwise the client would rename the buffer, see that the assigned ircuser doesn't match anymore
+ // and remove the ircuser from the querybuffer leading to a wrong on/offline state
+ ircuser->setNick(newnick);
coreSession()->renameBuffer(network()->networkId(), newnick, oldnick);
+
foreach(QString channel, ircuser->channels())
emit displayMsg(Message::Nick, BufferInfo::ChannelBuffer, channel, newnick, sender);
-
- ircuser->setNick(newnick);
}
void IrcServerHandler::handleNotice(const QString &prefix, const QList<QByteArray> ¶ms) {