lint fixes

uwcirg · Aug 22, 2023 · 818db28 · 818db28
1 parent e0549a5
commit 818db28
Show file tree

Hide file tree

Showing 6 changed files with 45 additions and 23 deletions.
diff --git a/patientsearch/src/__tests__/components/patientList/DetailPanel.test.js b/patientsearch/src/__tests__/components/patientList/DetailPanel.test.js
@@ -6,10 +6,10 @@ describe("DetailPanel", () => {
   it("DetailPanel component renders without crashing", () => {
     const data = {
       rowData: {
-        id: 1
-      }
-    }
-    const wrapper = shallow(<DetailPanel data={data}/>);
+        id: 1,
+      },
+    };
+    const wrapper = shallow(<DetailPanel data={data} />);
     expect(wrapper).toBeDefined();
   });
 });
diff --git a/patientsearch/src/js/components/patientList/DetailPanel.js b/patientsearch/src/js/components/patientList/DetailPanel.js
@@ -5,19 +5,31 @@ import Paper from "@material-ui/core/Paper";
 import useStyles from "../../../styles/patientListStyle";
 import { usePatientListContext } from "../../context/PatientListContextProvider";
 
-export default function DetailPanel({data}) {
+export default function DetailPanel({ data }) {
   const classes = useStyles();
   let {
     selectedMenuItem,
     getSelectedItemComponent,
     handleMenuClose,
     handleToggleDetailPanel,
-    shouldHideMoreMenu
+    shouldHideMoreMenu,
   } = usePatientListContext();
-  if (!shouldHideMoreMenu) shouldHideMoreMenu = function() { return true};
-  if (!handleToggleDetailPanel) shouldHideMoreMenu = function() { return true};
-  if (!handleMenuClose) handleMenuClose = function() { return true};
-  if (!getSelectedItemComponent) getSelectedItemComponent = function() { return null};
+  if (!shouldHideMoreMenu)
+    shouldHideMoreMenu = function () {
+      return true;
+    };
+  if (!handleToggleDetailPanel)
+    shouldHideMoreMenu = function () {
+      return true;
+    };
+  if (!handleMenuClose)
+    handleMenuClose = function () {
+      return true;
+    };
+  if (!getSelectedItemComponent)
+    getSelectedItemComponent = function () {
+      return null;
+    };
   if (shouldHideMoreMenu()) return false;
   return (
     <div className={classes.detailPanelWrapper}>
@@ -44,4 +56,4 @@ export default function DetailPanel({data}) {
 
 DetailPanel.propTypes = {
   data: PropTypes.object.isRequired,
-};
+};
diff --git a/patientsearch/src/js/components/patientList/DropdownMenu.js b/patientsearch/src/js/components/patientList/DropdownMenu.js
@@ -10,7 +10,10 @@ export default function DropdownMenu() {
     shouldHideMoreMenu,
     shouldShowMenuItem,
   } = usePatientListContext();
-  if (!shouldHideMoreMenu) shouldHideMoreMenu = function () { return true };
+  if (!shouldHideMoreMenu)
+    shouldHideMoreMenu = function () {
+      return true;
+    };
   if (shouldHideMoreMenu()) return false;
   return (
     <Dropdown

diff --git a/patientsearch/src/js/components/patientList/FilterRow.js b/patientsearch/src/js/components/patientList/FilterRow.js
@@ -49,7 +49,7 @@ const useStyles = makeStyles((theme) => ({
   },
 }));
 
-export default function FilterRow(props) {
+export default function FilterRow() {
   let {
     //methods
     handleSearch,

diff --git a/patientsearch/src/js/components/patientList/PatientListTable.js b/patientsearch/src/js/components/patientList/PatientListTable.js
@@ -47,15 +47,21 @@ export default function PatientListTable() {
     openLoadingModal,
   } = usePatientListContext();
 
-  if (!handleErrorCallback) handleErrorCallback = function() {};
-  if (!handlePageUnload) handlePageUnload = function() {};
-  if (!getColumns) getColumns = function() {return []};
-  if (!getPatientList) getPatientList = function() { return []};
-  if (!getTableActions) getTableActions = function() {};
-  if (!getTableRowEvent) getTableRowEvent = function() {};
-  if (!getTableEditableOptions) getTableEditableOptions = function() {};
-  if (!getTableLocalizations) getTableLocalizations = function() {};
-  if (!getTableOptions) getTableOptions = function() {}
+  if (!handleErrorCallback) handleErrorCallback = function () {};
+  if (!handlePageUnload) handlePageUnload = function () {};
+  if (!getColumns)
+    getColumns = function () {
+      return [];
+    };
+  if (!getPatientList)
+    getPatientList = function () {
+      return [];
+    };
+  if (!getTableActions) getTableActions = function () {};
+  if (!getTableRowEvent) getTableRowEvent = function () {};
+  if (!getTableEditableOptions) getTableEditableOptions = function () {};
+  if (!getTableLocalizations) getTableLocalizations = function () {};
+  if (!getTableOptions) getTableOptions = function () {};
 
   const renderPatientSearchRow = () => (
     <table className={classes.filterTable}>

diff --git a/patientsearch/src/js/context/PatientListContextProvider.js b/patientsearch/src/js/context/PatientListContextProvider.js
@@ -25,7 +25,8 @@ const PatientListContext = React.createContext({});
  */
 let filterIntervalId = 0;
 export default function PatientListContextProvider({ children }) {
-  const appSettings = useSettingContext().appSettings;
+  const settingsCxt = useSettingContext();
+  const appSettings = settingsCxt ? settingsCxt.appSettings : {};
   const { user, userError } = useUserContext();
   const { userName, roles } = user || {};
   const appClients = getClientsByRequiredRoles(