Merge pull request #458 from NIAEFEUP/fix/prevent-null-from-crashing

fix: prevent null from crashing

Author: tomaspalma

src/components/admin/AdminMarketplaceExchangesCard.tsx

@@ -109,7 +109,7 @@ export const AdminMarketplaceExhangesCard = ({
                         exchange.options.map(option => ({
                             participant_name: undefined,
                             participant_nmec: exchange.issuer_nmec,
-                            goes_from: option.class_issuer_goes_from.name,
+                            goes_from: option.class_issuer_goes_from?.name,
                             goes_to: option.class_issuer_goes_to.name,
                             course_acronym: option.course_info.acronym
                         }))

src/components/exchange/requests/view/cards/CommonRequestCard.tsx

@@ -93,6 +93,7 @@ export const CommonRequestCard = ({
       request.options.forEach((option) => {
         if (updatedOptions.get(option.course_info.acronym) === true) {
           const matchingClass = (type === "directexchange" ? option.class_participant_goes_to : option.class_issuer_goes_from);
+          if(!matchingClass) return;
           matchingClass.slots.forEach((slot) => {
             newExchangeSchedule.push({
               courseInfo: option.course_info,

src/components/exchange/requests/view/cards/ListRequestChanges.tsx

@@ -79,8 +79,8 @@ export const ListRequestChanges = ({
                             </TableHeader>
                             <TableBody>
                                 <TableRow>
-                                    <TableCell>{type === "directexchange" ? (option as DirectExchangeParticipant).class_participant_goes_from.name : (option as ExchangeOption).class_issuer_goes_from.name}</TableCell>
-                                    <TableCell>{type === "directexchange" ? (option as DirectExchangeParticipant).class_participant_goes_to.name : (option as ExchangeOption).class_issuer_goes_to.name}</TableCell>
+                                    <TableCell>{type === "directexchange" ? (option as DirectExchangeParticipant).class_participant_goes_from.name : (option as ExchangeOption).class_issuer_goes_from?.name}</TableCell>
+                                    <TableCell>{type === "directexchange" ? (option as DirectExchangeParticipant).class_participant_goes_to.name : (option as ExchangeOption).class_issuer_goes_to?.name}</TableCell>
                                 </TableRow>
                             </TableBody>
                         </Table>

src/hooks/useMarketplaceAcceptExchange.tsx

@@ -17,8 +17,8 @@ export default (request: MarketplaceRequest | DirectExchangeRequest, selectedOpt
                     {
                         courseUnitId: option.course_info.id,
                         courseUnitName: option.course_info.name,
-                        classNameRequesterGoesTo: (option as ExchangeOption).class_issuer_goes_from.name,
-                        classNameRequesterGoesFrom: (option as ExchangeOption).class_issuer_goes_to.name,
+                        classNameRequesterGoesTo: (option as ExchangeOption).class_issuer_goes_from?.name,
+                        classNameRequesterGoesFrom: (option as ExchangeOption).class_issuer_goes_to?.name,
                         other_student: {
                             name: request.issuer_name,
                             mecNumber: request.issuer_nmec