Null check node before we pass it to velocity

As presumably sometimes the node wasn't actually in the DOM for
whatever reason, so don't propagate exceptions out into the app.

Should fix https://github.com/vector-im/riot-web/issues/6515
This commit is contained in:
David Baker 2018-04-13 10:44:37 +01:00
parent 7e977d0016
commit ce74099698

View file

@ -147,7 +147,7 @@ module.exports = React.createClass({
// creating/destroying large numbers of elements"
// (https://github.com/julianshapiro/velocity/issues/47)
const domNode = ReactDom.findDOMNode(this.nodes[k]);
Velocity.Utilities.removeData(domNode);
if (domNode) Velocity.Utilities.removeData(domNode);
}
this.nodes[k] = node;
},