fix(metadata): fixed changing cover image in Readest apps, closes #2402 (#2410)

apps/readest-app/src/components/metadata/BookDetailEdit.tsx

@@ -151,9 +151,10 @@ const BookDetailEdit: React.FC<BookDetailEditProps> = ({
       const selectedFile = result.files[0]!;
       if (selectedFile.path && appService) {
         const filePath = selectedFile.path;
+        const imageUrl = await appService.getCachedImageUrl(filePath);
         onFieldChange('coverImageFile', filePath);
-        onFieldChange('coverImageUrl', await appService.getCachedImageUrl(filePath));
-        setNewCoverImageUrl(filePath);
+        onFieldChange('coverImageUrl', imageUrl);
+        setNewCoverImageUrl(imageUrl);
       } else if (selectedFile.file) {
         const coverImageBlobUrl = URL.createObjectURL(selectedFile.file);
         onFieldChange('coverImageBlobUrl', coverImageBlobUrl);

apps/readest-app/src/components/metadata/useMetadataEdit.ts

@@ -57,24 +57,28 @@ export const useMetadataEdit = (metadata: BookMetadata | null) => {
       return;
     }
 
-    const newMeta = { ...editedMeta } as { [key: string]: unknown };
-    switch (field) {
-      case 'subject':
-        newMeta['subject'] = value ? value.split(/,|;|，|、/).map((s) => s.trim()) : [];
-        break;
-      default:
-        newMeta[field] = value;
-    }
+    setEditedMeta((prevMeta) => {
+      const newMeta = { ...prevMeta } as { [key: string]: unknown };
+      switch (field) {
+        case 'subject':
+          newMeta['subject'] = value ? value.split(/,|;|，|、/).map((s) => s.trim()) : [];
+          break;
+        default:
+          newMeta[field] = value;
+      }
+      return newMeta as BookMetadata;
+    });
 
-    setEditedMeta(newMeta as BookMetadata);
     if (value !== undefined) {
       handleFieldValidation(field, value);
     }
 
     if (fieldSources[field]) {
-      const newSources = { ...fieldSources };
-      delete newSources[field];
-      setFieldSources(newSources);
+      setFieldSources((prevSources) => {
+        const newSources = { ...prevSources };
+        delete newSources[field];
+        return newSources;
+      });
     }
   };