diff --git a/src/components/Consents/Consents.tsx b/src/components/Consents/Consents.tsx index 886f81cd7095e431c744fac023d11591e49d52f6..e2cd8fbfb45e1d7307cbd52d58ccdbcc17a08bb6 100644 --- a/src/components/Consents/Consents.tsx +++ b/src/components/Consents/Consents.tsx @@ -35,9 +35,9 @@ export const Consents: React.FC<{ type: 'sge' | 'grdf' }> = ({ type }) => { const [totalRows, setTotalRows] = useState<number>(50) const { data: user } = useWhoAmI() - const toggleOpenModal = useCallback(() => { + const toggleOpenModal = () => { setOpenDownloadModal(prev => !prev) - }, []) + } const consentService = useMemo( () => (isGRDF ? new GrdfConsentService() : new SgeConsentService()), @@ -57,19 +57,16 @@ export const Consents: React.FC<{ type: 'sge' | 'grdf' }> = ({ type }) => { [isGRDF] ) - const handleChangePage = useCallback( - ( - _event: React.MouseEvent<HTMLButtonElement, MouseEvent> | null, - newPage: number - ) => { - setPage(newPage) - }, - [] - ) - const handleChangeRowsPerPage = useCallback((event: any) => { + const handleChangePage = ( + _event: React.MouseEvent<HTMLButtonElement, MouseEvent> | null, + newPage: number + ) => { + setPage(newPage) + } + const handleChangeRowsPerPage = (event: any) => { setRowsPerPage(event.target.value) setPage(0) - }, []) + } const checkSelectedNodes = useCallback(() => { if (gridApi && !gridApi.isDestroyed()) { diff --git a/src/components/Newsletter/Newsletter.tsx b/src/components/Newsletter/Newsletter.tsx index 22d3f1872f1ff2fe385cca45f66649c80334e9a5..2143d28fdc13759051399cfc973d534509595be3 100644 --- a/src/components/Newsletter/Newsletter.tsx +++ b/src/components/Newsletter/Newsletter.tsx @@ -100,9 +100,9 @@ const Newsletter: React.FC = () => { setIsTouched(false) } } - const handleCancel = useCallback(() => { + const handleCancel = () => { setRefreshData(true) - }, []) + } const handleDeleteMailSubject = async (): Promise<void> => { if (user) { @@ -110,7 +110,7 @@ const Newsletter: React.FC = () => { date, getAxiosXSRFHeader(user.xsrftoken) ) - setRefreshData(true) + handleCancel() } } const handleDeleteMonthlyInfo = async (): Promise<void> => { @@ -119,7 +119,7 @@ const Newsletter: React.FC = () => { date, getAxiosXSRFHeader(user.xsrftoken) ) - setRefreshData(true) + handleCancel() } } const handleDeleteMonthlyNews = async (): Promise<void> => { @@ -128,7 +128,7 @@ const Newsletter: React.FC = () => { date, getAxiosXSRFHeader(user.xsrftoken) ) - setRefreshData(true) + handleCancel() } } const handleDeletePoll = async (): Promise<void> => { @@ -137,7 +137,7 @@ const Newsletter: React.FC = () => { date, getAxiosXSRFHeader(user.xsrftoken) ) - setRefreshData(true) + handleCancel() } } const handleOpenDeleteModal = (target: ContentItems) => { diff --git a/src/components/Prices/PriceSection.tsx b/src/components/Prices/PriceSection.tsx index 693d3424803d48ccf595b771d9e7c132c8a4d0cf..708fa904e1cd7145be16bed53c37034b00e90443 100644 --- a/src/components/Prices/PriceSection.tsx +++ b/src/components/Prices/PriceSection.tsx @@ -39,14 +39,14 @@ const PriceSection: React.FC<PriceSectionProps> = ({ fluid, frequency }) => { }) const maxPerList = 8 - const handlePriceSelection = useCallback((val: string) => { + const handlePriceSelection = (val: string) => { if (val === '') val = '0' val = val.replace(/,/g, '.') val = val.replace(/([^0-9.]+)/, '') setPriceToSave(prev => { return { ...prev, price: val } }) - }, []) + } const savePrice = useCallback(async () => { if ( @@ -68,20 +68,20 @@ const PriceSection: React.FC<PriceSectionProps> = ({ fluid, frequency }) => { } }, [priceToSave, user]) - const toggleHistory = useCallback(() => { + const toggleHistory = () => { setShowHistory(prev => !prev) - }, []) + } - const getDate = useCallback((isoString: string): string => { + const getDate = (isoString: string): string => { const date = new Date(isoString) const month = date.toLocaleString('fr', { month: 'long' }) const year = date.toLocaleString('fr', { year: 'numeric' }) return `${month} ${year}` - }, []) + } - const toggleFullList = useCallback(() => { + const toggleFullList = () => { setShowFullList(prev => !prev) - }, []) + } useEffect(() => { let subscribed = true