Merge pull request #11 from kasmtech/develop

Add escaping for the JQuery selector on image friendly name
This commit is contained in:
Richard Koliser 2025-05-23 13:32:22 -04:00 committed by GitHub
commit f5f9e47b94
No known key found for this signature in database
GPG key ID: B5690EEEBB952194

View file

@ -334,15 +334,16 @@ async function pickImages(upgrade) {
// Select an individual image
function selectImage(image) {
let imageKey = image.replace(new RegExp('_', 'g'), ' ').replace('-', '.');
let safeSelectorId = $.escapeSelector(image); // Escape the ID for jQuery
if (installImages.includes(imageKey)) {
installImages = installImages.filter(e => e !== imageKey)
$('#' + image).css({
$('#' + safeSelectorId).css({
filter: 'grayscale(100%)',
background: ''
});
} else {
installImages.push(imageKey);
$('#' + image).css({
$('#' + safeSelectorId).css({
filter: '',
background: '#89cff0'
});
@ -356,7 +357,8 @@ function selectAll() {
selected = false;
for (let image of images.images) {
let imageElem = image.friendly_name.replace(new RegExp(' ', 'g'), '_').replace('.', '-');
$('#' + imageElem).css({
let safeSelectorId = $.escapeSelector(imageElem); // Escape the ID for jQuery
$('#' + safeSelectorId).css({
filter: 'grayscale(100%)',
background: ''
});
@ -365,8 +367,9 @@ function selectAll() {
selected = true;
for (let image of images.images) {
let imageElem = image.friendly_name.replace(new RegExp(' ', 'g'), '_').replace('.', '-');
let safeSelectorId = $.escapeSelector(imageElem); // Escape the ID for jQuery
installImages.push(image.friendly_name);
$('#' + imageElem).css({
$('#' + safeSelectorId).css({
filter: '',
background: '#89cff0'
});