refactor: renamed hooks to be more explicit

palladiumkenya · Aug 1, 2024 · 84d13c7 · 84d13c7
1 parent 817a237
commit 84d13c7
Show file tree

Hide file tree

Showing 8 changed files with 29 additions and 83 deletions.
diff --git a/...s/esm-patient-clinical-view-app/src/case-management/workspace/case-management.resource.ts b/...s/esm-patient-clinical-view-app/src/case-management/workspace/case-management.resource.ts
@@ -51,32 +51,6 @@ export const useCaseManagers = () => {
   return { data, error };
 };
 
-export const useCaseManagerRelationshipType = () => {
-  const customRepresentation = 'custom:(uuid,display,displayAIsToB,displayBIsToA)&q=Case manager';
-  const url = `/ws/rest/v1/relationshiptype?v=${customRepresentation}`;
-  const { data, error } = useSWRImmutable<{ data: RelationshipTypeResponse }>(url, openmrsFetch);
-
-  const mappedRelationshipTypes: Array<{ uuid: string; display: string; direction: string }> = [];
-
-  data?.data.results.forEach((type) => {
-    const aIsToB = {
-      display: type.displayAIsToB ? type.displayAIsToB : type.displayBIsToA,
-      uuid: type.uuid,
-      direction: 'aIsToB',
-    };
-    const bIsToA = {
-      display: type.displayBIsToA ? type.displayBIsToA : type.displayAIsToB,
-      uuid: type.uuid,
-      direction: 'bIsToA',
-    };
-    aIsToB.display === bIsToA.display
-      ? mappedRelationshipTypes.push(aIsToB)
-      : mappedRelationshipTypes.push(aIsToB, bIsToA);
-  });
-
-  return { data: mappedRelationshipTypes, error };
-};
-
 export const saveRelationship = (payload) => {
   const url = `/ws/rest/v1/relationship`;
   return openmrsFetch(url, {

diff --git a/...esm-patient-clinical-view-app/src/case-management/workspace/case-management.workspace.tsx b/...esm-patient-clinical-view-app/src/case-management/workspace/case-management.workspace.tsx
@@ -1,20 +1,15 @@
-import React, { useMemo, useState } from 'react';
+import React, { useState } from 'react';
 import { useTranslation } from 'react-i18next';
 import { Column, TextArea, Form, Stack, ButtonSet, ComboBox, Button, DatePicker, DatePickerInput } from '@carbon/react';
-import { useForm, Controller } from 'react-hook-form';
+import { useForm, Controller, SubmitHandler } from 'react-hook-form';
 import { zodResolver } from '@hookform/resolvers/zod';
 import { z } from 'zod';
 import styles from './case-management.scss';
-import { ExtensionSlot, navigate, showSnackbar, useSession } from '@openmrs/esm-framework';
-import {
-  saveRelationship,
-  useActivecases,
-  useCaseManagerRelationshipType,
-  useCaseManagers,
-} from './case-management.resource';
+import { ExtensionSlot, showSnackbar, useConfig, useSession } from '@openmrs/esm-framework';
+import { saveRelationship, useActivecases, useCaseManagers } from './case-management.resource';
 import { extractNameString, uppercaseText } from '../../utils/expression-helper';
 import PatientInfo from './patient-info.component';
-import { caseManagementConceptMap } from './case-management-concept-map';
+import { useMappedRelationshipTypes } from '../../family-partner-history/relationships.resource';
 
 const schema = z.object({
   caseManager: z.string().nonempty({ message: 'Case Manager is required' }),
@@ -24,6 +19,8 @@ const schema = z.object({
   notes: z.string().optional(),
 });
 
+type FormData = z.infer<typeof schema>;
+
 type CaseManagementProp = {
   closeWorkspace: () => void;
 };
@@ -34,14 +31,15 @@ const CaseManagementForm: React.FC<CaseManagementProp> = ({ closeWorkspace }) =>
   const [patientUuid, setPatientUuid] = useState('');
   const [patientSelected, setPatientSelected] = useState(false);
   const { data } = useCaseManagers();
+  const { data: relationshipTypes } = useMappedRelationshipTypes();
 
-  const { data: relationshipTypesData } = useCaseManagerRelationshipType();
-
-  const caseManagerRlshipType =
-    relationshipTypesData?.map((relationship) => ({
-      id: relationship.uuid,
-      text: relationship.display,
-    })) || [];
+  const caseManagerRelationshipTypeMapped =
+    relationshipTypes
+      .filter((relationshipType) => relationshipType.display === 'Case manager')
+      ?.map((relationship) => ({
+        id: relationship.uuid,
+        text: relationship.display,
+      })) || [];
 
   const caseManagerUuid = user?.person.uuid;
   const { mutate: fetchCases } = useActivecases(caseManagerUuid);
@@ -52,30 +50,24 @@ const CaseManagementForm: React.FC<CaseManagementProp> = ({ closeWorkspace }) =>
       text: manager.display,
     })) || [];
 
-  const conceptReasons = useMemo(() => {
-    return Object.keys(caseManagementConceptMap.answers).map((key) => ({
-      id: key,
-      text: caseManagementConceptMap.answers[key],
-    }));
-  }, []);
-
   const {
     control,
     handleSubmit,
     formState: { isValid },
-  } = useForm({
+  } = useForm<FormData>({
     mode: 'onChange',
     resolver: zodResolver(schema),
   });
 
-  const onSubmit = async (data) => {
+  const onSubmit: SubmitHandler<FormData> = async (data) => {
     const payload = {
       personA: data.caseManager,
       relationshipType: data.relationship,
       personB: patientUuid,
       startDate: data.startDate.toISOString(),
       endDate: data.endDate ? data.endDate.toISOString() : null,
     };
+
     try {
       await saveRelationship(payload);
       await fetchCases();
@@ -87,10 +79,8 @@ const CaseManagementForm: React.FC<CaseManagementProp> = ({ closeWorkspace }) =>
         isLowContrast: true,
       });
 
-      // Navigate or handle closing the form after submission
       closeWorkspace();
     } catch (err) {
-      console.error(err);
       showSnackbar({
         kind: 'error',
         title: t('RlshipError', 'Relationship Error'),
@@ -162,7 +152,7 @@ const CaseManagementForm: React.FC<CaseManagementProp> = ({ closeWorkspace }) =>
                 id="relationship_name"
                 titleText={t('relationship', 'Relationship')}
                 placeholder="Select Relationship"
-                items={caseManagerRlshipType}
+                items={caseManagerRelationshipTypeMapped}
                 itemToString={(item) => (item ? uppercaseText(item.text) : '')}
                 onChange={(e) => field.onChange(e.selectedItem?.id)}
                 invalid={!!fieldState.error}
@@ -210,25 +200,6 @@ const CaseManagementForm: React.FC<CaseManagementProp> = ({ closeWorkspace }) =>
           />
         </Column>
 
-        {/* <Column>
-          <Controller
-            name="reasons"
-            control={control}
-            render={({ field, fieldState }) => (
-              <ComboBox
-                id="reasons"
-                placeholder="Select Reason for Assignment"
-                titleText={t('reasons', 'Reason for Assignment')}
-                items={conceptReasons}
-                itemToString={(item) => (item ? uppercaseText(item.text) : '')}
-                onChange={(e) => field.onChange(e.selectedItem?.id)}
-                invalid={!!fieldState.error}
-                invalidText={fieldState.error?.message}
-              />
-            )}
-          />
-        </Column> */}
-
         <Column className={styles.textbox}>
           <Controller
             name="notes"

diff --git a/packages/esm-patient-clinical-view-app/src/config-schema.ts b/packages/esm-patient-clinical-view-app/src/config-schema.ts
@@ -1,4 +1,5 @@
 import { Type } from '@openmrs/esm-framework';
+import _default from 'react-hook-form/dist/logic/appendErrors';
 
 export const configSchema = {
   encounterTypes: {

diff --git a/...ges/esm-patient-clinical-view-app/src/family-partner-history/family-history.component.tsx b/...ges/esm-patient-clinical-view-app/src/family-partner-history/family-history.component.tsx
@@ -25,7 +25,7 @@ import {
   useLayoutType,
   usePagination,
 } from '@openmrs/esm-framework';
-import { useRelationships } from './relationships.resource';
+import { usePatientRelationships } from './relationships.resource';
 import ConceptObservations from './concept-obs.component';
 import type { ConfigObject } from '../config-schema';
 import styles from './family-history.scss';
@@ -40,7 +40,7 @@ const FamilyHistory: React.FC<FamilyHistoryProps> = ({ patientUuid }) => {
   const { concepts, familyRelationshipsTypeList } = config;
   const layout = useLayoutType();
   const [pageSize, setPageSize] = useState(10);
-  const { relationships, error, isLoading, isValidating } = useRelationships(patientUuid);
+  const { relationships, error, isLoading, isValidating } = usePatientRelationships(patientUuid);
 
   const familyRelationshipTypeUUIDs = new Set(familyRelationshipsTypeList.map((type) => type.uuid));
   const familyRelationships = relationships.filter((r) => familyRelationshipTypeUUIDs.has(r.relationshipTypeUUID));

diff --git a/...sm-patient-clinical-view-app/src/family-partner-history/family-relationship.workspace.tsx b/...sm-patient-clinical-view-app/src/family-partner-history/family-relationship.workspace.tsx
@@ -10,7 +10,7 @@ import { uppercaseText } from '../utils/expression-helper';
 import { saveRelationship } from '../case-management/workspace/case-management.resource';
 import PatientInfo from '../case-management/workspace/patient-info.component';
 import { mutate } from 'swr';
-import { useMappedRelationshipTypes, useRelationships } from './relationships.resource';
+import { useMappedRelationshipTypes, usePatientRelationships } from './relationships.resource';
 import { ConfigObject } from '../config-schema';
 
 const schema = z.object({
@@ -29,7 +29,7 @@ type RelationshipFormProps = {
 const FamilyRelationshipForm: React.FC<RelationshipFormProps> = ({ closeWorkspace, rootPersonUuid }) => {
   const { t } = useTranslation();
   const [relatedPersonUuid, setRelatedPersonUuid] = useState<string | undefined>(undefined);
-  const { relationshipsUrl } = useRelationships(rootPersonUuid);
+  const { relationshipsUrl } = usePatientRelationships(rootPersonUuid);
   const { data: mappedRelationshipTypes } = useMappedRelationshipTypes();
   const { familyRelationshipsTypeList } = useConfig<ConfigObject>();
   const familyRelationshipTypesUUIDs = new Set(familyRelationshipsTypeList.map((r) => r.uuid));

diff --git a/packages/esm-patient-clinical-view-app/src/family-partner-history/relationships.resource.tsx b/packages/esm-patient-clinical-view-app/src/family-partner-history/relationships.resource.tsx
@@ -123,7 +123,7 @@ export const useMappedRelationshipTypes = () => {
   return { data: relations, error };
 };
 
-export function useRelationships(patientUuid: string) {
+export function usePatientRelationships(patientUuid: string) {
   const customRepresentation =
     'custom:(display,uuid,personA:(uuid,age,display,dead,causeOfDeath),personB:(uuid,age,display,dead,causeOfDeath),relationshipType:(uuid,display,description,aIsToB,bIsToA))';
 

diff --git a/...s/esm-patient-clinical-view-app/src/other-relationships/other-relationships.component.tsx b/...s/esm-patient-clinical-view-app/src/other-relationships/other-relationships.component.tsx
@@ -27,7 +27,7 @@ import {
 } from '@openmrs/esm-framework';
 import type { ConfigObject } from '../config-schema';
 import styles from './other-relationships.scss';
-import { useRelationships } from '../family-partner-history/relationships.resource';
+import { usePatientRelationships } from '../family-partner-history/relationships.resource';
 import ConceptObservations from '../family-partner-history/concept-obs.component';
 
 interface OtherRelationshipsProps {
@@ -41,7 +41,7 @@ export const OtherRelationships: React.FC<OtherRelationshipsProps> = ({ patientU
   const { concepts, familyRelationshipsTypeList } = config;
   const [pageSize, setPageSize] = useState(10);
 
-  const { relationships, error, isLoading, isValidating } = useRelationships(patientUuid);
+  const { relationships, error, isLoading, isValidating } = usePatientRelationships(patientUuid);
   const familyRelationshipTypeUUIDs = new Set(familyRelationshipsTypeList.map((type) => type.uuid));
   const nonFamilyRelationships = relationships.filter((r) => !familyRelationshipTypeUUIDs.has(r.relationshipTypeUUID));
 

diff --git a/...s/esm-patient-clinical-view-app/src/other-relationships/other-relationships.workspace.tsx b/...s/esm-patient-clinical-view-app/src/other-relationships/other-relationships.workspace.tsx
@@ -10,7 +10,7 @@ import { uppercaseText } from '../utils/expression-helper';
 import { saveRelationship } from '../case-management/workspace/case-management.resource';
 import PatientInfo from '../case-management/workspace/patient-info.component';
 import { mutate } from 'swr';
-import { useMappedRelationshipTypes, useRelationships } from '../family-partner-history/relationships.resource';
+import { useMappedRelationshipTypes, usePatientRelationships } from '../family-partner-history/relationships.resource';
 import { ConfigObject } from '../config-schema';
 
 const schema = z.object({
@@ -29,7 +29,7 @@ type OtherRelationshipsFormProps = {
 export const OtherRelationshipsForm: React.FC<OtherRelationshipsFormProps> = ({ closeWorkspace, rootPersonUuid }) => {
   const { t } = useTranslation();
   const [relatedPersonUuid, setRelatedPersonUuid] = useState<string | undefined>(undefined);
-  const { relationshipsUrl } = useRelationships(rootPersonUuid);
+  const { relationshipsUrl } = usePatientRelationships(rootPersonUuid);
   const { data: mappedRelationshipTypes } = useMappedRelationshipTypes();
   const { familyRelationshipsTypeList } = useConfig<ConfigObject>();
   const familyRelationshipTypesUUIDs = new Set(familyRelationshipsTypeList.map((r) => r.uuid));