Reverted to commit dc3c210e40

src/components/moderation/ItemEditDialog.tsx

@@ -0,0 +1,215 @@
+import { useState } from 'react';
+import { Dialog, DialogContent, DialogHeader, DialogTitle } from '@/components/ui/dialog';
+import { Sheet, SheetContent, SheetHeader, SheetTitle } from '@/components/ui/sheet';
+import { Input } from '@/components/ui/input';
+import { Label } from '@/components/ui/label';
+import { Button } from '@/components/ui/button';
+import { ParkForm } from '@/components/admin/ParkForm';
+import { RideForm } from '@/components/admin/RideForm';
+import { ManufacturerForm } from '@/components/admin/ManufacturerForm';
+import { DesignerForm } from '@/components/admin/DesignerForm';
+import { OperatorForm } from '@/components/admin/OperatorForm';
+import { PropertyOwnerForm } from '@/components/admin/PropertyOwnerForm';
+import { RideModelForm } from '@/components/admin/RideModelForm';
+import { PhotoCaptionEditor } from '@/components/upload/PhotoCaptionEditor';
+import { useIsMobile } from '@/hooks/use-mobile';
+import { editSubmissionItem } from '@/lib/submissionItemsService';
+import { useAuth } from '@/hooks/useAuth';
+import { toast } from '@/hooks/use-toast';
+import type { SubmissionItemWithDeps } from '@/lib/submissionItemsService';
+
+interface ItemEditDialogProps {
+  open: boolean;
+  onOpenChange: (open: boolean) => void;
+  item: SubmissionItemWithDeps | null;
+  onEditComplete: () => void;
+}
+
+export function ItemEditDialog({ open, onOpenChange, item, onEditComplete }: ItemEditDialogProps) {
+  const isMobile = useIsMobile();
+  const { user } = useAuth();
+  const [isSubmitting, setIsSubmitting] = useState(false);
+
+  if (!item) return null;
+
+  const handleFormSubmit = async (data: any) => {
+    if (!user) {
+      toast({
+        title: "Authentication Required",
+        description: "You must be logged in to edit items.",
+        variant: "destructive"
+      });
+      return;
+    }
+
+    setIsSubmitting(true);
+    try {
+      await editSubmissionItem(item.id, data, user.id);
+      
+      toast({
+        title: "Item Updated",
+        description: "The submission item has been updated successfully."
+      });
+      
+      onOpenChange(false);
+      onEditComplete();
+    } catch (error: any) {
+      toast({
+        title: "Error",
+        description: error.message || "Failed to update submission item.",
+        variant: "destructive"
+      });
+    } finally {
+      setIsSubmitting(false);
+    }
+  };
+
+  const handleCancel = () => {
+    if (!isSubmitting) {
+      onOpenChange(false);
+    }
+  };
+
+  const renderForm = () => {
+    const itemData = item.item_data as any;
+
+    switch (item.item_type) {
+      case 'park':
+        return (
+          <ParkForm
+            initialData={itemData}
+            onSubmit={handleFormSubmit}
+            onCancel={handleCancel}
+            isEditing={true}
+          />
+        );
+
+      case 'ride':
+        return (
+          <RideForm
+            initialData={itemData}
+            onSubmit={handleFormSubmit}
+            onCancel={handleCancel}
+            isEditing={true}
+          />
+        );
+
+      case 'manufacturer':
+        return (
+          <ManufacturerForm
+            initialData={itemData}
+            onSubmit={handleFormSubmit}
+            onCancel={handleCancel}
+          />
+        );
+
+      case 'designer':
+        return (
+          <DesignerForm
+            initialData={itemData}
+            onSubmit={handleFormSubmit}
+            onCancel={handleCancel}
+          />
+        );
+
+      case 'operator':
+        return (
+          <OperatorForm
+            initialData={itemData}
+            onSubmit={handleFormSubmit}
+            onCancel={handleCancel}
+          />
+        );
+
+      case 'property_owner':
+        return (
+          <PropertyOwnerForm
+            initialData={itemData}
+            onSubmit={handleFormSubmit}
+            onCancel={handleCancel}
+          />
+        );
+
+      case 'ride_model':
+        return (
+          <RideModelForm
+            manufacturerName={itemData.manufacturer_name || 'Unknown'}
+            manufacturerId={itemData.manufacturer_id}
+            initialData={itemData}
+            onSubmit={handleFormSubmit}
+            onCancel={handleCancel}
+          />
+        );
+
+      case 'photo':
+        return (
+          <div className="space-y-4 p-4">
+            <div className="space-y-2">
+              <Label htmlFor="caption">Caption</Label>
+              <Input
+                id="caption"
+                defaultValue={itemData.caption || ''}
+                onChange={(e) => {
+                  itemData.caption = e.target.value;
+                }}
+                placeholder="Enter photo caption"
+              />
+            </div>
+            {itemData.image_url && (
+              <img 
+                src={itemData.image_url} 
+                alt="Preview" 
+                className="w-full rounded-md"
+              />
+            )}
+            <div className="flex gap-2 justify-end">
+              <Button variant="outline" onClick={handleCancel}>Cancel</Button>
+              <Button onClick={() => handleFormSubmit(itemData)}>Save</Button>
+            </div>
+          </div>
+        );
+
+      default:
+        return (
+          <div className="p-4 text-center text-muted-foreground">
+            Form not available for item type: {item.item_type}
+          </div>
+        );
+    }
+  };
+
+  const content = (
+    <>
+      {isMobile ? (
+        <SheetHeader>
+          <SheetTitle>Edit {item.item_type.replace('_', ' ')}</SheetTitle>
+        </SheetHeader>
+      ) : (
+        <DialogHeader>
+          <DialogTitle>Edit {item.item_type.replace('_', ' ')}</DialogTitle>
+        </DialogHeader>
+      )}
+      <div className="mt-4 max-h-[calc(100vh-150px)] overflow-y-auto">
+        {renderForm()}
+      </div>
+    </>
+  );
+
+  if (isMobile) {
+    return (
+      <Sheet open={open} onOpenChange={onOpenChange}>
+        <SheetContent side="bottom" className="h-[95vh]">
+          {content}
+        </SheetContent>
+      </Sheet>
+    );
+  }
+
+  return (
+    <Dialog open={open} onOpenChange={onOpenChange}>
+      <DialogContent className="max-w-4xl max-h-[90vh] overflow-y-auto">
+        {content}
+      </DialogContent>
+    </Dialog>
+  );
+}

