fix: properly refresh photo state after setting primary photo

Author: atomantic

client/src/components/person/PersonDetail.tsx

@@ -409,6 +409,24 @@ export function PersonDetail() {
     setClaims(newClaims);
   }, [dbId, personId]);
 
+  // Refresh photo state after setting a new primary photo
+  const refreshPhotoState = useCallback(async () => {
+    if (!personId) return;
+    const [photoCheck, wikiPhotoCheck, ancestryPhotoCheck, wikiTreePhotoCheck, linkedInPhotoCheck] = await Promise.all([
+      api.hasPhoto(personId).catch(() => ({ exists: false })),
+      api.hasWikiPhoto(personId).catch(() => ({ exists: false })),
+      api.hasAncestryPhoto(personId).catch(() => ({ exists: false })),
+      api.hasWikiTreePhoto(personId).catch(() => ({ exists: false })),
+      api.hasLinkedInPhoto(personId).catch(() => ({ exists: false })),
+    ]);
+    setHasPhoto(photoCheck?.exists ?? false);
+    setHasFsPhoto((photoCheck as { exists: boolean; fsExists?: boolean })?.fsExists ?? false);
+    setHasWikiPhoto(wikiPhotoCheck?.exists ?? false);
+    setHasAncestryPhoto(ancestryPhotoCheck?.exists ?? false);
+    setHasWikiTreePhoto(wikiTreePhotoCheck?.exists ?? false);
+    setHasLinkedInPhoto(linkedInPhotoCheck?.exists ?? false);
+  }, [personId]);
+
   const handleSavePersonField = useCallback(async (fieldName: string, value: string, originalValue: string | null) => {
     if (!dbId || !personId) return;
     await api.setPersonOverride(dbId, personId, {
@@ -1032,6 +1050,7 @@ export function PersonDetail() {
             onShowUploadDialog={() => setShowUploadDialog(true)}
             onShowAncestryUploadDialog={() => setShowAncestryUploadDialog(true)}
             onShowLinkInput={(platform) => setLinkingPlatform(platform)}
+            onPhotoChanged={refreshPhotoState}
             syncLoading={syncLoading}
             scrapeLoading={scrapeLoading}
             fetchingPhotoFrom={fetchingPhotoFrom}

client/src/components/person/ProviderDataTable.tsx

@@ -34,6 +34,7 @@ interface ProviderDataTableProps {
   onShowUploadDialog: () => void;
   onShowAncestryUploadDialog: () => void;
   onShowLinkInput: (platform: 'wikipedia' | 'ancestry' | 'wikitree' | 'linkedin') => void;
+  onPhotoChanged?: () => void;  // Called when primary photo changes to refresh parent state
   syncLoading: boolean;
   scrapeLoading: boolean;
   fetchingPhotoFrom: string | null;
@@ -137,6 +138,7 @@ export function ProviderDataTable({
   onShowUploadDialog,
   onShowAncestryUploadDialog,
   onShowLinkInput,
+  onPhotoChanged,
   syncLoading,
   scrapeLoading,
   fetchingPhotoFrom,
@@ -280,8 +282,8 @@ export function ProviderDataTable({
 
     if (result) {
       toast.success(`Set ${PROVIDER_INFO[provider]?.name || provider} photo as primary`);
-      // Trigger photo refresh in parent component by fetching updated photo
-      await onFetchPhoto(provider).catch(() => null);
+      // Notify parent to refresh photo state
+      onPhotoChanged?.();
     }
 
     setApplyingField(null);