matrix/matrix-js-sdk
1
0
Fork 0
mirror of https://github.com/matrix-org/matrix-js-sdk.git synced 2025-11-29 16:43:09 +03:00
Code Activity

only call setEventMetadata on unfiltered timelineSets

Browse Source
This commit is contained in:
Matthew Hodgson
2016-09-09 17:42:24 +01:00
parent bd32ed5598
commit bd9e3e5794
1 changed files with 22 additions and 16 deletions
Download Patch File Download Diff File Expand all files Collapse all files

38
lib/models/event-timeline.js
View File

@@ -234,23 +234,29 @@ EventTimeline.prototype.setNeighbouringTimeline = function(neighbour, direction)
EventTimeline.prototype.addEvent = function(event, atStart) { EventTimeline.prototype.addEvent = function(event, atStart) {
var stateContext = atStart ? this._startState : this._endState; var stateContext = atStart ? this._startState : this._endState;
EventTimeline.setEventMetadata(event, stateContext, atStart); // only call setEventMetadata on the unfiltered timelineSets
var timelineSet = this.getTimelineSet();
if (timelineSet.room &&
timelineSet.room.getUnfilteredTimelineSet() === timelineSet)
{
EventTimeline.setEventMetadata(event, stateContext, atStart);
// modify state // modify state
if (event.isState()) { if (event.isState()) {
stateContext.setStateEvents([event]); stateContext.setStateEvents([event]);
// it is possible that the act of setting the state event means we // it is possible that the act of setting the state event means we
// can set more metadata (specifically sender/target props), so try // can set more metadata (specifically sender/target props), so try
// it again if the prop wasn't previously set. It may also mean that // it again if the prop wasn't previously set. It may also mean that
// the sender/target is updated (if the event set was a room member event) // the sender/target is updated (if the event set was a room member event)
// so we want to use the *updated* member (new avatar/name) instead. // so we want to use the *updated* member (new avatar/name) instead.
// //
// However, we do NOT want to do this on member events if we're going // However, we do NOT want to do this on member events if we're going
// back in time, else we'll set the .sender value for BEFORE the given // back in time, else we'll set the .sender value for BEFORE the given
// member event, whereas we want to set the .sender value for the ACTUAL // member event, whereas we want to set the .sender value for the ACTUAL
// member event itself. // member event itself.
if (!event.sender || (event.getType() === "m.room.member" && !atStart)) { if (!event.sender || (event.getType() === "m.room.member" && !atStart)) {
EventTimeline.setEventMetadata(event, stateContext, atStart); EventTimeline.setEventMetadata(event, stateContext, atStart);
}
} }
} }