From 2606ea959673600fea94df0f169467ef325f979a Mon Sep 17 00:00:00 2001
From: Aviral Dasgupta <me@aviraldg.com>
Date: Tue, 14 Jun 2016 19:10:35 +0530
Subject: [PATCH] fixes and improvements in RichText

---
 src/RichText.js | 12 +++++-------
 1 file changed, 5 insertions(+), 7 deletions(-)

diff --git a/src/RichText.js b/src/RichText.js
index c0d80d2ec7..ba7a5f219e 100644
--- a/src/RichText.js
+++ b/src/RichText.js
@@ -38,7 +38,7 @@ export function contentStateToHTML(contentState: ContentState): string {
             () => true, // always return true => don't filter any ranges out
             (start, end) => {
                 // map style names to elements
-                let tags = block.getInlineStyleAt(start).map(style => STYLES[style]);
+                let tags = block.getInlineStyleAt(start).map(style => STYLES[style]).filter(style => !!style);
                 // combine them to get well-nested HTML
                 let open = tags.map(tag => `<${tag}>`).join('');
                 let close = tags.map(tag => `</${tag}>`).reverse().join('');
@@ -67,10 +67,8 @@ export function getScopedRTDecorators(scope: any): CompositeDecorator {
         },
         component: (props) => {
             let member = scope.room.getMember(props.children[0].props.text);
-            let name = null;
-            if (!!member) {
-                name = member.name; // unused until we make these decorators immutable (autocomplete needed)
-            }
+            // unused until we make these decorators immutable (autocomplete needed)
+            let name = member ? member.name : null;
             let avatar = member ? <MemberAvatar member={member} width={16} height={16}/> : null;
             return <span className="mx_UserPill">{avatar} {props.children}</span>;
         }
@@ -131,7 +129,7 @@ function findWithRegex(regex, contentBlock: ContentBlock, callback: (start: numb
  * Passes rangeToReplace to modifyFn and replaces it in contentState with the result.
  */
 export function modifyText(contentState: ContentState, rangeToReplace: SelectionState,
-                           modifyFn: (text: string) => string, ...rest): ContentState {
+                           modifyFn: (text: string) => string, inlineStyle, entityKey): ContentState {
     let getText = (key) => contentState.getBlockForKey(key).getText(),
         startKey = rangeToReplace.getStartKey(),
         startOffset = rangeToReplace.getStartOffset(),
@@ -152,5 +150,5 @@ export function modifyText(contentState: ContentState, rangeToReplace: Selection
     // add remaining part of last block
     text += getText(endKey).substring(startOffset, endOffset);
 
-    return Modifier.replaceText(contentState, rangeToReplace, modifyFn(text), ...rest);
+    return Modifier.replaceText(contentState, rangeToReplace, modifyFn(text), inlineStyle, entityKey);
 }