Skip to content

Commit 86140c4

Browse files
Fix failure in PanoramaWebPublicSearchTest due to invalid URL parameters (#596)
1 parent 7cb476d commit 86140c4

1 file changed

Lines changed: 9 additions & 4 deletions

File tree

panoramapublic/src/org/labkey/panoramapublic/view/search/panoramaWebSearch.jsp

Lines changed: 9 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -361,21 +361,26 @@
361361
let instrument = document.getElementById(instrumentItemId).value;
362362
363363
let expSearchParams = "";
364+
let separator = "";
364365
if (author) {
365366
expAnnotationFilters.push(createFilter(authorsItemId, author));
366-
expSearchParams += "Targeted MS Experiment List." + "authors~containsoneof" + "=" + encodeURIComponent(author) + "&";
367+
expSearchParams += separator + "Targeted MS Experiment List." + "authors~containsoneof=" + encodeURIComponent(author);
368+
separator = "&";
367369
}
368370
if (title) {
369371
expAnnotationFilters.push(createFilter(titleItemId, title));
370-
expSearchParams += "Targeted MS Experiment List." + "title~containsoneof" + "=" + encodeURIComponent(title) + "&";
372+
expSearchParams += separator + "Targeted MS Experiment List." + "title~containsoneof=" + encodeURIComponent(title);
373+
separator = "&";
371374
}
372375
if (organism) {
373376
expAnnotationFilters.push(createFilter(organismItemId, organism));
374-
expSearchParams += "Targeted MS Experiment List." + "organism~containsoneof" + "=" + encodeURIComponent(organism.replaceAll(",", ";")) + "&";
377+
expSearchParams += separator + "Targeted MS Experiment List." + "organism~containsoneof=" + encodeURIComponent(organism.replaceAll(",", ";"));
378+
separator = "&";
375379
}
376380
if (instrument) {
377381
expAnnotationFilters.push(createFilter(instrumentItemId, instrument));
378-
expSearchParams += "Targeted MS Experiment List." + "instrument~containsoneof" + "=" + encodeURIComponent(instrument.replaceAll(",", ";"));
382+
expSearchParams += separator + "Targeted MS Experiment List." + "instrument~containsoneof" + "=" + encodeURIComponent(instrument.replaceAll(",", ";"));
383+
separator = "&";
379384
}
380385
if (expSearchParams !== "") {
381386
location.replace(window.location.href + "?" + expSearchParams);

0 commit comments

Comments
 (0)