Make the digital reservation label functionality clearer

By renaming the parameters and thereby make the purpose of the functions and their
parameters self explanatory. Hopefully it is more clear :)

src/apps/reservation-list/reservation-material/reservation-info.tsx

@@ -51,9 +51,14 @@ const ReservationInfo: FC<ReservationInfoProps> = ({
       return "";
     }
 
+    const date = (isDigital ? expiryDate : pickupDeadline) ?? null;
+    if (!date) {
+      return "";
+    }
+
     return getReservationStatusInfoLabel({
-      pickupBranch: pickupBranch ?? undefined,
-      pickupDeadline: (isDigital ? expiryDate : pickupDeadline) ?? "",
+      pickupBranch,
+      date,
       t,
       isDigital
     });

src/apps/reservation-list/utils/helpers.ts

@@ -39,28 +39,32 @@ export const getReadyForPickup = (list: ReservationType[]) => {
   });
 };
 
+export const infoLabelTextType = {
+  pickUpLatest: "reservationPickUpLatestText",
+  loanBefore: "reservationListLoanBeforeText"
+} as const;
+
 export const getReservationStatusInfoLabel = ({
   pickupBranch,
-  pickupDeadline,
+  date,
   isDigital,
   t
 }: {
-  pickupBranch?: string;
-  pickupDeadline: string;
+  pickupBranch: string | null | undefined;
+  date: string;
   isDigital: boolean;
   t: UseTextFunction;
 }) => {
   const textKey = pickupBranch
-    ? "reservationPickUpLatestText"
-    : "reservationListLoanBeforeText";
-  const date = formatDateDependingOnDigitalMaterial({
-    date: pickupDeadline,
-    materialIsDigital: isDigital
-  });
+    ? infoLabelTextType.pickUpLatest
+    : infoLabelTextType.loanBefore;
 
   return t(textKey, {
     placeholders: {
-      "@date": date
+      "@date": formatDateDependingOnDigitalMaterial({
+        date,
+        isDigital
+      })
     }
   });
 };

src/components/GroupModal/GroupModalReservationsList.tsx

@@ -91,7 +91,7 @@ const GroupModalReservationsList: FC<GroupModalReservationsListProps> = ({
                     ) : (
                       expiryDate && (
                         <ReservationStatusInfoLabel
-                          expiryDate={expiryDate}
+                          date={expiryDate}
                           isDigital
                         />
                       )

src/components/reservation/ReservationStatusInfoLabel.tsx

@@ -5,22 +5,22 @@ import { useText } from "../../core/utils/text";
 import { getReservationStatusInfoLabel } from "../../apps/reservation-list/utils/helpers";
 
 export interface ReservationStatusInfoLabelProps {
-  pickupBranch?: string;
-  expiryDate: string;
+  pickupBranch?: string | null;
+  date: string;
   isDigital: boolean;
 }
 
 const ReservationStatusInfoLabel: FC<ReservationStatusInfoLabelProps> = ({
   pickupBranch,
-  expiryDate,
+  date,
   isDigital
 }) => {
   const t = useText();
   return (
     <InfoLabel>
       {getReservationStatusInfoLabel({
         pickupBranch,
-        pickupDeadline: expiryDate,
+        date,
         t,
         isDigital
       })}

src/core/utils/helpers/date.ts

@@ -15,12 +15,12 @@ export const formatDateTime = (date: string) => {
 
 export const formatDateDependingOnDigitalMaterial = ({
   date,
-  materialIsDigital
+  isDigital
 }: {
   date: string;
-  materialIsDigital: boolean;
+  isDigital: boolean;
 }) => {
-  return materialIsDigital ? formatDateTime(date) : formatDate(date);
+  return isDigital ? formatDateTime(date) : formatDate(date);
 };
 
 export default getCurrentUnixTime;