X-Git-Url: https://git.quassel-irc.org/?p=quassel.git;a=blobdiff_plain;f=src%2Fclient%2Fmessagemodel.cpp;h=07d0ccafcbd97a25106d2cde7a7d4d7b6131a7ee;hp=5269d0ad3b30b9910a5ca0850eeb4f673d785236;hb=7bde124b28584f12341f7b9c00b0e7cbe00c3b24;hpb=46ddb293c61aeb90295fa1a14be71e8896f34ade diff --git a/src/client/messagemodel.cpp b/src/client/messagemodel.cpp index 5269d0ad..07d0ccaf 100644 --- a/src/client/messagemodel.cpp +++ b/src/client/messagemodel.cpp @@ -32,6 +32,14 @@ public: MessageModel::MessageModel(QObject *parent) : QAbstractItemModel(parent) { + QDateTime now = QDateTime::currentDateTime(); + now.setTimeSpec(Qt::UTC); + _nextDayChange.setTimeSpec(Qt::UTC); + _nextDayChange.setTime_t(((now.toTime_t() / 86400) + 1) * 86400); + _nextDayChange.setTimeSpec(Qt::LocalTime); + _dayChangeTimer.setInterval(QDateTime::currentDateTime().secsTo(_nextDayChange) * 1000); + _dayChangeTimer.start(); + connect(&_dayChangeTimer, SIGNAL(timeout()), this, SLOT(changeOfDay())); } QVariant MessageModel::data(const QModelIndex &index, int role) const { @@ -68,10 +76,7 @@ bool MessageModel::insertMessage(const Message &msg, bool fakeMsg) { return false; } - MessageModelItem *item = createMessageModelItem(msg); - beginInsertRows(QModelIndex(), idx, idx); - _messageList.insert(idx, item); - endInsertRows(); + insertMessageGroup(QList() << msg); return true; } @@ -79,43 +84,81 @@ void MessageModel::insertMessages(const QList &msglist) { if(msglist.isEmpty()) return; -// if(_messageList.isEmpty()) { -// insertMessageGroup(msglist); -// } else { - int processedMsgs = insertMessagesGracefully(msglist); - int remainingMsgs = msglist.count() - processedMsgs; - if(remainingMsgs > 0) { - if(msglist.first().msgId() < msglist.last().msgId()) { - // in Order - _messageBuffer << msglist.mid(0, remainingMsgs); - } else { - _messageBuffer << msglist.mid(processedMsgs); - } - qSort(_messageBuffer); - QCoreApplication::postEvent(this, new ProcessBufferEvent()); + int processedMsgs = insertMessagesGracefully(msglist); + int remainingMsgs = msglist.count() - processedMsgs; + if(remainingMsgs > 0) { + if(msglist.first().msgId() < msglist.last().msgId()) { + // in Order + _messageBuffer << msglist.mid(0, remainingMsgs); + } else { + _messageBuffer << msglist.mid(processedMsgs); } -// } + qSort(_messageBuffer); + QCoreApplication::postEvent(this, new ProcessBufferEvent()); + } } void MessageModel::insertMessageGroup(const QList &msglist) { - int idx = indexForId(msglist.first().msgId()); - if(idx > 0) { - int prevIdx = idx - 1; - if(_messageList[prevIdx]->msgType() == Message::DayChange - && _messageList[prevIdx]->timeStamp() > msglist.value(0).timestamp()) { + Q_ASSERT(!msglist.isEmpty()); // the msglist can be assumed to be non empty + int start = indexForId(msglist.first().msgId()); + int end = start + msglist.count() - 1; + MessageModelItem *dayChangeItem = 0; + + if(start > 0) { + // check if the preceeding msg is a daychange message and if so if + // we have to drop or relocate it at the end of this chunk + int prevIdx = start - 1; + if(_messageList.at(prevIdx)->msgType() == Message::DayChange + && _messageList.at(prevIdx)->timeStamp() > msglist.at(0).timestamp()) { beginRemoveRows(QModelIndex(), prevIdx, prevIdx); MessageModelItem *oldItem = _messageList.takeAt(prevIdx); + if(msglist.last().timestamp() < oldItem->timeStamp()) { + // we have to reinsert it (with changed msgId -> thus we need to recreate it) + Message dayChangeMsg = Message::ChangeOfDay(oldItem->timeStamp()); + dayChangeMsg.setMsgId(msglist.last().msgId()); + dayChangeItem = createMessageModelItem(dayChangeMsg); + } delete oldItem; endRemoveRows(); - idx--; + start--; + } + } + + if(!dayChangeItem && start < _messageList.count()) { + // check if we need to insert a daychange message at the end of the this group + + // if this assert triggers then indexForId() would have found a spot right before a DayChangeMsg + // this should never happen as daychange messages share the msgId with the preceeding message + Q_ASSERT(_messageList[start]->msgType() != Message::DayChange); + QDateTime nextTs = _messageList[start]->timeStamp(); + QDateTime prevTs = msglist.last().timestamp(); + nextTs.setTimeSpec(Qt::UTC); + prevTs.setTimeSpec(Qt::UTC); + uint nextDay = nextTs.toTime_t() / 86400; + uint prevDay = prevTs.toTime_t() / 86400; + if(nextDay != prevDay) { + nextTs.setTime_t(nextDay * 86400); + nextTs.setTimeSpec(Qt::LocalTime); + Message dayChangeMsg = Message::ChangeOfDay(nextTs); + dayChangeMsg.setMsgId(msglist.last().msgId()); + dayChangeItem = createMessageModelItem(dayChangeMsg); } } - beginInsertRows(QModelIndex(), idx, idx+msglist.count()-1); + + if(dayChangeItem) + end++; + + beginInsertRows(QModelIndex(), start, end); foreach(Message msg, msglist) { - _messageList.insert(idx, createMessageModelItem(msg)); - idx++; + _messageList.insert(start, createMessageModelItem(msg)); + start++; + } + if(dayChangeItem) { + _messageList.insert(start, dayChangeItem); + start++; // needed for the following assert } endInsertRows(); + Q_ASSERT(start - 1 == end); } int MessageModel::insertMessagesGracefully(const QList &msglist) { @@ -190,7 +233,20 @@ int MessageModel::insertMessagesGracefully(const QList &msglist) { if((*iter).msgId() != dupeId) { if(!grouplist.isEmpty()) { - qWarning() << "copy day change check"; + QDateTime nextTs = grouplist.value(0).timestamp(); + QDateTime prevTs = (*iter).timestamp(); + nextTs.setTimeSpec(Qt::UTC); + prevTs.setTimeSpec(Qt::UTC); + uint nextDay = nextTs.toTime_t() / 86400; + uint prevDay = prevTs.toTime_t() / 86400; + if(nextDay != prevDay) { + nextTs.setTime_t(nextDay * 86400); + nextTs.setTimeSpec(Qt::LocalTime); + Message dayChangeMsg = Message::ChangeOfDay(nextTs); + dayChangeMsg.setMsgId((*iter).msgId()); + grouplist.prepend(dayChangeMsg); + dupeCount--; + } } grouplist.prepend(*iter); } else { @@ -252,6 +308,23 @@ int MessageModel::indexForId(MsgId id) { } } +void MessageModel::changeOfDay() { + _dayChangeTimer.setInterval(86400000); + qDebug() << _nextDayChange; + if(!_messageList.isEmpty()) { + int idx = _messageList.count(); + while(idx > 0 && _messageList[idx - 1]->timeStamp() > _nextDayChange) { + idx--; + } + beginInsertRows(QModelIndex(), idx, idx); + Message dayChangeMsg = Message::ChangeOfDay(_nextDayChange); + dayChangeMsg.setMsgId(_messageList[idx - 1]->msgId()); + _messageList.insert(idx, createMessageModelItem(dayChangeMsg)); + endInsertRows(); + } + _nextDayChange = _nextDayChange.addSecs(86400); +} + /**********************************************************************************/ MessageModelItem::MessageModelItem(const Message &msg) :