From 511423a6b668d90525091f13e4d1c51575374063 Mon Sep 17 00:00:00 2001
From: Marco Pedone <m.pedone98@gmail.com>
Date: Tue, 30 Dec 2025 17:25:37 +0100
Subject: [PATCH] refactor: simplify document step logic in
 ContattiProprietarioModal

---
 .../components/servizio/modale_contatti.tsx   | 27 +++++--------------
 1 file changed, 6 insertions(+), 21 deletions(-)

diff --git a/apps/infoalloggi/src/components/servizio/modale_contatti.tsx b/apps/infoalloggi/src/components/servizio/modale_contatti.tsx
index 81a4b24..5abf491 100644
--- a/apps/infoalloggi/src/components/servizio/modale_contatti.tsx
+++ b/apps/infoalloggi/src/components/servizio/modale_contatti.tsx
@@ -28,20 +28,17 @@ export const ContattiProprietarioModal = () => {
 	const { data } = useServizioAnnuncio();
 	const { t } = useTranslation();
 	const isUnlocked = data.open_contatti_at != null;
+
 	const needsDocuments =
 		!servizio.skipControlloDoc &&
 		(!servizio.doc_personale_fronte || !servizio.doc_personale_retro);
-	const [showDocStep, setShowDocStep] = useState(
-		open && (needsDocuments || !isUnlocked),
-	);
+
+	console.log("needsDocuments", needsDocuments, servizio);
 
 	return (
 		<Credenza
 			onOpenChange={(o) => {
 				setOpen(o);
-				if (o && (needsDocuments || !isUnlocked)) {
-					setShowDocStep(true);
-				}
 			}}
 			open={open}
 		>
@@ -73,8 +70,8 @@ export const ContattiProprietarioModal = () => {
 						{t.contatto.descrizione}
 					</CredenzaDescription>
 				</CredenzaHeader>
-				{showDocStep ? (
-					<DocCheckpoint setStep={setShowDocStep} />
+				{needsDocuments ? (
+					<DocCheckpoint />
 				) : isUnlocked ? (
 					<PostUnlock />
 				) : (
@@ -85,7 +82,7 @@ export const ContattiProprietarioModal = () => {
 	);
 };
 
-const DocCheckpoint = ({ setStep }: { setStep: (value: boolean) => void }) => {
+const DocCheckpoint = () => {
 	const { t } = useTranslation();
 	const { userId, servizio, isAdmin } = useServizio();
 	const utils = api.useUtils();
@@ -170,18 +167,6 @@ const DocCheckpoint = ({ setStep }: { setStep: (value: boolean) => void }) => {
 				</div>
 			</CredenzaBody>
 			<CredenzaFooter>
-				<Button
-					aria-label="Procedi"
-					disabled={
-						isPending ||
-						!servizio.doc_personale_fronte?.storageId ||
-						!servizio.doc_personale_retro?.storageId
-					}
-					onClick={() => setStep(false)}
-					variant="success"
-				>
-					Procedi
-				</Button>
 				<CredenzaClose asChild>
 					<Button aria-label="Chiudi" disabled={isPending} variant="outline">
 						{t.chiudi}