Activate eslint no-unused-expressions

Enables the eslint rule no-unused-expressions.

Author: Arnei

eslint.config.js

@@ -19,7 +19,6 @@ export default [
             "max-len": "off",
             "no-extra-boolean-cast": "off",
             "no-tabs": "off",
-            "no-unused-expressions": "off",
             "object-curly-spacing": "off",
             "prefer-const": "off",
             "quotes": "off",
@@ -34,7 +33,6 @@ export default [
             "@typescript-eslint/no-unsafe-assignment": "off",
             "@typescript-eslint/no-unsafe-call": "off",
             "@typescript-eslint/no-unsafe-member-access": "off",
-            "@typescript-eslint/no-unused-expressions": "off",
             "@typescript-eslint/no-unused-vars": "off",
             "@typescript-eslint/no-unsafe-return": "off",
             "@typescript-eslint/require-await": "off",

src/components/NavBar.tsx

@@ -52,12 +52,10 @@ const NavBar = ({
 	const newResourceModalRef = useRef<ModalHandle>(null);
 
 	const showNewResourceModal = async () => {
-		create && create.onShowModal && await create.onShowModal()
 		newResourceModalRef.current?.open()
 	};
 
 	const hideNewResourceModal = () => {
-		create && create.onHideModal && create.onHideModal()
 		newResourceModalRef.current?.close?.()
 	};
 

src/components/events/partials/ModalTabsAndPages/NewSourcePage.tsx

@@ -403,17 +403,21 @@ const Schedule = <T extends {
 									selected={typeof formik.values.scheduleStartDate === "string" ? parseISO(formik.values.scheduleStartDate) : formik.values.scheduleStartDate}
 									onChange={(value) => {
 										if (formik.values.sourceMode === "SCHEDULE_MULTIPLE") {
-											value && changeStartDateMultiple(
-												value,
-												formik.values,
-												formik.setFieldValue
-											);
+											if (value) {
+												changeStartDateMultiple(
+													value,
+													formik.values,
+													formik.setFieldValue
+												);
+											}
 										} else {
-											value && changeStartDate(
-												value,
-												formik.values,
-												formik.setFieldValue
-											);
+											if (value) {
+												changeStartDate(
+													value,
+													formik.values,
+													formik.setFieldValue
+												);
+											}
 										}
 									}}
 									showYearDropdown

src/components/shared/NavigationButtons.tsx

@@ -40,7 +40,9 @@ const NavigationButtons = ({
 					className={cn(submitClassName, submitActiveClassName)}
 					disabled={disabled}
 					onClick={() => {
-						!!nextPage && nextPage()
+						if (nextPage) {
+							nextPage()
+						}
 					}}
 					tabIndex={100}
 				>
@@ -52,7 +54,9 @@ const NavigationButtons = ({
 					className={cn(submitClassName, submitActiveClassName)}
 					disabled={disabled}
 					onClick={() => {
-						!!nextPage && nextPage();
+						if (nextPage) {
+							nextPage();
+						}
 					}}
 					tabIndex={100}
 				>
@@ -63,7 +67,9 @@ const NavigationButtons = ({
 				<button
 					className="cancel"
 					onClick={() => {
-						!!previousPage && previousPage()
+						if (previousPage) {
+							previousPage()
+						}
 					}}
 					tabIndex={101}
 				>

src/components/shared/modals/ResourceDetailsAccessPolicyTab.tsx

@@ -123,9 +123,11 @@ const ResourceDetailsAccessPolicyTab = ({
 			fetchRolesWithTarget("ACL").then((roles) => setRoles(roles));
 			if (fetchHasActiveTransactions) {
 				const fetchTransactionResult = await dispatch(fetchHasActiveTransactions(resourceId)).then(unwrapResult)
-				fetchTransactionResult.active !== undefined
-					? setTransactions({ read_only: fetchTransactionResult.active })
-					: setTransactions({ read_only: true });
+				if (fetchTransactionResult.active !== undefined) {
+					setTransactions({ read_only: fetchTransactionResult.active })
+				} else {
+					setTransactions({ read_only: true });
+				}
 				if (
 					fetchTransactionResult.active === undefined ||
 					fetchTransactionResult.active

src/components/shared/wizard/RenderField.tsx

@@ -39,8 +39,12 @@ const RenderField = ({
 		<div
 			onClick={() => {
 				if (editableRef.current) {
-					editableRef.current.focus && editableRef.current.focus()
-					editableRef.current.setFocus && editableRef.current.setFocus() // For DatePicker
+					if (editableRef.current.focus) {
+						editableRef.current.focus()
+					}
+					if (editableRef.current.setFocus) {
+						editableRef.current.setFocus() // For DatePicker
+					}
 				}
 			}}
 			onFocus={onFocus}

src/components/shared/wizard/WizardNavigationButtons.tsx

@@ -40,11 +40,25 @@ const WizardNavigationButtons = <T, >({
 			isSubmitDisabled={disabled}
 			nextPage={
 				isLast
-				? () => { !!submitPage ? submitPage() : formik.handleSubmit(); }
-				: () => { !!nextPage && nextPage(formik.values); }
+				? () => {
+					if (submitPage) {
+						submitPage()
+					 } else {
+						formik.handleSubmit();
+					 }
+				}
+				: () => {
+					if (nextPage) {
+						nextPage(formik.values);
+					}
+				}
 			}
 			previousPage={
-				() => { !!previousPage && previousPage(formik.values) }
+				() => {
+					if (previousPage) {
+						previousPage(formik.values)
+					}
+				}
 			}
 			nextTranslationString={
 				isLast

src/slices/eventDetailsSlice.ts

@@ -1343,7 +1343,9 @@ export const performWorkflowAction = createAppAsyncThunk('eventDetails/performWo
 					context: NOTIFICATION_CONTEXT
 				})
 			);
-			close && close();
+			if (close) {
+				close();
+			}
 		})
 		.catch((response) => {
 			dispatch(

src/utils/resourceUtils.ts

@@ -88,10 +88,18 @@ export const buildUserBody = (values: NewUser | UpdateUser) => {
 	let data = new URLSearchParams();
 	// fill form data with user inputs
 	data.append("username", values.username);
-	values.name && data.append("name", values.name);
-	values.email && data.append("email", values.email);
-	values.password && data.append("password", values.password);
-	values.roles && data.append("roles", JSON.stringify(values.roles));
+	if (values.name) {
+		data.append("name", values.name);
+	}
+	if (values.email) {
+		data.append("email", values.email);
+	}
+	if (values.password) {
+		data.append("password", values.password);
+	}
+	if (values.roles) {
+		data.append("roles", JSON.stringify(values.roles));
+	}
 
 	return data;
 };