Hotfix: Include guards for .split

Author: leekahung

src/components/CivicProfileForms/FormLayout.jsx

@@ -21,7 +21,7 @@ import HMIS_FORM_LIST from './FormList';
  */
 const FormLayout = ({ children }) => {
   const location = useLocation();
-  const path = location.pathname.split('/').pop();
+  const path = location.pathname?.split('/')?.pop() || '';
 
   localStorage.setItem('restorePath', location.pathname);
   const pageIdx = HMIS_FORM_LIST.findIndex((form) => form.path === path);

src/components/Documents/DocumentCard.jsx

@@ -43,7 +43,7 @@ import { truncateText, getTypeText } from '@utils';
  */
 const DocumentCard = ({ document, onShare, onDelete, onPreview }) => {
   const location = useLocation();
-  const profileWebId = decodeURIComponent(location.pathname.split('/')[2]);
+  const profileWebId = decodeURIComponent(location.pathname?.split('/')?.[2] || '');
 
   const [anchorEl, setAnchorEl] = useState(null);
   const [openMenu, setOpenMenu] = useState(null);

src/components/Modals/AddContactModal.jsx

@@ -81,7 +81,7 @@ const AddContactModal = ({
   const isSmallScreen = useMediaQuery(theme.breakpoints.down('sm'));
   const [oidcProviders] = useState(() =>
     ENV.VITE_SUGGESTED_OIDC_OPTIONS !== undefined
-      ? [...ENV.VITE_SUGGESTED_OIDC_OPTIONS.split(', '), 'Other']
+      ? [...(ENV.VITE_SUGGESTED_OIDC_OPTIONS?.split(', ') || []), 'Other']
       : ['Other']
   );
   const [Oidc, setOIDC] = useState('');

src/components/Modals/NewMessageModal.jsx

@@ -51,7 +51,9 @@ const NewMessageModal = ({ showModal, setShowModal, oldMessage = '', toField = '
 
   const [message, setMessage] = useState({
     recipientPodUrl:
-      oldMessage && oldMessage.senderWebId ? oldMessage.senderWebId.split('profile')[0] : '',
+      oldMessage && oldMessage.senderWebId
+        ? oldMessage.senderWebId?.split('profile')?.[0] || ''
+        : '',
     title: oldMessage ? `RE:${oldMessage.title}`.replace('RE:RE:', 'RE:') : '',
     message: '',
     inReplyTo: oldMessage ? oldMessage.messageId : '',

src/components/Profile/CreateQRCode.jsx

@@ -26,7 +26,7 @@ const CreateQRCode = ({ webId }) => {
 
   const createQrCode = () => {
     setQrIconClicked(!qrIconClicked);
-    const trimmedWebId = webId && webId.includes('#') ? webId.split('#')[0] : webId;
+    const trimmedWebId = webId && webId.includes('#') ? webId?.split('#')?.[0] || webId : webId;
     setDocUrl(trimmedWebId);
   };
 

src/components/Signup/podSignupHelpers/initializePod.js

@@ -68,7 +68,7 @@ const initializePod = async (
     let contactsList = createSolidDataset();
     let builder = buildThing(createThing({ name: encodeURIComponent(caseManagerWebId) }))
       .addUrl(RDF_PREDICATES.identifier, caseManagerWebId)
-      .addUrl(RDF_PREDICATES.URL, caseManagerWebId.split('profile')[0]);
+      .addUrl(RDF_PREDICATES.URL, caseManagerWebId?.split('profile')?.[0] || '');
 
     if (caseManagerFirstName) {
       builder = builder.addStringNoLocale(RDF_PREDICATES.givenName, caseManagerFirstName);

src/contexts/SignedInUserContext.jsx

@@ -56,7 +56,7 @@ export const SignedInUserContextProvider = ({ children }) => {
     const loadUserInfo = async () => {
       try {
         let fetchedPodUrl = (await getPodUrlAll(webId, { fetch: session.fetch }))[0];
-        fetchedPodUrl = fetchedPodUrl || webId.split('profile')[0];
+        fetchedPodUrl = fetchedPodUrl || webId?.split('profile')?.[0] || '';
         setPodUrl(fetchedPodUrl);
         const fetchedProfileData = await fetchProfileInfo(webId);
         if (fetchedProfileData.profileInfo.profileImage) {

src/layouts/Breadcrumbs.jsx

@@ -17,17 +17,14 @@ import Typography from '@mui/material/Typography';
 const Breadcrumbs = () => {
   const location = useLocation();
 
-  const crumbs = location?.pathname
-    .split('/')
-    .slice(1)
-    .map((crumb) => {
+  const crumbs =
+    location?.pathname?.split('/') ||
+    [].slice(1).map((crumb) => {
       let string = crumb;
       string = decodeURIComponent(crumb.replace('-', ' '));
       if (!string.includes('http')) {
-        string = string
-          .split(' ')
-          .map((s) => s.charAt(0).toUpperCase() + s.slice(1))
-          .join(' ');
+        string =
+          string?.split(' ') || [].map((s) => s.charAt(0).toUpperCase() + s.slice(1)).join(' ');
       }
       return string;
     });

src/model-helpers/Profile.js

@@ -103,7 +103,7 @@ export const updateProfileInfo = async (session, profileData, inputValues) => {
 export const uploadProfileImage = async (session, profileData, inputImage) => {
   let { profileDataset, profileThing } = profileData;
 
-  const profileContainer = `${session.info.webId.split('card')[0]}`;
+  const profileContainer = `${session.info.webId?.split('card')?.[0] || ''}`;
   const profileImgFilename = inputImage.name.replace(' ', '%20');
 
   const savedFile = await saveFileInContainer(profileContainer, inputImage, {

src/pages/CivicProfile.jsx

@@ -20,7 +20,7 @@ const CivicProfile = () => {
   const location = useLocation();
 
   localStorage.setItem('restorePath', location.pathname);
-  const currentForm = location.pathname.split('/').pop();
+  const currentForm = location.pathname?.split('/')?.pop() || '';
 
   return (
     <Container>