Skip to content

Commit c4cc50c

Browse files
author
meow
committed
replace innerHTML to safer textContent where possible
1 parent 9b09d36 commit c4cc50c

File tree

8 files changed

+15
-15
lines changed

8 files changed

+15
-15
lines changed

assets/js/community.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
'use strict';
2-
var community_data = JSON.parse(document.getElementById('community_data').innerHTML);
2+
var community_data = JSON.parse(document.getElementById('community_data').textContent);
33

44
String.prototype.supplant = function (o) {
55
return this.replace(/{([^{}]*)}/g, function (a, b) {

assets/js/embed.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
'use strict';
2-
var video_data = JSON.parse(document.getElementById('video_data').innerHTML);
2+
var video_data = JSON.parse(document.getElementById('video_data').textContent);
33

44
function get_playlist(plid, retries) {
55
if (retries === undefined) retries = 5;

assets/js/notifications.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
'use strict';
2-
var notification_data = JSON.parse(document.getElementById('notification_data').innerHTML);
2+
var notification_data = JSON.parse(document.getElementById('notification_data').textContent);
33

44
var notifications, delivered;
55

assets/js/player.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
'use strict';
2-
var player_data = JSON.parse(document.getElementById('player_data').innerHTML);
3-
var video_data = JSON.parse(document.getElementById('video_data').innerHTML);
2+
var player_data = JSON.parse(document.getElementById('player_data').textContent);
3+
var video_data = JSON.parse(document.getElementById('video_data').textContent);
44

55
var options = {
66
preload: 'auto',

assets/js/playlist_widget.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
'use strict';
2-
var playlist_data = JSON.parse(document.getElementById('playlist_data').innerHTML);
2+
var playlist_data = JSON.parse(document.getElementById('playlist_data').textContent);
33

44
function add_playlist_video(target) {
55
var select = target.parentNode.children[0].children[1];

assets/js/subscribe_widget.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
'use strict';
2-
var subscribe_data = JSON.parse(document.getElementById('subscribe_data').innerHTML);
2+
var subscribe_data = JSON.parse(document.getElementById('subscribe_data').textContent);
33

44
var subscribe_button = document.getElementById('subscribe');
55
subscribe_button.parentNode['action'] = 'javascript:void(0)';

assets/js/watch.js

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
'use strict';
2-
var video_data = JSON.parse(document.getElementById('video_data').innerHTML);
2+
var video_data = JSON.parse(document.getElementById('video_data').textContent);
33

44
String.prototype.supplant = function (o) {
55
return this.replace(/{([^{}]*)}/g, function (a, b) {
@@ -11,10 +11,10 @@ String.prototype.supplant = function (o) {
1111
function toggle_parent(target) {
1212
var body = target.parentNode.parentNode.children[1];
1313
if (body.style.display === null || body.style.display === '') {
14-
target.innerHTML = '[ + ]';
14+
target.textContent = '[ + ]';
1515
body.style.display = 'none';
1616
} else {
17-
target.innerHTML = '[ - ]';
17+
target.textContent = '[ - ]';
1818
body.style.display = '';
1919
}
2020
}
@@ -23,10 +23,10 @@ function toggle_comments(event) {
2323
var target = event.target;
2424
var body = target.parentNode.parentNode.parentNode.children[1];
2525
if (body.style.display === null || body.style.display === '') {
26-
target.innerHTML = '[ + ]';
26+
target.textContent = '[ + ]';
2727
body.style.display = 'none';
2828
} else {
29-
target.innerHTML = '[ - ]';
29+
target.textContent = '[ - ]';
3030
body.style.display = '';
3131
}
3232
}
@@ -50,7 +50,7 @@ function hide_youtube_replies(event) {
5050
var body = target.parentNode.parentNode.children[1];
5151
body.style.display = 'none';
5252

53-
target.innerHTML = sub_text;
53+
target.textContent = sub_text;
5454
target.onclick = show_youtube_replies;
5555
target.setAttribute('data-inner-text', inner_text);
5656
target.setAttribute('data-sub-text', sub_text);
@@ -65,7 +65,7 @@ function show_youtube_replies(event) {
6565
var body = target.parentNode.parentNode.children[1];
6666
body.style.display = '';
6767

68-
target.innerHTML = sub_text;
68+
target.textContent = sub_text;
6969
target.onclick = hide_youtube_replies;
7070
target.setAttribute('data-inner-text', inner_text);
7171
target.setAttribute('data-sub-text', sub_text);

assets/js/watched_widget.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
'use strict';
2-
var watched_data = JSON.parse(document.getElementById('watched_data').innerHTML);
2+
var watched_data = JSON.parse(document.getElementById('watched_data').textContent);
33

44
function mark_watched(target) {
55
var tile = target.parentNode.parentNode.parentNode.parentNode.parentNode;

0 commit comments

Comments
 (0)