touch-ups per review feedback

common/lib/xmodule/xmodule/course_module.py

@@ -6,7 +6,6 @@ from path import path  # NOTE (THK): Only used for detecting presence of syllabu
 import requests
 from datetime import datetime
 import dateutil.parser
-import cgi
 
 from xmodule.modulestore import Location
 from xmodule.seq_module import SequenceDescriptor, SequenceModule
@@ -945,7 +944,7 @@ class CourseDescriptor(CourseFields, SequenceDescriptor):
         Return a display course number if it has been specified, otherwise return the 'course' that is in the location
         """
         if self.display_coursenumber:
-            return cgi.escape(self.display_coursenumber)
+            return self.display_coursenumber
 
         return self.number
 
@@ -959,6 +958,6 @@ class CourseDescriptor(CourseFields, SequenceDescriptor):
         Return a display organization if it has been specified, otherwise return the 'org' that is in the location
         """
         if self.display_organization:
-            return cgi.escape(self.display_organization)
+            return self.display_organization
 
         return self.org

lms/templates/courseware/course_about.html

@@ -94,7 +94,7 @@
                     </a>
                 %endif
         %else:
-            <a href="#" class="register">${_("Register for {course.display_number_with_default}").format(course=course)}</a>
+            <a href="#" class="register">${_("Register for {course.display_number_with_default}").format(course=course) | h}</a>
 
             <div id="register_error"></div>
         %endif

lms/templates/test_center_register.html

@@ -95,7 +95,7 @@
   <section class="introduction">
     <header>
       <hgroup>
-        <h2><a href="${reverse('dashboard')}">${get_course_about_section(course, 'university')} ${course.display_number_with_default | h} ${course.display_name_with_default}</a></h2>
+        <h2><a href="${reverse('dashboard')}">${get_course_about_section(course, 'university')} ${course.display_number_with_default | h} ${course.display_name_with_default | h}</a></h2>
 
         % if registration:
         <h1>${_('Your Pearson VUE Proctored Exam Registration')}</h1>
@@ -442,7 +442,7 @@
    % endif
 
     <div class="details details-course">
-      <h4>${_("About {university} {course_number}").format(university=get_course_about_section(course, 'university'), course_number=course.course.display_number_with_default)}</h4>
+      <h4>${_("About {university} {course_number}").format(university=get_course_about_section(course, 'university'), course_number=course.course.display_number_with_default) | h}</h4>
       <p>
       % if course.has_ended():
       <span class="label">${_('Course Completed:')}</span> <span class="value">${course.end_date_text}</span>