X-Git-Url: https://git.quassel-irc.org/?p=quassel.git;a=blobdiff_plain;f=src%2Fclient%2Fmessagemodel.cpp;h=76104bd241a2dcd0e84c21fd8463d356a6d7b52a;hp=4e4b264f5e60c750de7744c25b7beb53fade18ef;hb=3a9668461efb05bd5b24ba5eb741a7f7f75086c9;hpb=f3fc0324c8860dff6af722dafbeb05fcb69a0c41

diff --git a/src/client/messagemodel.cpp b/src/client/messagemodel.cpp
index 4e4b264f..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;
@@ -51,8 +52,8 @@ bool MessageModel::setData(const QModelIndex &index, const QVariant &value, int
 
 void MessageModel::insertMessage(const Message &msg) {
   MsgId id = msg.msgId();
-  int idx = indexForId(id); qDebug() << "inserting at" << idx << msg.text();
-  MessageItem *item = createMessageItem(msg);
+  int idx = indexForId(id);
+  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,18 +91,18 @@ MessageItem::MessageItem(const Message &msg) {
 
 }
 
-MessageItem::~MessageItem() {
+MessageModelItem::~MessageModelItem() {
 
 }
 
-QVariant MessageItem::data(int column, int role) const {
-  if(column < TimestampRole || column > TextRole) return QVariant();
+QVariant MessageModelItem::data(int column, int role) const {
+  if(column < MessageModel::TimestampColumn || column > MessageModel::ContentsColumn) return QVariant();
   switch(role) {
-    case MsgIdRole: return _msgId;
-    case BufferIdRole: return _bufferId;
-    case TypeRole: return _type;
-    case FlagsRole: return _flags;
-    case TimestampRole: return _timestamp;
+    case MessageModel::MsgIdRole: return QVariant::fromValue<MsgId>(_msgId);
+    case MessageModel::BufferIdRole: return QVariant::fromValue<BufferId>(_bufferId);
+    case MessageModel::TypeRole: return _type;
+    case MessageModel::FlagsRole: return (int)_flags;
+    case MessageModel::TimestampRole: return _timestamp;
     default: return QVariant();
   }
 }