Fix SyntaxWarnings

Author: ihalaij1

course/staff_views.py

@@ -45,13 +45,12 @@ def get_common_objects(self):
             k: str(Enrollment.ENROLLMENT_STATUS[getattr(Enrollment.ENROLLMENT_STATUS, k)])
             for k in Enrollment.ENROLLMENT_STATUS.keys()
         }
-        self.enrollment_statuses_json = json.dumps(self.enrollment_statuses)
-        self.participants = json.dumps(self._get_students_with_tags())
+        self.participants = self._get_students_with_tags()
         self.internal_user_label = settings_text('INTERNAL_USER_LABEL')
         self.external_user_label = settings_text('EXTERNAL_USER_LABEL')
         self.note(
             'participants', 'tags',
-            'enrollment_statuses', 'enrollment_statuses_json',
+            'enrollment_statuses',
             'internal_user_label', 'external_user_label',
         )
 

course/templates/course/staff/participants.html

@@ -258,12 +258,15 @@ <h4 class="modal-title" id="enrollment-remove-modal-label">
 {% endif %}
 
 <script src="{% static 'js/participants.js' %}"></script>
+
+{{ participants|json_script:"participants-data" }}
+{{ enrollment_statuses|json_script:"enrollment-statuses-data" }}
 <script>
 $(function () {
-	const participants = {{ participants|escape_slashes|safe }};
+	const participants = JSON.parse(document.getElementById('participants-data').textContent);
 	const api_url = "{% url 'api:course-detail' 2 instance.id %}";
 	const is_teacher = {{ is_teacher|yesno:"true,false" }};
-	const enrollment_statuses = {{ enrollment_statuses_json|escape_slashes|safe }};
+	const enrollment_statuses = JSON.parse(document.getElementById('enrollment-statuses-data').textContent);
 	participants_list(participants, api_url, is_teacher, enrollment_statuses);
 });
 </script>

course/templatetags/course.py

@@ -46,9 +46,6 @@ def group_select(context):
         'selected': selected,
     }
 
-@register.filter
-def escape_slashes(string):
-    return str(string).replace('/', '\/') # noqa: W605
 
 @register.filter
 def parse_localization(entry):

lib/remote_page.py

@@ -226,9 +226,9 @@ def fix_relative_urls(self, is_multilingual_course):
     # pylint: disable-next=too-many-locals too-many-branches
     def _fix_relative_urls(self, url, tag_name, attr_name, is_multilingual_course):
         # Starts with "#", "//" or "https:".
-        test = re.compile('^(#|\/\/|\w+:)', re.IGNORECASE) # noqa: W605
+        test = re.compile(r'^(#|//|\w+:)', re.IGNORECASE)
         # Ends with filename extension ".html" and possibly "#anchor".
-        chapter = re.compile('.*\.html(#.+)?$', re.IGNORECASE) # noqa: W605
+        chapter = re.compile(r'.*\.html(#.+)?$', re.IGNORECASE)
         # Starts with at least one "../".
         start_dotdot_path = re.compile(r"^(../)+")
         # May end with the language suffix _en or _en/#anchor or _en#anchor.