X-Git-Url: https://git.quassel-irc.org/?p=quassel.git;a=blobdiff_plain;f=src%2Fclient%2Fmessagemodel.cpp;h=76104bd241a2dcd0e84c21fd8463d356a6d7b52a;hp=c41ad9733105483d92487a3a5b722f6fa4c3cb3d;hb=3a9668461efb05bd5b24ba5eb741a7f7f75086c9;hpb=22f141be889377b07472bb967d92186dad23be3e diff --git a/src/client/messagemodel.cpp b/src/client/messagemodel.cpp index c41ad973..76104bd2 100644 --- a/src/client/messagemodel.cpp +++ b/src/client/messagemodel.cpp @@ -34,8 +34,9 @@ MessageModel::~MessageModel() { } QVariant MessageModel::data(const QModelIndex &index, int role) const { - int row = index.row(); - if(row < 0 || row >= _messageList.count()) return QVariant(); + int row = index.row(); int column = index.column(); + if(row < 0 || row >= _messageList.count() || column < 0) return QVariant(); + if(role == ColumnTypeRole) return column; return _messageList[row]->data(index.column(), role); } @@ -43,7 +44,7 @@ bool MessageModel::setData(const QModelIndex &index, const QVariant &value, int int row = index.row(); if(row < 0 || row >= _messageList.count()) return false; if(_messageList[row]->setData(index.column(), value, role)) { - emit dataChanged(index, index); // FIXME make msg emit this (too) + emit dataChanged(index, index); return true; } return false; @@ -52,7 +53,7 @@ bool MessageModel::setData(const QModelIndex &index, const QVariant &value, int void MessageModel::insertMessage(const Message &msg) { MsgId id = msg.msgId(); int idx = indexForId(id); - MessageItem *item = createMessageItem(msg); + MessageModelItem *item = createMessageModelItem(msg); beginInsertRows(QModelIndex(), idx, idx); _messageList.insert(idx, item); endInsertRows(); @@ -81,7 +82,7 @@ int MessageModel::indexForId(MsgId id) { /**********************************************************************************/ -MessageItem::MessageItem(const Message &msg) { +MessageModelItem::MessageModelItem(const Message &msg) { _timestamp = msg.timestamp(); _msgId = msg.msgId(); _bufferId = msg.bufferInfo().bufferId(); @@ -90,12 +91,12 @@ MessageItem::MessageItem(const Message &msg) { } -MessageItem::~MessageItem() { +MessageModelItem::~MessageModelItem() { } -QVariant MessageItem::data(int column, int role) const { - if(column < TimestampColumn || column > TextColumn) return QVariant(); +QVariant MessageModelItem::data(int column, int role) const { + if(column < MessageModel::TimestampColumn || column > MessageModel::ContentsColumn) return QVariant(); switch(role) { case MessageModel::MsgIdRole: return QVariant::fromValue(_msgId); case MessageModel::BufferIdRole: return QVariant::fromValue(_bufferId);