From 752b3072c4854752a0e6f4827c959917b555992f Mon Sep 17 00:00:00 2001
From: Ned Batchelder <ned@edx.org>
Date: Tue, 21 May 2019 17:05:22 -0400
Subject: [PATCH] Fix CSSlint issues

---
 cms/templates/widgets/footer.html          |  9 ++++++---
 themes/red-theme/lms/templates/footer.html | 19 ++++++++++---------
 2 files changed, 16 insertions(+), 12 deletions(-)

diff --git a/cms/templates/widgets/footer.html b/cms/templates/widgets/footer.html
index 3a94777c1f..b1828017b9 100644
--- a/cms/templates/widgets/footer.html
+++ b/cms/templates/widgets/footer.html
@@ -1,3 +1,5 @@
+## mako
+<%page expression_filter="h"/>
 <%!
 from django.utils.translation import ugettext as _
 from django.urls import reverse
@@ -5,6 +7,7 @@ from datetime import datetime
 from django.conf import settings
 import pytz
 from cms.djangoapps.contentstore.config.waffle import waffle, ENABLE_ACCESSIBILITY_POLICY_PAGE
+from openedx.core.djangolib.markup import HTML, Text
 %>
 
 <div class="wrapper-footer wrapper">
@@ -44,9 +47,9 @@ from cms.djangoapps.contentstore.config.waffle import waffle, ENABLE_ACCESSIBILI
         ## Site operators: Please do not remove this paragraph! This attributes back to edX and makes your acknowledgement of edX's trademarks clear.
         <p>
           ## Translators: 'edX' and 'Open edX' are trademarks of 'edX Inc.'. Please do not translate any of these trademarks and company names.
-          ${_("edX, Open edX, and the edX and Open edX logos are registered trademarks of {link_start}edX Inc.{link_end}").format(
-            link_start=u"<a data-rel='edx.org' href='https://www.edx.org/'>",
-            link_end=u"</a>"
+          ${Text(_("edX, Open edX, and the edX and Open edX logos are registered trademarks of {link_start}edX Inc.{link_end}")).format(
+            link_start=HTML(u"<a data-rel='edx.org' href='https://www.edx.org/'>"),
+            link_end=HTML(u"</a>")
           )}
         </p>
       </div>
diff --git a/themes/red-theme/lms/templates/footer.html b/themes/red-theme/lms/templates/footer.html
index 61f53d4091..24ebd2483d 100755
--- a/themes/red-theme/lms/templates/footer.html
+++ b/themes/red-theme/lms/templates/footer.html
@@ -1,4 +1,5 @@
 ## mako
+<%page expression_filter="h"/>
 <%namespace name='static' file='static_content.html'/>
 <%!
 from django.urls import reverse
@@ -9,6 +10,7 @@ from datetime import datetime
 import pytz
 
 from openedx.core.djangoapps.lang_pref.api import footer_language_selector_is_enabled
+from openedx.core.djangolib.markup import HTML, Text
 %>
 
 <div class="wrapper wrapper-footer">
@@ -68,9 +70,9 @@ from openedx.core.djangoapps.lang_pref.api import footer_language_selector_is_en
       ## Site operators: Please do not remove this paragraph! This attributes back to edX and makes your acknowledgement of edX's trademarks clear.
       <p class="copyright">
         ## Translators: 'edX' and 'Open edX' are trademarks of 'edX Inc.'. Please do not translate any of these trademarks and company names.
-        ${_("edX, Open edX, and the edX and Open edX logos are registered trademarks of {link_start}edX Inc.{link_end}").format(
-            link_start=u"<a href='https://www.edx.org/'>",
-            link_end=u"</a>"
+        ${Text(_("edX, Open edX, and the edX and Open edX logos are registered trademarks of {link_start}edX Inc.{link_end}")).format(
+            link_start=HTML(u"<a href='https://www.edx.org/'>"),
+            link_end=HTML(u"</a>")
         )}
       </p>
       <nav class="nav-legal" aria-label="${_('Legal')}">
@@ -78,16 +80,15 @@ from openedx.core.djangoapps.lang_pref.api import footer_language_selector_is_en
         % if marketing_link('HONOR') and marketing_link('HONOR') != '#':
           <li class="nav-legal-01">
             <%
-              tos_link = u"<a href='{}'>".format(marketing_link('TOS'))
-              honor_link = u"<a href='{}'>".format(marketing_link('HONOR'))
+              tos_link = HTML(u"<a href='{}'>").format(marketing_link('TOS'))
+              honor_link = HTML(u"<a href='{}'>").format(marketing_link('HONOR'))
             %>
             ${
-              _(
-                "{tos_link_start}Terms of Service{tos_link_end} and {honor_link_start}Honor Code{honor_link_end}").format(
+              Text(_("{tos_link_start}Terms of Service{tos_link_end} and {honor_link_start}Honor Code{honor_link_end}")).format(
                 tos_link_start=tos_link,
-                tos_link_end="</a>",
+                tos_link_end=HTML("</a>"),
                 honor_link_start=honor_link,
-                honor_link_end="</a>",
+                honor_link_end=HTML("</a>"),
               )
             }
           </li>