fix: remove usage of false in component return (#1487)

graasp · Sep 30, 2024 · 9def0fb · 9def0fb
1 parent b9e6afe
commit 9def0fb
Show file tree

Hide file tree

Showing 6 changed files with 7 additions and 24 deletions.
diff --git a/cypress/support/commands.ts b/cypress/support/commands.ts
@@ -61,7 +61,6 @@ import {
   mockGetMembers,
   mockGetMembersBy,
   mockGetMembershipRequestsForItem,
-  mockGetOwnItems,
   mockGetOwnMembershipRequests,
   mockGetParents,
   mockGetPublicationStatus,
@@ -188,8 +187,6 @@ Cypress.Commands.add(
 
     mockGetAccessibleItems(cachedItems);
 
-    mockGetOwnItems(cachedItems);
-
     mockGetSharedItems({ items: cachedItems, member: currentMember });
 
     mockPostItem(cachedItems, postItemError);

diff --git a/cypress/support/server.ts b/cypress/support/server.ts
@@ -56,7 +56,6 @@ const {
   buildEditItemRoute,
   buildItemUnpublishRoute,
   buildGetItemRoute,
-  GET_OWN_ITEMS_ROUTE,
   buildGetMemberRoute,
   buildPostManyItemMembershipsRoute,
   ITEMS_ROUTE,
@@ -165,19 +164,6 @@ export const mockGetCurrentMember = (
   ).as('getCurrentMember');
 };
 
-export const mockGetOwnItems = (items: ItemForTest[]): void => {
-  cy.intercept(
-    {
-      method: HttpMethod.Get,
-      url: `${API_HOST}/${GET_OWN_ITEMS_ROUTE}`,
-    },
-    (req) => {
-      const own = items.filter(isRootItem);
-      req.reply(own);
-    },
-  ).as('getOwnItems');
-};
-
 export const mockGetAccessibleItems = (items: ItemForTest[]): void => {
   cy.intercept(
     {

diff --git a/src/components/alerts/MemberValidationBanner.tsx b/src/components/alerts/MemberValidationBanner.tsx
@@ -36,7 +36,7 @@ const buildLocalizedDocumentationLink = (lang: string): string => {
   return `${buildLocalizedDocumentationOrigin(lang)}${MEMBER_VALIDATION_DOCUMENTATION_LINK}`;
 };
 
-const MemberValidationBanner = (): JSX.Element | false => {
+const MemberValidationBanner = (): JSX.Element | null => {
   const { isOpen, closeModal } = useModalStatus({
     isInitiallyOpen: true,
   });
@@ -71,6 +71,6 @@ const MemberValidationBanner = (): JSX.Element | false => {
       </Alert>
     );
   }
-  return false;
+  return null;
 };
 export default MemberValidationBanner;
diff --git a/src/components/common/ShowChatboxButton.tsx b/src/components/common/ShowChatboxButton.tsx
@@ -35,7 +35,7 @@ const ShowChatboxButton = ({
   return (
     <ChatboxButton
       color={showChatbox ? 'primary' : 'inherit'}
-      showChat={showChatbox || false}
+      showChat={showChatbox ?? false}
       type={type}
       onClick={onClick}
       showChatText={translateBuilder(showChatTextKey)}

diff --git a/src/components/item/sharing/csvImport/DisplayInvitationSummary.tsx b/src/components/item/sharing/csvImport/DisplayInvitationSummary.tsx
@@ -54,7 +54,7 @@ type Props = {
 const DisplayInvitationSummary = ({
   userCsvData,
   error,
-}: Props): JSX.Element | false => {
+}: Props): JSX.Element | null => {
   const { t } = useBuilderTranslation();
   if (error) {
     const additionalMessage = getErrorFromPayload(error);
@@ -115,6 +115,6 @@ const DisplayInvitationSummary = ({
   }
 
   // no error and no data, display nothing
-  return false;
+  return null;
 };
 export default DisplayInvitationSummary;
diff --git a/src/components/main/MainMenu.tsx b/src/components/main/MainMenu.tsx
@@ -46,7 +46,7 @@ const ResourceLinks = () => {
   );
 };
 
-const MainMenu = (): JSX.Element | false => {
+const MainMenu = (): JSX.Element | null => {
   const { t } = useBuilderTranslation();
   const navigate = useNavigate();
   const { pathname } = useLocation();
@@ -57,7 +57,7 @@ const MainMenu = (): JSX.Element | false => {
   };
 
   if (!member || !member.id) {
-    return false;
+    return null;
   }
 
   const individualMenuItems =