changed tag url seperator to semikolon to avoid conflicts with tags that contain commas

This commit is contained in:
Anton Tranelis 2024-07-30 21:26:29 +02:00
parent e3a1b4dc54
commit e229b272ca
2 changed files with 14 additions and 11 deletions

View File

@ -24,10 +24,12 @@ const filterTags = useFilterTags()
useEffect(() => { useEffect(() => {
let params = new URLSearchParams(location.search); let params = new URLSearchParams(location.search);
let urlTags = params.get("tags")?.split(","); let urlTags = params.get("tags")
if(urlTags?.some(ut => !filterTags.find(ft => ut.toLocaleLowerCase() === ft.name.toLocaleLowerCase()))||filterTags?.some(ft => !urlTags?.find(ut => ut.toLocaleLowerCase() === ft.name.toLocaleLowerCase()))) let decodedTags = urlTags ? decodeURIComponent(urlTags) : "";
let decodedTagsArray = decodedTags.split(";");
if(decodedTagsArray?.some(ut => !filterTags.find(ft => ut.toLocaleLowerCase() === ft.name.toLocaleLowerCase()))||filterTags?.some(ft => !decodedTagsArray?.find(ut => ut.toLocaleLowerCase() === ft.name.toLocaleLowerCase())))
{resetFilterTags() {resetFilterTags()
urlTags?.map(urlTag => { decodedTagsArray?.map(urlTag => {
const tag = tags.find(t => t.name.toLocaleLowerCase() === urlTag.toLocaleLowerCase()) const tag = tags.find(t => t.name.toLocaleLowerCase() === urlTag.toLocaleLowerCase())
tag && addFilterTag(tag) tag && addFilterTag(tag)
});} });}

View File

@ -127,12 +127,13 @@ function useFilterManager(initialTags: Tag[]): {
const [searchPhrase, searchPhraseSet] = React.useState<string>(""); const [searchPhrase, searchPhraseSet] = React.useState<string>("");
const addFilterTag = (tag: Tag) => { const addFilterTag = useCallback((tag: Tag) => {
let params = new URLSearchParams(location.search);
let urlTags = params.get("tags")
let decodedTags = urlTags ? decodeURIComponent(urlTags) : "";
let params = new URLSearchParams(window.location.search); if(!decodedTags?.includes(tag.name))
let urlTags = params.get("tags"); params.set("tags", `${urlTags ? urlTags : ""}${urlTags? ';' : ''}${tag.name}`)
if(!urlTags?.includes(tag.name))
params.set("tags", `${urlTags ? urlTags : ""}${urlTags? ',' : ''}${tag.name}`)
if(windowDimensions.width < 786 && location.pathname.split("/").length > 2) navigate("/" + `${params ? `?${params}` : ""}`); if(windowDimensions.width < 786 && location.pathname.split("/").length > 2) navigate("/" + `${params ? `?${params}` : ""}`);
else navigate(location.pathname + `${params ? `?${params}` : ""}`); else navigate(location.pathname + `${params ? `?${params}` : ""}`);
@ -144,14 +145,14 @@ function useFilterManager(initialTags: Tag[]): {
tag, tag,
}); });
}; }, []);
const removeFilterTag = useCallback((name: string) => { const removeFilterTag = useCallback((name: string) => {
let params = new URLSearchParams(window.location.search); let params = new URLSearchParams(window.location.search);
let urlTags = params.get("tags"); let urlTags = params.get("tags");
let newUrlTags = ""; let newUrlTags = "";
let tags = urlTags?.split(","); let tags = urlTags?.split(";");
if(tags?.length==0 && urlTags?.length && urlTags?.length > 0) tags[0]=urlTags; if(tags?.length==0 && urlTags?.length && urlTags?.length > 0) tags[0]=urlTags;
tags?.map(urlTag => { tags?.map(urlTag => {
if(!(urlTag.toLocaleLowerCase() === name.toLocaleLowerCase())) if(!(urlTag.toLocaleLowerCase() === name.toLocaleLowerCase()))