From d7c72795235a293c25546e9e52aecdb31dfbc84f Mon Sep 17 00:00:00 2001
From: Tomasz Kolek <tomasz-kolek@o2.pl>
Date: Sun, 10 Jul 2016 22:58:46 +0200
Subject: [PATCH] Add is_status_message method to Message model.

---
 zerver/models.py | 17 ++++++++++++-----
 1 file changed, 12 insertions(+), 5 deletions(-)

diff --git a/zerver/models.py b/zerver/models.py
index ebc55a6021..bcfce5a487 100644
--- a/zerver/models.py
+++ b/zerver/models.py
@@ -815,11 +815,7 @@ class Message(ModelReprMixin, models.Model):
             domain = u"mit.edu/zephyr_mirror"
         rendered_content = bugdown.convert(content, domain, self)
 
-        # For /me syntax, JS can detect the is_me_message flag
-        # and do special rendering.
-        if content.startswith('/me ') and '\n' not in content:
-            if rendered_content.startswith('<p>') and rendered_content.endswith('</p>'):
-                self.is_me_message = True
+        self.is_me_message = Message.is_status_message(content, rendered_content)
 
         return rendered_content
 
@@ -1117,6 +1113,17 @@ class Message(ModelReprMixin, models.Model):
         # type: (text_type) -> bool
         return 'http://' in content or 'https://' in content or '/user_uploads' in content
 
+    @staticmethod
+    def is_status_message(content, rendered_content):
+        # type: (text_type, text_type) -> bool
+        """
+        Returns True if content and rendered_content are from 'me_message'
+        """
+        if content.startswith('/me ') and '\n' not in content:
+            if rendered_content.startswith('<p>') and rendered_content.endswith('</p>'):
+                return True
+        return False
+
     def update_calculated_fields(self):
         # type: () -> None
         # TODO: rendered_content could also be considered a calculated field