From 10b7c13652cdb448f7ffcff86de1c5d3a2d83de0 Mon Sep 17 00:00:00 2001
From: Chris Dodge <cdodge@edx.org>
Date: Fri, 6 Sep 2013 16:05:41 -0400
Subject: [PATCH] add comment

---
 cms/static/js/views/settings/main_settings_view.js | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/cms/static/js/views/settings/main_settings_view.js b/cms/static/js/views/settings/main_settings_view.js
index 409c8b1c84..07289667d6 100644
--- a/cms/static/js/views/settings/main_settings_view.js
+++ b/cms/static/js/views/settings/main_settings_view.js
@@ -188,6 +188,11 @@ CMS.Views.Settings.Details = CMS.Views.ValidatingView.extend({
             thisTarget = e.currentTarget;
         } else
         {
+            // e and forcedTarget can be null so don't deference it
+            // This is because in cases where we have a marketing site
+            // we don't display the codeMirrors for editing the marketing
+            // materials, except we do need to show the 'set course image'
+            // workflow. So in this case e = forcedTarget = null.
             return;
         }