Improve update_query_string with enum action

This commit is contained in:
Antoine Bartuccio 2024-08-10 23:32:50 +02:00
parent b35e1a476e
commit 589119c9ee
2 changed files with 21 additions and 6 deletions

View File

@ -69,20 +69,35 @@ function getCSRFToken() {
const initialUrlParams = new URLSearchParams(window.location.search);
function update_query_string(key, value, push = true) {
/**
* @readonly
* @enum {number}
*/
const History = {
PUSH: 1,
REPLACE: 2,
};
/**
* @param {string} key
* @param {string | string[] | null} value
* @param {History} action
*/
function update_query_string(key, value, action = History.REPLACE) {
const url = new URL(window.location.href);
if (!value) {
// If the value is null, undefined or empty => delete it
url.searchParams.delete(key)
} else if (Array.isArray(value)) {
url.searchParams.delete(key)
value.forEach((v) => url.searchParams.append(key, v))
} else {
url.searchParams.set(key, value);
}
if (push){
if (action === History.PUSH) {
history.pushState(null, document.title, url.toString());
} else if (action === History.REPLACE) {
history.replaceState(null, document.title, url.toString());
}
}

View File

@ -129,7 +129,7 @@
Alpine.data("pictures", () => ({
pictures: {},
page: parseInt(initialUrlParams.get("page")) || 1,
pushstate: true, /* Used to avoid pushing a state on a back action */
pushstate: History.PUSH, /* Used to avoid pushing a state on a back action */
loading: false,
async init() {
@ -139,12 +139,12 @@
this.page === 1 ? null : this.page,
this.pushstate
);
this.pushstate = true;
this.pushstate = History.PUSH;
this.fetch_pictures();
});
window.addEventListener("popstate", () => {
this.pushstate = false;
this.pushstate = History.REPLACE;
this.page = parseInt(
new URLSearchParams(window.location.search).get("page")
) || 1;