Skip to content
Snippets Groups Projects
Commit 240c318b authored by Bastien DUMONT's avatar Bastien DUMONT :angel:
Browse files

chore: remove useless useCallback

parent 23ec55f6
Branches
No related tags found
1 merge request!157Release to master
Pipeline #99657 passed
...@@ -35,9 +35,9 @@ export const Consents: React.FC<{ type: 'sge' | 'grdf' }> = ({ type }) => { ...@@ -35,9 +35,9 @@ export const Consents: React.FC<{ type: 'sge' | 'grdf' }> = ({ type }) => {
const [totalRows, setTotalRows] = useState<number>(50) const [totalRows, setTotalRows] = useState<number>(50)
const { data: user } = useWhoAmI() const { data: user } = useWhoAmI()
const toggleOpenModal = useCallback(() => { const toggleOpenModal = () => {
setOpenDownloadModal(prev => !prev) setOpenDownloadModal(prev => !prev)
}, []) }
const consentService = useMemo( const consentService = useMemo(
() => (isGRDF ? new GrdfConsentService() : new SgeConsentService()), () => (isGRDF ? new GrdfConsentService() : new SgeConsentService()),
...@@ -57,19 +57,16 @@ export const Consents: React.FC<{ type: 'sge' | 'grdf' }> = ({ type }) => { ...@@ -57,19 +57,16 @@ export const Consents: React.FC<{ type: 'sge' | 'grdf' }> = ({ type }) => {
[isGRDF] [isGRDF]
) )
const handleChangePage = useCallback( const handleChangePage = (
( _event: React.MouseEvent<HTMLButtonElement, MouseEvent> | null,
_event: React.MouseEvent<HTMLButtonElement, MouseEvent> | null, newPage: number
newPage: number ) => {
) => { setPage(newPage)
setPage(newPage) }
}, const handleChangeRowsPerPage = (event: any) => {
[]
)
const handleChangeRowsPerPage = useCallback((event: any) => {
setRowsPerPage(event.target.value) setRowsPerPage(event.target.value)
setPage(0) setPage(0)
}, []) }
const checkSelectedNodes = useCallback(() => { const checkSelectedNodes = useCallback(() => {
if (gridApi && !gridApi.isDestroyed()) { if (gridApi && !gridApi.isDestroyed()) {
......
...@@ -100,9 +100,9 @@ const Newsletter: React.FC = () => { ...@@ -100,9 +100,9 @@ const Newsletter: React.FC = () => {
setIsTouched(false) setIsTouched(false)
} }
} }
const handleCancel = useCallback(() => { const handleCancel = () => {
setRefreshData(true) setRefreshData(true)
}, []) }
const handleDeleteMailSubject = async (): Promise<void> => { const handleDeleteMailSubject = async (): Promise<void> => {
if (user) { if (user) {
...@@ -110,7 +110,7 @@ const Newsletter: React.FC = () => { ...@@ -110,7 +110,7 @@ const Newsletter: React.FC = () => {
date, date,
getAxiosXSRFHeader(user.xsrftoken) getAxiosXSRFHeader(user.xsrftoken)
) )
setRefreshData(true) handleCancel()
} }
} }
const handleDeleteMonthlyInfo = async (): Promise<void> => { const handleDeleteMonthlyInfo = async (): Promise<void> => {
...@@ -119,7 +119,7 @@ const Newsletter: React.FC = () => { ...@@ -119,7 +119,7 @@ const Newsletter: React.FC = () => {
date, date,
getAxiosXSRFHeader(user.xsrftoken) getAxiosXSRFHeader(user.xsrftoken)
) )
setRefreshData(true) handleCancel()
} }
} }
const handleDeleteMonthlyNews = async (): Promise<void> => { const handleDeleteMonthlyNews = async (): Promise<void> => {
...@@ -128,7 +128,7 @@ const Newsletter: React.FC = () => { ...@@ -128,7 +128,7 @@ const Newsletter: React.FC = () => {
date, date,
getAxiosXSRFHeader(user.xsrftoken) getAxiosXSRFHeader(user.xsrftoken)
) )
setRefreshData(true) handleCancel()
} }
} }
const handleDeletePoll = async (): Promise<void> => { const handleDeletePoll = async (): Promise<void> => {
...@@ -137,7 +137,7 @@ const Newsletter: React.FC = () => { ...@@ -137,7 +137,7 @@ const Newsletter: React.FC = () => {
date, date,
getAxiosXSRFHeader(user.xsrftoken) getAxiosXSRFHeader(user.xsrftoken)
) )
setRefreshData(true) handleCancel()
} }
} }
const handleOpenDeleteModal = (target: ContentItems) => { const handleOpenDeleteModal = (target: ContentItems) => {
......
...@@ -39,14 +39,14 @@ const PriceSection: React.FC<PriceSectionProps> = ({ fluid, frequency }) => { ...@@ -39,14 +39,14 @@ const PriceSection: React.FC<PriceSectionProps> = ({ fluid, frequency }) => {
}) })
const maxPerList = 8 const maxPerList = 8
const handlePriceSelection = useCallback((val: string) => { const handlePriceSelection = (val: string) => {
if (val === '') val = '0' if (val === '') val = '0'
val = val.replace(/,/g, '.') val = val.replace(/,/g, '.')
val = val.replace(/([^0-9.]+)/, '') val = val.replace(/([^0-9.]+)/, '')
setPriceToSave(prev => { setPriceToSave(prev => {
return { ...prev, price: val } return { ...prev, price: val }
}) })
}, []) }
const savePrice = useCallback(async () => { const savePrice = useCallback(async () => {
if ( if (
...@@ -68,20 +68,20 @@ const PriceSection: React.FC<PriceSectionProps> = ({ fluid, frequency }) => { ...@@ -68,20 +68,20 @@ const PriceSection: React.FC<PriceSectionProps> = ({ fluid, frequency }) => {
} }
}, [priceToSave, user]) }, [priceToSave, user])
const toggleHistory = useCallback(() => { const toggleHistory = () => {
setShowHistory(prev => !prev) setShowHistory(prev => !prev)
}, []) }
const getDate = useCallback((isoString: string): string => { const getDate = (isoString: string): string => {
const date = new Date(isoString) const date = new Date(isoString)
const month = date.toLocaleString('fr', { month: 'long' }) const month = date.toLocaleString('fr', { month: 'long' })
const year = date.toLocaleString('fr', { year: 'numeric' }) const year = date.toLocaleString('fr', { year: 'numeric' })
return `${month} ${year}` return `${month} ${year}`
}, []) }
const toggleFullList = useCallback(() => { const toggleFullList = () => {
setShowFullList(prev => !prev) setShowFullList(prev => !prev)
}, []) }
useEffect(() => { useEffect(() => {
let subscribed = true let subscribed = true
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment