UIQM-588 Use Authority search to get Authority linked records count.

CHANGELOG.md

@@ -10,6 +10,7 @@
 * [UIQM-594](https://issues.folio.org/browse/UIQM-594) *BREAKING* Add authority-source-files interface.
 * [UIQM-522](https://issues.folio.org/browse/UIQM-522) Create/Derive a new MARC bib record & Create a MARC holdings | Default state of Save & close button should be disabled.
 * [UIQM-534](https://issues.folio.org/browse/UIQM-534) Remove fields that have no MARC tag and no subfield value.  
+* [UIQM-588](https://issues.folio.org/browse/UIQM-588) *BREAKING* Use Authority search to get Authority linked records count.
 
 ## [7.0.4](https://github.com/folio-org/ui-quick-marc/tree/v7.0.4) (2023-11-09)
 

package.json

@@ -15,6 +15,7 @@
       "inventory": "13.0",
       "marc-records-editor": "5.0",
       "permissions": "5.6",
+      "search": "1.2",
       "marc-specifications": "1.0",
       "authority-source-files": "2.0",
       "instance-authority-links": "2.1"
@@ -126,7 +127,8 @@
           "marc-records-editor.item.get",
           "marc-records-editor.item.put",
           "inventory-storage.locations.collection.get",
-          "marc-specifications.item.get"
+          "marc-specifications.item.get",
+          "search.authorities.collection.get"
         ],
         "visible": true
       },

src/QuickMarcEditor/QuickMarcEditorContainer.js

@@ -85,7 +85,6 @@ const QuickMarcEditorContainer = ({
   const [marcRecord, setMarcRecord] = useState();
   const [locations, setLocations] = useState();
   const [isLoading, setIsLoading] = useState(true);
-  const [linksCount, setLinksCount] = useState(0);
   const [fixedFieldSpec, setFixedFieldSpec] = useState();
 
   const searchParams = new URLSearchParams(location.search);
@@ -96,7 +95,7 @@ const QuickMarcEditorContainer = ({
     && action !== QUICK_MARC_ACTIONS.CREATE;
 
   const showCallout = useShowCallout();
-  const { fetchLinksCount } = useAuthorityLinksCount();
+  const { linksCount } = useAuthorityLinksCount({ id: marcType === MARC_TYPES.AUTHORITY && externalId });
 
   const closeEditor = useCallback((id) => {
     if (marcType === MARC_TYPES.HOLDINGS && action !== QUICK_MARC_ACTIONS.CREATE) {
@@ -147,10 +146,6 @@ const QuickMarcEditorContainer = ({
     // must be with the central tenant id when user derives shared record
     const linkingRulesPromise = mutator.linkingRules.GET(headers);
 
-    const linksCountPromise = marcType === MARC_TYPES.AUTHORITY
-      ? fetchLinksCount([externalId])
-      : Promise.resolve();
-
     const fixedFieldSpecPromise = mutator.fixedFieldSpec.GET({
       path: `${MARC_SPEC_API}/${marcType}/008`,
       ...headers,
@@ -160,22 +155,16 @@ const QuickMarcEditorContainer = ({
       instancePromise,
       marcRecordPromise,
       locationsPromise,
-      linksCountPromise,
       linkingRulesPromise,
       fixedFieldSpecPromise,
     ])
       .then(([
         instanceResponse,
         marcRecordResponse,
         locationsResponse,
-        linksCountResponse,
         linkingRulesResponse,
         fixedFieldSpecResponse,
       ]) => {
-        if (marcType === MARC_TYPES.AUTHORITY) {
-          setLinksCount(linksCountResponse.links[0].totalLinks);
-        }
-
         if (action !== QUICK_MARC_ACTIONS.CREATE) {
           searchParams.set('relatedRecordVersion', instanceResponse._version);
 
@@ -211,7 +200,6 @@ const QuickMarcEditorContainer = ({
     externalId,
     history,
     marcType,
-    fetchLinksCount,
     centralTenantId,
     token,
     locale,

src/QuickMarcEditor/QuickMarcEditorContainer.test.js

@@ -24,8 +24,6 @@ import buildStripes from '../../test/jest/__mock__/stripesCore.mock';
 import { useAuthorityLinksCount } from '../queries';
 import { applyCentralTenantInHeaders } from './utils';
 
-const mockFetchLinksCount = jest.fn().mockResolvedValue();
-
 const match = {
   path: '/marc-authorities/quick-marc/edit-authority/:externalId',
   url: '/marc-authorities/quick-marc/edit-authority/external-id',
@@ -160,10 +158,6 @@ describe('Given Quick Marc Editor Container', () => {
 
   describe('when the marc type is authority', () => {
     it('should make a request to get the number of links', async () => {
-      useAuthorityLinksCount.mockClear().mockReturnValue({
-        fetchLinksCount: mockFetchLinksCount,
-      });
-
       await act(async () => {
         await renderQuickMarcEditorContainer({
           mutator,
@@ -174,7 +168,7 @@ describe('Given Quick Marc Editor Container', () => {
         });
       });
 
-      expect(mockFetchLinksCount).toHaveBeenCalledWith([match.params.externalId]);
+      expect(useAuthorityLinksCount).toHaveBeenCalledWith({ id: match.params.externalId });
     });
   });
 

src/queries/useAuthorityLinksCount/useAuthorityLinksCount.js

@@ -1,15 +1,29 @@
-import { useMutation } from 'react-query';
-import { useTenantKy } from '../../temp';
+import { useQuery } from 'react-query';
 
-const useAuthorityLinksCount = ({ tenantId } = {}) => {
-  const ky = useTenantKy({ tenantId });
+import {
+  useNamespace,
+  useOkapiKy,
+} from '@folio/stripes/core';
 
-  const { mutateAsync, isLoading } = useMutation({
-    mutationFn: ids => ky.post('links/authorities/bulk/count', { json: { ids } }).json(),
-  });
+const useAuthorityLinksCount = ({ id } = {}) => {
+  const ky = useOkapiKy();
+  const [namespace] = useNamespace();
+
+  const searchParams = {
+    query: `(id==${id} and authRefType==("Authorized"))`, // only Authorized records have links counts, so we need to get them
+  };
+
+  const { data = {}, isLoading } = useQuery(
+    [namespace, 'authority', id],
+    () => ky.get('search/authorities', { searchParams }).json(),
+    {
+      keepPreviousData: true,
+      enabled: Boolean(id),
+    },
+  );
 
   return {
-    fetchLinksCount: mutateAsync,
+    linksCount: data.authorities?.[0]?.numberOfTitles || 0,
     isLoading,
   };
 };

src/queries/useAuthorityLinksCount/useAuthorityLinksCount.test.js

@@ -4,16 +4,11 @@ import {
 } from 'react-query';
 import { renderHook } from '@folio/jest-config-stripes/testing-library/react';
 
-import '../../../test/jest/__mock__';
-
 import { useOkapiKy } from '@folio/stripes/core';
 
 import useAuthorityLinksCount from './useAuthorityLinksCount';
 
-jest.mock('react-router-dom', () => ({
-  ...jest.requireActual('react-router-dom'),
-  useLocation: jest.fn(),
-}));
+import '../../../test/jest/__mock__';
 
 const queryClient = new QueryClient();
 
@@ -23,22 +18,30 @@ const wrapper = ({ children }) => (
   </QueryClientProvider>
 );
 
-const mockPost = jest.fn().mockReturnValue({
-  json: jest.fn().mockResolvedValue({ links: [] }),
+const mockGet = jest.fn().mockReturnValue({
+  json: jest.fn().mockResolvedValue({ authorities: [{ numberOfTitles: 2 }] }),
 });
 
 describe('Given useAuthorityLinksCount', () => {
   beforeEach(() => {
     useOkapiKy.mockClear().mockReturnValue({
-      post: mockPost,
+      get: mockGet,
     });
   });
 
-  it('should fetch links count', async () => {
-    const { result } = renderHook(() => useAuthorityLinksCount(), { wrapper });
+  it('should fetch authorities', async () => {
+    renderHook(() => useAuthorityLinksCount({ id: 'fakeId' }), { wrapper });
+
+    expect(mockGet).toHaveBeenCalledWith('search/authorities', {
+      searchParams: {
+        query: '(id==fakeId and authRefType==("Authorized"))',
+      },
+    });
+  });
 
-    await result.current.fetchLinksCount(['fakeId']);
+  it('should return links count', async () => {
+    const { result } = renderHook(() => useAuthorityLinksCount({ id: 'fakeId' }), { wrapper });
 
-    expect(mockPost).toHaveBeenCalledWith('links/authorities/bulk/count', { json: { ids: ['fakeId'] } });
+    expect(result.current.linksCount).toEqual(2);
   });
 });