From 6b71f5bd5fb43ccf2fbd2d91018b3d102e3d2a4b Mon Sep 17 00:00:00 2001
From: Steve Howell <showell30@yahoo.com>
Date: Tue, 4 Oct 2016 07:49:16 -0700
Subject: [PATCH] Inline most calls to set_rendered_content().

This is part of breaking the circular dependency on
bugdown in models.py.  A subsequent commit will fully
kill off set_rendered_content().
---
 zerver/lib/actions.py | 11 ++++++++---
 zerver/lib/message.py |  4 +++-
 2 files changed, 11 insertions(+), 4 deletions(-)

diff --git a/zerver/lib/actions.py b/zerver/lib/actions.py
index 637289416b..5d47fb3abd 100644
--- a/zerver/lib/actions.py
+++ b/zerver/lib/actions.py
@@ -9,7 +9,10 @@ from django.utils.translation import ugettext as _
 from django.conf import settings
 from django.core import validators
 from django.contrib.sessions.models import Session
-from zerver.lib.bugdown import BugdownRenderingException
+from zerver.lib.bugdown import (
+    BugdownRenderingException,
+    version as bugdown_version
+)
 from zerver.lib.cache import (
     to_dict_cache_key,
     to_dict_cache_key_id,
@@ -714,7 +717,8 @@ def do_send_messages(messages):
             message['message'],
             message['message'].content,
             message_users=message['active_recipients'])
-        message['message'].set_rendered_content(rendered_content)
+        message['message'].rendered_content = rendered_content
+        message['message'].rendered_content_version = bugdown_version
 
     for message in messages:
         message['message'].update_calculated_fields()
@@ -2498,7 +2502,8 @@ def do_update_message(user_profile, message, subject, propagate_mode, content, r
         edit_history_event["prev_rendered_content"] = message.rendered_content
         edit_history_event["prev_rendered_content_version"] = message.rendered_content_version
         message.content = content
-        message.set_rendered_content(rendered_content)
+        message.rendered_content = rendered_content
+        message.rendered_content_version = bugdown_version
         event["content"] = content
         event["rendered_content"] = rendered_content
 
diff --git a/zerver/lib/message.py b/zerver/lib/message.py
index 44d266996a..01c034040f 100644
--- a/zerver/lib/message.py
+++ b/zerver/lib/message.py
@@ -197,7 +197,9 @@ class MessageDict(object):
                 # It's unfortunate that we need to have side effects on the message
                 # in some cases.
                 rendered_content = message.render_markdown(content, sender_realm_domain)
-                message.set_rendered_content(rendered_content, True)
+                message.rendered_content = rendered_content
+                message.rendered_content_version = bugdown.version
+                message.save_rendered_content()
 
             if rendered_content is not None:
                 obj['content'] = rendered_content