refactor. variable name and remove unwanted code

Bahmni · Aug 4, 2023 · 18d92ef · 18d92ef
1 parent 8efc370
commit 18d92ef
Showing 1 changed file with 31 additions and 46 deletions.
diff --git a/src/entries/DrugChartModal/DrugChartModal.jsx b/src/entries/DrugChartModal/DrugChartModal.jsx
@@ -27,11 +27,11 @@ export default function DrugChartModal(props) {
   );
   const enable24HourTimers = hostData?.enable24HourTimers || true;
 
-  const initialStringArray = Array.from(
+  const schedulesLength = Array.from(
     { length: enableSchedule?.frequencyPerDay },
     () => ""
   );
-  const [schedules, setSchedules] = useState(initialStringArray);
+  const [schedules, setSchedules] = useState(schedulesLength);
   const [startTime, setStartTime] = useState("");
   const [
     showStartTimeBeyondNextDoseWarning,
@@ -278,19 +278,6 @@ export default function DrugChartModal(props) {
     hostApi.onModalClose?.();
   };
 
-  // const getAndSetFrequencies = async () => {
-  //   try {
-  //     const frequencies = await getDrugOrderFrequencies();
-  //     setAllFrequencies(frequencies);
-  //   } catch (error) {
-  //     console.log(error);
-  //   }
-  // };
-
-  // useEffect(() => {
-  //   getAndSetFrequencies();
-  // }, []);
-
   return (
     <>
       <I18nProvider>
@@ -395,37 +382,35 @@ export default function DrugChartModal(props) {
               <div className="schedule-section">
                 <Title text="Schedule(s)" isRequired={true} />
                 <div className="inline-field" id="schedule">
-                  {Array.from(
-                    { length: enableSchedule.frequencyPerDay },
-                    (_, index) =>
-                      enable24HourTimers ? (
-                        <div className="schedule-time" key={index}>
-                          <TimePicker24Hour
-                            key={index}
-                            id={`schedule-${index}`}
-                            defaultTime={schedules[index]}
-                            onChange={(time) => {
-                              handleSchedule(time, index);
-                            }}
-                            labelText=" "
-                            width="70%"
-                            invalidText={invalidTimeText24Hour}
-                          />
-                        </div>
-                      ) : (
-                        <div className="schedule-time" key={index}>
-                          <TimePicker
-                            key={index}
-                            labelText=" "
-                            defaultTime={schedules[index]}
-                            onChange={(time) => {
-                              handleSchedule(time, index);
-                            }}
-                            id={`schedule-${index}`}
-                            invalidText={invalidTimeText12Hour}
-                          />
-                        </div>
-                      )
+                  {Array.from({ length: schedulesLength }, (_, index) =>
+                    enable24HourTimers ? (
+                      <div className="schedule-time" key={index}>
+                        <TimePicker24Hour
+                          key={index}
+                          id={`schedule-${index}`}
+                          defaultTime={schedules[index]}
+                          onChange={(time) => {
+                            handleSchedule(time, index);
+                          }}
+                          labelText=" "
+                          width="70%"
+                          invalidText={invalidTimeText24Hour}
+                        />
+                      </div>
+                    ) : (
+                      <div className="schedule-time" key={index}>
+                        <TimePicker
+                          key={index}
+                          labelText=" "
+                          defaultTime={schedules[index]}
+                          onChange={(time) => {
+                            handleSchedule(time, index);
+                          }}
+                          id={`schedule-${index}`}
+                          invalidText={invalidTimeText12Hour}
+                        />
+                      </div>
+                    )
                   )}
                 </div>
                 {showScheduleOrderWarning && (