diff options
author | Magnus Hagander | 2020-12-08 15:32:04 +0000 |
---|---|---|
committer | Magnus Hagander | 2020-12-08 21:17:47 +0000 |
commit | b9d9db88fffad60798397b006445e505f6b5e447 (patch) | |
tree | 4600400d194202bfd80a38053544f35f98db85f9 | |
parent | 6a118a0164c2abdd4daf3a086e8238a7fea1eed4 (diff) |
Properly escape list of messages when creating a new entry
This has been broken since day 1... oops.
Noted by Vik Fearing
-rw-r--r-- | pgcommitfest/commitfest/static/commitfest/js/commitfest.js | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/pgcommitfest/commitfest/static/commitfest/js/commitfest.js b/pgcommitfest/commitfest/static/commitfest/js/commitfest.js index 69c41ec..2f1edda 100644 --- a/pgcommitfest/commitfest/static/commitfest/js/commitfest.js +++ b/pgcommitfest/commitfest/static/commitfest/js/commitfest.js @@ -20,7 +20,7 @@ function findLatestThreads() { sel = $('#attachThreadList'); sel.find('option').remove(); $.each(data, function(m,i) { - sel.append('<option value="' + i.msgid + '">' + i.from + ': ' + i.subj + ' (' + i.date + ')</option>'); + sel.append($('<option/>').text(i.from + ': ' + i.subj + ' (' + i.date + ')').val(i.msgid)); }); }).always(function() { $('#attachThreadListWrap').removeClass('loading'); |