use pipe delimiter for language filter on admin and public opportunit…

…y pages (#740)
didx-xyz · Apr 18, 2024 · 5a2ddaf · 5a2ddaf
1 parent da0877e
commit 5a2ddaf
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 6 deletions.
diff --git a/src/web/src/pages/admin/opportunities/[[...query]]/index.tsx b/src/web/src/pages/admin/opportunities/[[...query]]/index.tsx
@@ -250,7 +250,7 @@ const OpportunitiesAdmin: NextPageWithLayout<{
             languages != undefined
               ? languages
                   ?.toString()
-                  .split(",")
+                  .split("|") // use | delimiter as some languages contain ',' e.g (Catalan, Valencian)
                   .map((x) => {
                     const item = lookups_languages?.find((y) => y.name === x);
                     return item ? item?.id : "";
@@ -331,7 +331,7 @@ const OpportunitiesAdmin: NextPageWithLayout<{
             ? countries?.toString().split(",")
             : null,
         languages:
-          languages != undefined ? languages?.toString().split(",") : null,
+          languages != undefined ? languages?.toString().split("|") : null, // use | delimiter as some languages contain ',' e.g (Catalan, Valencian)
         organizations:
           organizations != undefined
             ? organizations?.toString().split(",")
@@ -467,7 +467,7 @@ const OpportunitiesAdmin: NextPageWithLayout<{
         opportunitySearchFilter?.languages?.length !== undefined &&
         opportunitySearchFilter.languages.length > 0
       )
-        params.append("languages", opportunitySearchFilter.languages.join(","));
+        params.append("languages", opportunitySearchFilter.languages.join("|")); // use | delimiter as some languages contain ',' e.g (Catalan, Valencian)
       if (
         opportunitySearchFilter?.types?.length !== undefined &&
         opportunitySearchFilter.types.length > 0

diff --git a/src/web/src/pages/opportunities/[[...query]]/index.tsx b/src/web/src/pages/opportunities/[[...query]]/index.tsx
@@ -330,7 +330,7 @@ const Opportunities: NextPageWithLayout<{
             languages != undefined
               ? languages
                   ?.toString()
-                  .split(",")
+                  .split("|") // use | delimiter as some languages contain ',' e.g (Catalan, Valencian)
                   .map((x) => {
                     const item = lookups_languages.find((y) => y.name === x);
                     return item ? item?.id : "";
@@ -398,7 +398,7 @@ const Opportunities: NextPageWithLayout<{
             ? countries?.toString().split(",")
             : null,
         languages:
-          languages != undefined ? languages?.toString().split(",") : null,
+          languages != undefined ? languages?.toString().split("|") : null, // use | delimiter as some languages contain ',' e.g (Catalan, Valencian)
         organizations:
           organizations != undefined
             ? organizations?.toString().split(",")
@@ -490,7 +490,7 @@ const Opportunities: NextPageWithLayout<{
         opportunitySearchFilter?.languages?.length !== undefined &&
         opportunitySearchFilter.languages.length > 0
       )
-        params.append("languages", opportunitySearchFilter.languages.join(","));
+        params.append("languages", opportunitySearchFilter.languages.join("|")); // use | delimiter as some languages contain ',' e.g (Catalan, Valencian)
       if (
         opportunitySearchFilter?.types?.length !== undefined &&
         opportunitySearchFilter.types.length > 0