From 3bc47e9bc03a5093b0d16cb3a0d2740157139e3f Mon Sep 17 00:00:00 2001 From: Christian Westgaard Date: Thu, 22 Dec 2022 14:48:41 +0100 Subject: [PATCH] Edit Synonyms is broken #681 --- .../editSynonyms/useEditSynonymsState.ts | 28 +++++++------------ .../graphQL/synonym/addQuerySynonymsField.ts | 19 +++++++------ 2 files changed, 20 insertions(+), 27 deletions(-) diff --git a/src/main/resources/assets/react/thesaurus/editSynonyms/useEditSynonymsState.ts b/src/main/resources/assets/react/thesaurus/editSynonyms/useEditSynonymsState.ts index 1b28d73e..408e7415 100644 --- a/src/main/resources/assets/react/thesaurus/editSynonyms/useEditSynonymsState.ts +++ b/src/main/resources/assets/react/thesaurus/editSynonyms/useEditSynonymsState.ts @@ -34,8 +34,8 @@ export function useEditSynonymsState({ servicesBaseUrl, thesaurusName } :{ - servicesBaseUrl :string - thesaurusName ?:string + servicesBaseUrl: string + thesaurusName?: string }) { //────────────────────────────────────────────────────────────────────────── // State @@ -151,28 +151,20 @@ export function useEditSynonymsState({ //console.debug('EditSynonyms result', result); return { aggregations, - column, - direction, + column, setColumn, + direction, setDirection, end, - from, + from, setFrom, isLoading, - languages, + languages, setLanguages, memoizedQuerySynonyms, - page, - perPage, + page, setPage, + perPage, setPerPage, result: state, - setColumn, - setDirection, - setFrom, - setLanguages, - setPage, - setPerPage, - setThesauri, - setTo, sortAfterColumnClick, start, - thesauri, - to, + thesauri, setThesauri, + to, setTo, total, totalPages }; diff --git a/src/main/resources/services/graphQL/synonym/addQuerySynonymsField.ts b/src/main/resources/services/graphQL/synonym/addQuerySynonymsField.ts index 70544a4b..d19c5e87 100644 --- a/src/main/resources/services/graphQL/synonym/addQuerySynonymsField.ts +++ b/src/main/resources/services/graphQL/synonym/addQuerySynonymsField.ts @@ -8,8 +8,9 @@ import type {Glue} from '../Glue'; import { addQueryFilter, - storage//, - //toStr + arrayIncludes, + storage, + // toStr, } from '@enonic/js-utils'; import {PRINCIPAL_EXPLORER_READ} from '/lib/explorer/constants'; import {hasValue} from '/lib/explorer/query/hasValue'; @@ -109,8 +110,8 @@ export function addQuerySynonymsField({ _name: thesaurusName }); for (let i = 0; i < allowedLanguages.length; i++) { - const locale = allowedLanguages[i]; - if (!highlightLocales.includes(locale)) { + const locale = allowedLanguages[i]; + if (!arrayIncludes(highlightLocales, locale)) { highlightLocales.push(locale); } if (locale !== 'zxx' && !localeToStemmingLanguage[locale]) { @@ -125,12 +126,12 @@ export function addQuerySynonymsField({ getSynonymsCount: false }); for (let i = 0; i < hits.length; i++) { // TODO Refactor - const { + const { allowedLanguages } = hits[i]; for (let j = 0; j < allowedLanguages.length; j++) { - const locale = allowedLanguages[j]; - if (!highlightLocales.includes(locale)) { + const locale = allowedLanguages[j]; + if (!arrayIncludes(highlightLocales, locale)) { highlightLocales.push(locale); } if (locale !== 'zxx' && !localeToStemmingLanguage[locale]) { @@ -139,12 +140,12 @@ export function addQuerySynonymsField({ } } } - //log.debug('highlightLocales:%s', toStr(highlightLocales)); + // log.debug('highlightLocales:%s', toStr(highlightLocales)); //log.debug('localeToStemmingLanguage:%s', toStr(localeToStemmingLanguage)); if (languages && languages.length) { for (let i = 0; i < languages.length; i++) { - const locale = languages[i]; + const locale = languages[i]; filters = addQueryFilter({ clause: 'must', filters,