From b15f47425d89a2c30395e5fd380f022de064ec10 Mon Sep 17 00:00:00 2001
From: Syed Ali Abbas Zaidi <syed.ali.abbas@arbisoft.com>
Date: Sat, 6 May 2023 00:37:18 +0500
Subject: [PATCH] fix: failing xss-lint issues

---
 cms/static/js/views/baseview.js                             | 3 +--
 common/static/js/capa/schematicinput.js                     | 2 +-
 .../js/certificates/views/certificate_bulk_allowlist.js     | 6 +++---
 3 files changed, 5 insertions(+), 6 deletions(-)

diff --git a/cms/static/js/views/baseview.js b/cms/static/js/views/baseview.js
index 266264bb6c8d..3893269e6fba 100644
--- a/cms/static/js/views/baseview.js
+++ b/cms/static/js/views/baseview.js
@@ -46,8 +46,7 @@ function($, _, Backbone, gettext, IframeUtils, TemplateUtils, ViewUtils) {
 
             var _this = this;
             // xss-lint: disable=javascript-jquery-insertion
-            // eslint-disable-next-line no-shadow
-            this.render = _.wrap(this.render, function(render, options) {
+            this.render = _.wrap(this.render, function(render, options) { // eslint-disable-line no-shadow
                 _this.beforeRender();
                 render(options);
                 _this.afterRender();
diff --git a/common/static/js/capa/schematicinput.js b/common/static/js/capa/schematicinput.js
index 6940ae704c00..71c0f8fd15d2 100644
--- a/common/static/js/capa/schematicinput.js
+++ b/common/static/js/capa/schematicinput.js
@@ -2,8 +2,8 @@ $(function() {
     // TODO: someone should fix all of this...
     // $("a[rel*=leanModal]").leanModal(); //TODO: Make this work with the new modal library. Try and integrate this with the "slices"
 
+    /* eslint-disable no-multi-str */
     // xss-lint: disable=javascript-jquery-append
-    // eslint-disable-next-line no-multi-str
     $('body').append('\
     <div id="circuit_editor_modal" class="modal hide fade"> \
       <div class="modal-body"> \
diff --git a/lms/static/js/certificates/views/certificate_bulk_allowlist.js b/lms/static/js/certificates/views/certificate_bulk_allowlist.js
index ab934878419b..702f66816657 100644
--- a/lms/static/js/certificates/views/certificate_bulk_allowlist.js
+++ b/lms/static/js/certificates/views/certificate_bulk_allowlist.js
@@ -82,8 +82,8 @@
                     // inner function generate div and display response messages.
                     $('<div/>', {
                         class: 'message ' + group
-                    }).appendTo('.bulk-exception-results').prepend( // eslint-disable-line max-len, xss-lint: disable=javascript-jquery-insert-into-target,javascript-jquery-prepend
-                        "<button type='button' id= '" + group + "' class='arrow'> + </button>" + heading) // eslint-disable-line max-len, xss-lint: disable=javascript-concat-html
+                    }).appendTo('.bulk-exception-results').prepend( // xss-lint: disable=javascript-jquery-insert-into-target,javascript-jquery-prepend
+                        "<button type='button' id= '" + group + "' class='arrow'> + </button>" + heading) // xss-lint: disable=javascript-concat-html
                         .append($('<ul/>', {
                             class: group
                         }));
@@ -91,7 +91,7 @@
                     for (var i = 0; i < displayData.length; i++) { // eslint-disable-line vars-on-top
                         $('<li/>', {
                             text: displayData[i]
-                        }).appendTo('div.message > .' + group); // eslint-disable-line max-len, xss-lint: disable=javascript-jquery-insert-into-target
+                        }).appendTo('div.message > .' + group); // xss-lint: disable=javascript-jquery-insert-into-target
                     }
                     $('div.message > .' + group).hide();
                 }