From f9778dd9d93b01a7b62ff06fa57696c6352882b8 Mon Sep 17 00:00:00 2001
From: "gpt-engineer-app[bot]"
 <159125892+gpt-engineer-app[bot]@users.noreply.github.com>
Date: Mon, 29 Sep 2025 00:06:57 +0000
Subject: [PATCH] Fix submission deletion and preserve filters

---
 src/components/moderation/ModerationQueue.tsx | 27 +++++++++++++++----
 src/pages/Admin.tsx                           | 14 +++++-----
 2 files changed, 29 insertions(+), 12 deletions(-)

diff --git a/src/components/moderation/ModerationQueue.tsx b/src/components/moderation/ModerationQueue.tsx
index 46e2f2a9..b5a14579 100644
--- a/src/components/moderation/ModerationQueue.tsx
+++ b/src/components/moderation/ModerationQueue.tsx
@@ -1,4 +1,4 @@
-import { useState, useEffect } from 'react';
+import { useState, useEffect, useImperativeHandle, forwardRef } from 'react';
 import { CheckCircle, XCircle, Eye, Calendar, User, Filter, MessageSquare, FileText, Image, X, Trash2 } from 'lucide-react';
 import { Button } from '@/components/ui/button';
 import { Badge } from '@/components/ui/badge';
@@ -28,7 +28,11 @@ interface ModerationItem {
 type EntityFilter = 'all' | 'reviews' | 'submissions' | 'photos';
 type StatusFilter = 'all' | 'pending' | 'flagged' | 'approved' | 'rejected';
 
-export function ModerationQueue() {
+export interface ModerationQueueRef {
+  refresh: () => void;
+}
+
+export const ModerationQueue = forwardRef<ModerationQueueRef>((props, ref) => {
   const [items, setItems] = useState<ModerationItem[]>([]);
   const [loading, setLoading] = useState(true);
   const [actionLoading, setActionLoading] = useState<string | null>(null);
@@ -38,6 +42,13 @@ export function ModerationQueue() {
   const { toast } = useToast();
   const { isAdmin, isSuperuser } = useUserRole();
 
+  // Expose refresh method via ref
+  useImperativeHandle(ref, () => ({
+    refresh: () => {
+      fetchItems(activeEntityFilter, activeStatusFilter);
+    }
+  }), [activeEntityFilter, activeStatusFilter]);
+
   const fetchItems = async (entityFilter: EntityFilter = 'all', statusFilter: StatusFilter = 'pending') => {
     try {
       setLoading(true);
@@ -292,8 +303,14 @@ export function ModerationQueue() {
       const photoIds: string[] = [];
       if (item.content?.photos && Array.isArray(item.content.photos)) {
         for (const photo of item.content.photos) {
-          if (photo.imageId) {
-            photoIds.push(photo.imageId);
+          if (photo.url) {
+            // Extract UUID from blob URL: blob:https://domain/[uuid]
+            const urlParts = photo.url.split('/');
+            const imageId = urlParts[urlParts.length - 1];
+            // Basic UUID validation (should be at least 32 characters)
+            if (imageId && imageId.length >= 32) {
+              photoIds.push(imageId);
+            }
           }
         }
       }
@@ -755,4 +772,4 @@ export function ModerationQueue() {
       <QueueContent />
     </div>
   );
-}
\ No newline at end of file
+});
\ No newline at end of file
diff --git a/src/pages/Admin.tsx b/src/pages/Admin.tsx
index 77957639..68b40879 100644
--- a/src/pages/Admin.tsx
+++ b/src/pages/Admin.tsx
@@ -1,4 +1,4 @@
-import { useEffect, useState, useCallback } from 'react';
+import { useRef, useEffect, useCallback } from 'react';
 import { useNavigate } from 'react-router-dom';
 import { Shield, Users, FileText, Flag, AlertCircle } from 'lucide-react';
 import { useUserRole } from '@/hooks/useUserRole';
@@ -6,7 +6,7 @@ import { useAuth } from '@/hooks/useAuth';
 import { Card, CardContent, CardDescription, CardHeader, CardTitle } from '@/components/ui/card';
 import { Badge } from '@/components/ui/badge';
 import { Tabs, TabsContent, TabsList, TabsTrigger } from '@/components/ui/tabs';
-import { ModerationQueue } from '@/components/moderation/ModerationQueue';
+import { ModerationQueue, ModerationQueueRef } from '@/components/moderation/ModerationQueue';
 import { ReportsQueue } from '@/components/moderation/ReportsQueue';
 import { UserManagement } from '@/components/admin/UserManagement';
 import { AdminHeader } from '@/components/layout/AdminHeader';
@@ -15,10 +15,10 @@ export default function Admin() {
   const { user, loading: authLoading } = useAuth();
   const { isModerator, loading: roleLoading } = useUserRole();
   const navigate = useNavigate();
-  const [refreshKey, setRefreshKey] = useState(0);
+  const moderationQueueRef = useRef<ModerationQueueRef>(null);
 
   const handleRefresh = useCallback(() => {
-    setRefreshKey(prev => prev + 1);
+    moderationQueueRef.current?.refresh();
   }, []);
 
   useEffect(() => {
@@ -128,11 +128,11 @@ export default function Admin() {
             </TabsList>
             
             <TabsContent value="queue">
-              <ModerationQueue key={`moderation-queue-${refreshKey}`} />
+              <ModerationQueue ref={moderationQueueRef} />
             </TabsContent>
             
             <TabsContent value="reports">
-              <ReportsQueue key={`reports-queue-${refreshKey}`} />
+              <ReportsQueue />
             </TabsContent>
           </Tabs>
         </CardContent>
@@ -150,7 +150,7 @@ export default function Admin() {
           </CardDescription>
         </CardHeader>
         <CardContent>
-          <UserManagement key={`user-management-${refreshKey}`} />
+          <UserManagement />
         </CardContent>
       </Card>
       </div>