From 8e811fc78e90aaa60d1a1216ada1d84f0e45f90e Mon Sep 17 00:00:00 2001 From: "J. Ryan Stinnett" Date: Tue, 4 Jun 2019 15:25:54 +0100 Subject: [PATCH] Use the existing room object For some reason, we were getting the room object for every event during read receipt processing, even though it has been passed in as a prop already. --- src/components/structures/MessagePanel.js | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/src/components/structures/MessagePanel.js b/src/components/structures/MessagePanel.js index bc9795ff2f..0078a3fc21 100644 --- a/src/components/structures/MessagePanel.js +++ b/src/components/structures/MessagePanel.js @@ -52,6 +52,10 @@ module.exports = React.createClass({ // ID of an event to highlight. If undefined, no event will be highlighted. highlightedEventId: PropTypes.string, + // The room these events are all in together, if any. + // (The notification panel won't have a room here, for example.) + room: PropTypes.object, + // Should we show URL Previews showUrlPreview: PropTypes.bool, @@ -615,7 +619,7 @@ module.exports = React.createClass({ const myUserId = MatrixClientPeg.get().credentials.userId; // get list of read receipts, sorted most recent first - const room = MatrixClientPeg.get().getRoom(event.getRoomId()); + const { room } = this.props; if (!room) { return null; }