Merge pull request #1173 from matrix-org/luke/fix-rte-forgets-me

Strip out "/me" after committing to RTE history
This commit is contained in:
David Baker 2017-07-03 15:59:07 +01:00 committed by GitHub
commit 72eed70a84

View file

@ -581,14 +581,6 @@ export default class MessageComposerInput extends React.Component {
let sendHtmlFn = this.client.sendHtmlMessage;
let sendTextFn = this.client.sendTextMessage;
if (contentText.startsWith('/me')) {
contentText = contentText.substring(4);
// bit of a hack, but the alternative would be quite complicated
if (contentHTML) contentHTML = contentHTML.replace(/\/me ?/, '');
sendHtmlFn = this.client.sendHtmlEmote;
sendTextFn = this.client.sendEmoteMessage;
}
if (this.state.isRichtextEnabled) {
this.historyManager.addItem(
contentHTML ? contentHTML : contentText,
@ -599,6 +591,14 @@ export default class MessageComposerInput extends React.Component {
this.historyManager.addItem(contentText, 'markdown');
}
if (contentText.startsWith('/me')) {
contentText = contentText.substring(4);
// bit of a hack, but the alternative would be quite complicated
if (contentHTML) contentHTML = contentHTML.replace(/\/me ?/, '');
sendHtmlFn = this.client.sendHtmlEmote;
sendTextFn = this.client.sendEmoteMessage;
}
let sendMessagePromise;
if (contentHTML) {
sendMessagePromise = sendHtmlFn.call(