src/components/moderation/ItemReviewCard.tsx

@@ -141,6 +141,11 @@ export function ItemReviewCard({ item, onEdit, onStatusChange }: ItemReviewCardP
                 <Edit className="w-3 h-3" />
               </Button>
             )}
+            {item.original_data && (
+              <Badge variant="secondary" className="text-xs">
+                Edited
+              </Badge>
+            )}
           </div>
         </div>
       </CardHeader>

src/components/moderation/SubmissionReviewManager.tsx

@@ -27,6 +27,7 @@ import { DependencyVisualizer } from './DependencyVisualizer';
 import { ConflictResolutionDialog } from './ConflictResolutionDialog';
 import { EscalationDialog } from './EscalationDialog';
 import { RejectionDialog } from './RejectionDialog';
+import { ItemEditDialog } from './ItemEditDialog';
 
 interface SubmissionReviewManagerProps {
   submissionId: string;
@@ -48,6 +49,8 @@ export function SubmissionReviewManager({
   const [showConflictDialog, setShowConflictDialog] = useState(false);
   const [showEscalationDialog, setShowEscalationDialog] = useState(false);
   const [showRejectionDialog, setShowRejectionDialog] = useState(false);
+  const [showEditDialog, setShowEditDialog] = useState(false);
+  const [editingItem, setEditingItem] = useState<SubmissionItemWithDeps | null>(null);
   const [activeTab, setActiveTab] = useState<'items' | 'dependencies'>('items');
   
   const { toast } = useToast();
@@ -240,6 +243,12 @@ export function SubmissionReviewManager({
     }
   };
 
+  const handleEditComplete = async () => {
+    // Reload items after edit
+    await loadSubmissionItems();
+    setEditingItem(null);
+  };
+
   const pendingCount = items.filter(item => item.status === 'pending').length;
   const selectedCount = selectedItemIds.size;
 
@@ -292,6 +301,13 @@ export function SubmissionReviewManager({
         )}
         onReject={handleReject}
       />
+
+      <ItemEditDialog
+        open={showEditDialog}
+        onOpenChange={setShowEditDialog}
+        item={editingItem}
+        onEditComplete={handleEditComplete}
+      />
     </>
   );
 
@@ -331,7 +347,10 @@ export function SubmissionReviewManager({
                     />
                     <ItemReviewCard
                       item={item}
-                      onEdit={() => {/* TODO: Implement editing */}}
+                      onEdit={() => {
+                        setEditingItem(item);
+                        setShowEditDialog(true);
+                      }}
                       onStatusChange={(status) => {/* TODO: Update status */}}
                     />
                   </div>

src/lib/submissionItemsService.ts

@@ -357,6 +357,102 @@ export async function resolveConflicts(
   return { updatedItems, newConflicts };
 }
 
+/**
+ * Edit a submission item
+ * Moderators can edit directly, regular users trigger auto-escalation
+ */
+export async function editSubmissionItem(
+  itemId: string,
+  newData: any,
+  userId: string
+): Promise<void> {
+  // Check user permissions
+  const { data: userRoles } = await supabase
+    .from('user_roles')
+    .select('role')
+    .eq('user_id', userId);
+
+  const isModerator = userRoles?.some(r => 
+    ['moderator', 'admin', 'superuser'].includes(r.role)
+  );
+
+  // Get current item
+  const { data: currentItem, error: fetchError } = await supabase
+    .from('submission_items')
+    .select('*')
+    .eq('id', itemId)
+    .single();
+
+  if (fetchError || !currentItem) {
+    throw new Error('Failed to fetch submission item');
+  }
+
+  // Preserve original data if not already saved
+  const originalData = currentItem.original_data || currentItem.item_data;
+
+  if (isModerator) {
+    // Moderators can edit directly
+    const { error } = await supabase
+      .from('submission_items')
+      .update({
+        item_data: newData,
+        original_data: originalData,
+        updated_at: new Date().toISOString()
+      })
+      .eq('id', itemId);
+
+    if (error) {
+      console.error('Error updating submission item:', error);
+      throw new Error(`Failed to update item: ${error.message}`);
+    }
+
+    // Log the edit
+    await supabase
+      .from('admin_audit_log')
+      .insert({
+        admin_user_id: userId,
+        target_user_id: currentItem.submission_id,
+        action: 'edit_submission_item',
+        details: {
+          item_id: itemId,
+          item_type: currentItem.item_type,
+          changes: newData
+        }
+      });
+  } else {
+    // Regular users trigger auto-escalation
+    const { data: submission } = await supabase
+      .from('content_submissions')
+      .select('id, status')
+      .eq('id', currentItem.submission_id)
+      .single();
+
+    if (submission && submission.status !== 'escalated') {
+      await escalateSubmission(
+        currentItem.submission_id,
+        `User requested edit for ${currentItem.item_type} item`,
+        userId
+      );
+    }
+
+    // Update item with edit request
+    const { error } = await supabase
+      .from('submission_items')
+      .update({
+        item_data: newData,
+        original_data: originalData,
+        status: 'pending',
+        updated_at: new Date().toISOString()
+      })
+      .eq('id', itemId);
+
+    if (error) {
+      console.error('Error updating submission item:', error);
+      throw new Error(`Failed to update item: ${error.message}`);
+    }
+  }
+}
+
 /**
  * Update individual submission item status
  */