From 928136ff18f9d70e19e1ec91c8171e5af3cdfef0 Mon Sep 17 00:00:00 2001 From: rstubryan Date: Fri, 6 Mar 2026 14:29:21 +0700 Subject: [PATCH] refactor(FE): Refactor handleViewUniformityDetails to inline definition --- .../uniformity/detail/UniformityDetail.tsx | 47 +++++++++++-------- 1 file changed, 27 insertions(+), 20 deletions(-) diff --git a/src/components/pages/production/uniformity/detail/UniformityDetail.tsx b/src/components/pages/production/uniformity/detail/UniformityDetail.tsx index adda86a1..244201fa 100644 --- a/src/components/pages/production/uniformity/detail/UniformityDetail.tsx +++ b/src/components/pages/production/uniformity/detail/UniformityDetail.tsx @@ -60,25 +60,6 @@ const UniformityDetail: React.FC = ({ router.push(`/production/uniformity?action=reject&id=${initialValues.id}`); }; - const handleViewUniformityDetails = () => { - if (!uniformity_details || uniformity_details.length === 0) { - setShouldFetchDetails(true); - return; - } - - setExpandedDrawerContent( - - ); - - setTimeout(() => { - setExpandedDrawerOpen(true); - }, 0); - }; - useEffect(() => { if ( shouldFetchDetails && @@ -202,6 +183,25 @@ const UniformityDetail: React.FC = ({ } if (id === 'document-name') { + const handleViewUniformityDetails = () => { + if (!uniformity_details || uniformity_details.length === 0) { + setShouldFetchDetails(true); + return; + } + + setExpandedDrawerContent( + + ); + + setTimeout(() => { + setExpandedDrawerOpen(true); + }, 0); + }; + return (
{valueMap[id]} @@ -231,7 +231,14 @@ const UniformityDetail: React.FC = ({ }, }, ], - [initialValues, handleViewUniformityDetails, isLoading] + [ + initialValues, + isLoading, + uniformity_details, + setShouldFetchDetails, + setExpandedDrawerContent, + setExpandedDrawerOpen, + ] ); const samplingTableData: DetailOptionType[] = useMemo(() => {