dbroqua/MusicTopus
dbroqua/MusicTopus
1
1
Fork 0
Code Issues 9 Pull requests Releases 14 Activity

{BUGFIX} Espace double quote on artists

Browse source
This commit is contained in:
Damien Broqua 2024-10-07 20:23:37 +02:00
parent 8289ed7116
commit 48287f2bae
2 changed files with 6 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
javascripts/collection.js
View file

@ -43,7 +43,10 @@ Vue.createApp({
}, },
methods: { methods: {
formatParams(param) { formatParams(param) {
return param.replace("&", "%26").replace("+", "%2B"); return param
.replace("&", "%26")
.replace("+", "%2B")
.replace('"', "%22");
}, },
fetch() { fetch() {
this.loading = true; this.loading = true;
@ -63,7 +66,7 @@ Vue.createApp({
const [key, value] = entry; const [key, value] = entry;
switch (key) { switch (key) {
case "artists_sort": case "artists_sort":
this.artist = value; this.artist = value.replaceAll('"', "%22");
break; break;
default: default:
if (["order", "sort"].indexOf(key) !== -1) { if (["order", "sort"].indexOf(key) !== -1) {

2
views/components/filters/artist.ejs
View file

@ -4,7 +4,7 @@
<option value="">Tous</option> <option value="">Tous</option>
<% <%
for (let i = 0; i < page.artists.length; i += 1 ) { for (let i = 0; i < page.artists.length; i += 1 ) {
__append(`<option value="${page.artists[i]}">${page.artists[i]}</option>`); __append(`<option value="${page.artists[i].replaceAll('"','%22')}">${page.artists[i]}</option>`);
} }
%> %>
</select> </select>