From c1006f395c5c7d00c9c117d3bd6694c78969faa1 Mon Sep 17 00:00:00 2001
From: AI Assistant <ai@architools.local>
Date: Tue, 24 Mar 2026 23:23:48 +0200
Subject: [PATCH] =?UTF-8?q?fix(rgi):=20remove=20wrong=20dueDate=20lock=20?=
 =?UTF-8?q?=E2=80=94=20always=20show=20download=20button?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

The dueDate-based lock was incorrect: some documents with future
dueDate ARE downloadable. The availability depends on eTerra internal
rules, not predictably on dueDate.

Now all documents show a download button. If server-side download
fails (fileVisibility 404), it redirects to eTerra direct URL
which works in the user's browser session.

Filters changed to: Solutionate / Confirmate / Toate

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
---
 src/app/(modules)/rgi-test/page.tsx | 68 ++++++++++-------------------
 1 file changed, 22 insertions(+), 46 deletions(-)

diff --git a/src/app/(modules)/rgi-test/page.tsx b/src/app/(modules)/rgi-test/page.tsx
index a5cb90c..a215077 100644
--- a/src/app/(modules)/rgi-test/page.tsx
+++ b/src/app/(modules)/rgi-test/page.tsx
@@ -260,22 +260,12 @@ function IssuedDocsPanel({
     );
   }
 
-  const isLocked = dueDate > Date.now();
-
   return (
     <div className="space-y-1.5 py-2">
-      <div className="flex items-center gap-2 mb-2">
-        <p className="text-xs font-medium text-muted-foreground">
-          {docs.length} document{docs.length > 1 ? "e" : ""} eliberat
-          {docs.length > 1 ? "e" : ""}
-        </p>
-        {isLocked && (
-          <Badge variant="outline" className="text-[10px] border-amber-300 text-amber-600 dark:text-amber-400">
-            <Clock className="h-3 w-3 mr-0.5" />
-            Disponibile de la {fmtTs(dueDate)}
-          </Badge>
-        )}
-      </div>
+      <p className="text-xs font-medium text-muted-foreground mb-2">
+        {docs.length} document{docs.length > 1 ? "e" : ""} eliberat
+        {docs.length > 1 ? "e" : ""}:
+      </p>
       {docs.map((doc, i) => (
         <div
           key={doc.documentPk || i}
@@ -306,25 +296,16 @@ function IssuedDocsPanel({
               </div>
             </div>
           </div>
-          <div className="flex items-center gap-1 shrink-0">
-            {isLocked ? (
-              <Badge variant="secondary" className="text-[10px] text-muted-foreground">
-                <Clock className="h-3 w-3 mr-0.5" />
-                {fmtTs(dueDate)}
-              </Badge>
-            ) : (
-              <Button size="sm" variant="outline" className="gap-1" asChild>
-                <a
-                  href={`/api/eterra/rgi/download-doc?workspaceId=${doc.workspaceId || workspaceId}&applicationId=${doc.applicationId || applicationPk}&documentPk=${doc.documentPk}&documentTypeId=${doc.documentTypeId}`}
-                  target="_blank"
-                  rel="noopener noreferrer"
-                >
-                  <Download className="h-3.5 w-3.5" />
-                  Descarca
-                </a>
-              </Button>
-            )}
-          </div>
+          <Button size="sm" variant="outline" className="gap-1 shrink-0" asChild>
+            <a
+              href={`/api/eterra/rgi/download-doc?workspaceId=${doc.workspaceId || workspaceId}&applicationId=${doc.applicationId || applicationPk}&documentPk=${doc.documentPk}&documentTypeId=${doc.documentTypeId}`}
+              target="_blank"
+              rel="noopener noreferrer"
+            >
+              <Download className="h-3.5 w-3.5" />
+              Descarca
+            </a>
+          </Button>
         </div>
       ))}
     </div>
@@ -403,16 +384,14 @@ export default function RgiTestPage() {
     setLoading(false);
   }, [workspaceId, orgUnitId, year]);
 
-  const [filterMode, setFilterMode] = useState<"all" | "ready" | "pending">("ready");
+  const [filterMode, setFilterMode] = useState<"all" | "solved" | "confirmed">("solved");
 
   // Client-side filter
   const filtered = useMemo(() => {
     if (filterMode === "all") return applications;
-    const now = Date.now();
     return applications.filter((a) => {
-      if (a.hasSolution !== 1) return false;
-      if (filterMode === "ready") return a.dueDate <= now; // termen trecut = descarcabil
-      if (filterMode === "pending") return a.dueDate > now; // termen viitor = blocat
+      if (filterMode === "solved") return a.hasSolution === 1;
+      if (filterMode === "confirmed") return a.stateCode === "CONFIRMED";
       return true;
     });
   }, [applications, filterMode]);
@@ -498,8 +477,8 @@ export default function RgiTestPage() {
           <div className="flex items-center gap-3 pt-2 border-t flex-wrap">
             <div className="flex gap-1 p-0.5 bg-muted rounded-md">
               {([
-                { id: "ready" as const, label: "Descarcabile acum", desc: "solutionate + termen trecut" },
-                { id: "pending" as const, label: "In asteptare", desc: "solutionate + termen viitor (blocate)" },
+                { id: "solved" as const, label: "Solutionate", desc: "lucrari cu solutie" },
+                { id: "confirmed" as const, label: "Confirmate", desc: "solutie confirmata" },
                 { id: "all" as const, label: "Toate", desc: "" },
               ]).map((opt) => (
                 <button
@@ -572,7 +551,6 @@ export default function RgiTestPage() {
                     const pk = app.applicationPk;
                     const isExpanded = expandedPk === pk;
                     const solved = app.hasSolution === 1;
-                    const downloadable = solved && app.dueDate <= Date.now();
 
                     return (
                       <React.Fragment key={pk}>
@@ -585,11 +563,9 @@ export default function RgiTestPage() {
                             setExpandedPk(isExpanded ? null : pk)
                           }
                         >
-                          <td className="px-2 py-2.5 w-8" title={downloadable ? "Descarcabil" : solved ? `Blocat pana la ${fmtTs(app.dueDate)}` : "In lucru"}>
-                            {downloadable ? (
-                              <Download className="h-4 w-4 text-emerald-500" />
-                            ) : solved ? (
-                              <Clock className="h-4 w-4 text-amber-500" />
+                          <td className="px-2 py-2.5 w-8" title={solved ? "Solutionata" : "In lucru"}>
+                            {solved ? (
+                              <CheckCircle2 className="h-4 w-4 text-emerald-500" />
                             ) : (
                               <Clock className="h-4 w-4 text-muted-foreground" />
                             )}