From 15445939dd4b2de85898de068e80e00ad453f3f4 Mon Sep 17 00:00:00 2001
From: Oleksandr Hladchenko1 <oleksandr_hladchenko1@epam.com>
Date: Fri, 3 Nov 2023 15:40:26 +0200
Subject: [PATCH 1/4] UIIN-2647: Show only local authorities when share local
 instance

---
 src/ViewInstance.js      | 56 ++++++++++++++++++++--------------------
 src/ViewInstance.test.js | 18 ++++++++++---
 src/utils.js             | 14 ++++++++++
 3 files changed, 56 insertions(+), 32 deletions(-)

diff --git a/src/ViewInstance.js b/src/ViewInstance.js
index f687c918a..1c471f9f0 100644
--- a/src/ViewInstance.js
+++ b/src/ViewInstance.js
@@ -9,7 +9,6 @@ import {
 import {
   flowRight,
   isEmpty,
-  pick,
 } from 'lodash';
 
 import {
@@ -38,9 +37,10 @@ import {
   handleKeyCommand,
   isInstanceShadowCopy,
   isMARCSource,
+  getLinkedAuthorityIds,
 } from './utils';
 import {
-  AUTHORITY_LINKED_FIELDS,
+  CONSORTIUM_PREFIX,
   indentifierTypeNames,
   INSTANCE_SHARING_STATUSES,
   layers,
@@ -167,6 +167,12 @@ class ViewInstance extends React.Component {
       accumulate: true,
       throwErrors: false,
     },
+    authorities: {
+      type: 'okapi',
+      path: 'authority-storage/authorities',
+      accumulate: true,
+      throwErrors: false,
+    }
   });
 
   constructor(props) {
@@ -519,34 +525,25 @@ class ViewInstance extends React.Component {
   }
 
   checkIfHasLinkedAuthorities = (instance = {}) => {
-    const linkedAuthorities = pick(this.props.selectedInstance, AUTHORITY_LINKED_FIELDS);
-
-    const findLinkedAuthorities = authorities => {
-      const linkedAuthoritiesLength = AUTHORITY_LINKED_FIELDS.reduce((total, field) => {
-        const authoritiesAmount = authorities[field].filter(item => item.authorityId).length;
-        return total + authoritiesAmount;
-      }, 0);
+    const authorityIds = getLinkedAuthorityIds(instance).join(' or ');
 
-      return {
-        hasLinkedAuthorities: !!linkedAuthoritiesLength,
-        linkedAuthoritiesLength,
-      };
-    };
-
-    const {
-      hasLinkedAuthorities,
-      linkedAuthoritiesLength,
-    } = findLinkedAuthorities(linkedAuthorities);
+    this.props.mutator.authorities.GET({
+      params: {
+        query: `id==${authorityIds}`,
+      }
+    }).then(({ authorities }) => {
+      const localAuthorities = authorities.filter(authority => !authority.source.startsWith(CONSORTIUM_PREFIX));
 
-    if (hasLinkedAuthorities) {
-      this.setState({
-        linkedAuthoritiesLength,
-        isShareLocalInstanceModalOpen: false,
-        isUnlinkAuthoritiesModalOpen: true,
-      });
-    } else {
-      this.handleShareLocalInstance(instance);
-    }
+      if (localAuthorities.length) {
+        this.setState({
+          linkedAuthoritiesLength: localAuthorities.length,
+          isShareLocalInstanceModalOpen: false,
+          isUnlinkAuthoritiesModalOpen: true,
+        });
+      } else {
+        this.handleShareLocalInstance(instance);
+      }
+    });
   }
 
   toggleCopyrightModal = () => {
@@ -1066,6 +1063,9 @@ ViewInstance.propTypes = {
       POST: PropTypes.func.isRequired,
       GET: PropTypes.func.isRequired,
     }).isRequired,
+    authorities: PropTypes.shape({
+      GET: PropTypes.func.isRequired,
+    }).isRequired,
   }),
   onClose: PropTypes.func,
   onCopy: PropTypes.func,
diff --git a/src/ViewInstance.test.js b/src/ViewInstance.test.js
index 188eb8000..8e23c34ca 100644
--- a/src/ViewInstance.test.js
+++ b/src/ViewInstance.test.js
@@ -107,9 +107,10 @@ ConfirmationModal.mockImplementation(({
   open,
   onCancel,
   onConfirm,
+  heading,
 }) => (open ? (
   <div>
-    <span>Confirmation modal</span>
+    <span>{heading}</span>
     <button
       type="button"
       onClick={onCancel}
@@ -190,6 +191,9 @@ const defaultProp = {
       POST: jest.fn(),
       GET: jest.fn(() => Promise.resolve({ sharingInstances: [{ status: 'COMPLETE' }] })),
     },
+    authorities: {
+      GET: jest.fn(),
+    }
   },
   onClose: mockonClose,
   onCopy: jest.fn(),
@@ -593,7 +597,7 @@ describe('ViewInstance', () => {
             fireEvent.click(screen.getByRole('button', { name: 'Actions' }));
             fireEvent.click(screen.getByRole('button', { name: 'Share local instance' }));
 
-            expect(screen.getByText('Confirmation modal')).toBeInTheDocument();
+            expect(screen.getByText('Are you sure you want to share this instance?')).toBeInTheDocument();
           });
         });
 
