Switch some dirty hacking to using real infrastructure. A Chatline now contains three...

[quassel.git] / src / client / messagemodel.cpp

diff --git a/src/client/messagemodel.cpp b/src/client/messagemodel.cpp
index 4e4b264..6784c4f 100644 (file)
--- a/src/client/messagemodel.cpp
+++ b/src/client/messagemodel.cpp
@@ -43,7 +43,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 +51,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 +81,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 +90,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();
   }
 }