Only calculate displayed groups if flair enabled

This commit is contained in:
Luke Barnard 2017-11-28 15:49:11 +00:00
parent b32e4a57bc
commit 204bbebfca

View file

@ -105,11 +105,20 @@ export default React.createClass({
return <span />; // emote message must include the name so don't duplicate it return <span />; // emote message must include the name so don't duplicate it
} }
const displayedGroups = this._getDisplayedGroups( let flair = <div />;
this.state.userGroups, this.state.relatedGroups, if (this.props.enableFlair) {
); const displayedGroups = this._getDisplayedGroups(
this.state.userGroups, this.state.relatedGroups,
);
name = displayedGroups.length > 0 ? name.replace(' (IRC)', '') : name; // Backwards-compatible replacing of "(IRC)" with AS user flair
name = displayedGroups.length > 0 ? name.replace(' (IRC)', '') : name;
flair = <Flair key='flair'
userId={mxEvent.getSender()}
groups={displayedGroups}
/>;
}
const nameElem = <EmojiText key='name'>{ name || '' }</EmojiText>; const nameElem = <EmojiText key='name'>{ name || '' }</EmojiText>;
@ -118,13 +127,7 @@ export default React.createClass({
<span className="mx_SenderProfile_name"> <span className="mx_SenderProfile_name">
{ nameElem } { nameElem }
</span> </span>
{ this.props.enableFlair ? { flair }
<Flair key='flair'
userId={mxEvent.getSender()}
groups={displayedGroups}
/>
: null
}
</span>; </span>;
const content = this.props.text ? const content = this.props.text ?