@@ -620,14 +624,20 @@ describe('ViewInstance', () => {
               fireEvent.click(screen.getByRole('button', { name: 'Share local instance' }));
               fireEvent.click(screen.getByRole('button', { name: 'Confirm' }));
 
-              expect(screen.getByText('Confirmation modal')).toBeInTheDocument();
+              expect(screen.getByText('Are you sure you want to share this instance?')).toBeInTheDocument();
             });
 
             describe('when proceed sharing', () => {
-              it('should make POST request to share local instance', () => {
+              it('should make POST request to share local instance', async () => {
+                defaultProp.mutator.authorities.GET.mockResolvedValueOnce({ authorities: [{ source: 'MARC' }] });
+
                 fireEvent.click(screen.getByRole('button', { name: 'Actions' }));
                 fireEvent.click(screen.getByRole('button', { name: 'Share local instance' }));
                 fireEvent.click(screen.getByRole('button', { name: 'Confirm' }));
+
+                const localAuthoritiesModal = await screen.findByText('Linked to local authorities');
+                expect(localAuthoritiesModal).toBeInTheDocument();
+
                 fireEvent.click(screen.getByRole('button', { name: 'Confirm' }));
 
                 expect(defaultProp.mutator.shareInstance.POST).toHaveBeenCalled();
diff --git a/src/utils.js b/src/utils.js
index 88c1a9eb4..30eba4b9b 100644
--- a/src/utils.js
+++ b/src/utils.js
@@ -18,6 +18,7 @@ import {
   omit,
   chunk,
   flatten,
+  pick,
 } from 'lodash';
 import moment from 'moment';
 
@@ -41,6 +42,7 @@ import {
   OKAPI_TENANT_HEADER,
   CONTENT_TYPE_HEADER,
   OKAPI_TOKEN_HEADER,
+  AUTHORITY_LINKED_FIELDS,
 } from './constants';
 
 export const areAllFieldsEmpty = fields => fields.every(item => (isArray(item)
@@ -850,3 +852,15 @@ export const switchAffiliation = async (stripes, tenantId, move) => {
     move();
   }
 };
+
+export const getLinkedAuthorityIds = instance => {
+  // Pick fields with authorityId
+  const linkedAuthorities = pick(instance, AUTHORITY_LINKED_FIELDS);
+
+  // Retrieve only authorityId
+  const authorityIdList = AUTHORITY_LINKED_FIELDS.map(field => {
+    return linkedAuthorities[field].filter(auth => !!auth.authorityId).map(auth => auth.authorityId);
+  });
+
+  return flatten(authorityIdList);
+};

From 6146b45d9b9eb7997db6193ef08747b6e6e3f5a6 Mon Sep 17 00:00:00 2001
From: Oleksandr Hladchenko1 <oleksandr_hladchenko1@epam.com>
Date: Fri, 3 Nov 2023 15:41:51 +0200
Subject: [PATCH 2/4] UIIN-2647: Update CHANGELOG.md

---
 CHANGELOG.md | 1 +
 1 file changed, 1 insertion(+)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 316daf9f9..ada3de9f2 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -20,6 +20,7 @@
 * Create new instance success toast no longer shows the instance HRID. Fixes UIIN-2635.
 * Optimistic locking message not working for instances in non-consortial tenant. Fixes UIIN-2628.
 * Add immediate warning message when a local instance is shared. Refs UIIN-2617.
+* Show only local MARC Authorities when share local instance. Fixes UIIN-2647.
 
 ## [10.0.0](https://github.com/folio-org/ui-inventory/tree/v10.0.0) (2023-10-13)
 [Full Changelog](https://github.com/folio-org/ui-inventory/compare/v9.4.12...v10.0.0)

From 7f97b7b435f8f01a14132c7b0668fd5b20d6491b Mon Sep 17 00:00:00 2001
From: Oleksandr Hladchenko1 <oleksandr_hladchenko1@epam.com>
Date: Fri, 3 Nov 2023 15:52:20 +0200
Subject: [PATCH 3/4] UIIN-2647: Add brackets to query

---
 src/ViewInstance.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/ViewInstance.js b/src/ViewInstance.js
index 1c471f9f0..4ced3f855 100644
--- a/src/ViewInstance.js
+++ b/src/ViewInstance.js
@@ -529,7 +529,7 @@ class ViewInstance extends React.Component {
 
     this.props.mutator.authorities.GET({
       params: {
-        query: `id==${authorityIds}`,
+        query: `id==(${authorityIds})`,
       }
     }).then(({ authorities }) => {
       const localAuthorities = authorities.filter(authority => !authority.source.startsWith(CONSORTIUM_PREFIX));

From b119f959d4b74044b9483d1a752c9dadeee3205d Mon Sep 17 00:00:00 2001
From: Oleksandr Hladchenko1 <oleksandr_hladchenko1@epam.com>
Date: Mon, 6 Nov 2023 18:23:08 +0200
Subject: [PATCH 4/4] UIIN-2647: Fix test

---
 src/ViewInstance.test.js | 1 +
 1 file changed, 1 insertion(+)

diff --git a/src/ViewInstance.test.js b/src/ViewInstance.test.js
index 91a968489..c8c51c36d 100644
--- a/src/ViewInstance.test.js
+++ b/src/ViewInstance.test.js
@@ -298,6 +298,7 @@ describe('ViewInstance', () => {
 
   it('should show a correct Warning message banner when instance sharing is in progress', async () => {
     defaultProp.mutator.shareInstance.POST.mockResolvedValue({});
+    defaultProp.mutator.authorities.GET.mockResolvedValueOnce({ authorities: [] });
     checkIfUserInMemberTenant.mockClear().mockReturnValue(true);
 
     renderViewInstance({ isShared: false });