From 9432e43c14333f70c0d3c3c8f3595fd1b0889337 Mon Sep 17 00:00:00 2001
From: Gauthier Roebroeck <gauthier.roebroeck@gmail.com>
Date: Fri, 17 Oct 2025 09:48:55 +0800
Subject: [PATCH] import books simplify

---
 .../import/books/TransientBooksTable.vue      | 59 +++++++++----------
 1 file changed, 28 insertions(+), 31 deletions(-)

diff --git a/next-ui/src/components/import/books/TransientBooksTable.vue b/next-ui/src/components/import/books/TransientBooksTable.vue
index fcf496d77..85de20696 100644
--- a/next-ui/src/components/import/books/TransientBooksTable.vue
+++ b/next-ui/src/components/import/books/TransientBooksTable.vue
@@ -520,48 +520,51 @@ function fileNamePicked(name: string) {
 //endregion
 
 function analyzeBook(book: BookImport) {
-  const { refresh } = useQuery(transientBookAnalyze, () => ({
+  void useQuery(transientBookAnalyze, () => ({
     transientBookId: book.transientBook.id,
   }))
-  void refresh().then(({ data }) => {
-    if (data) {
-      book.transientBook = data
-      if (book.transientBook.seriesId && book.transientBook.seriesId !== book.series?.id)
-        fetchSeries(book)
-    }
-  })
+    .refresh()
+    .then(({ data }) => {
+      if (data) {
+        book.transientBook = data
+        if (book.transientBook.seriesId && book.transientBook.seriesId !== book.series?.id)
+          fetchSeries(book)
+      }
+    })
 }
 
 function fetchSeries(book: BookImport) {
-  const { refresh } = useQuery(seriesDetailQuery, () => ({
+  void useQuery(seriesDetailQuery, () => ({
     seriesId: book.transientBook.seriesId!,
   }))
-  void refresh().then(({ data }) => {
-    if (data) {
-      assignSeries(book, data)
-    }
-  })
+    .refresh()
+    .then(({ data }) => {
+      if (data) assignSeries(book, data)
+    })
 }
 
 function fetchBooks(book: BookImport) {
-  const { refresh } = useQuery(bookListQuery, () => ({
+  void useQuery(bookListQuery, () => ({
     search: {
       condition: {
         seriesId: { operator: 'Is', value: book.series!.id },
       },
     } as components['schemas']['BookSearch'],
   }))
-  void refresh().then(({ data }) => {
-    if (data) {
-      book.seriesBooks = data.content
-      if (book.transientBook.number) assignBookNumber(book, book.transientBook.number)
-    }
-  })
+    .refresh()
+    .then(({ data }) => {
+      if (data) {
+        book.seriesBooks = data.content
+        if (book.series?.oneshot) book.upgradeBook = data.content?.at(0)
+        else if (book.transientBook.number) assignBookNumber(book, book.transientBook.number)
+      }
+    })
 }
 
 function assignSeries(book: BookImport, series: components['schemas']['SeriesDto']) {
   book.series = series
   fetchBooks(book)
+  // auto-select importable books
   if (book.importable && !selectedBookIds.value.includes(book.transientBook.id))
     selectedBookIds.value.push(book.transientBook.id)
 }
@@ -574,15 +577,12 @@ function unassignBook(book: BookImport) {
   book.upgradeBook = undefined
 }
 
-const importing = ref<boolean>(false)
+const { mutateAsync: postImportBooks, isLoading: importing } = useMutation({
+  mutation: () => komgaClient.POST('/api/v1/books/import', { body: importBatch.value }),
+})
 
 function doImportBooks() {
-  importing.value = true
-  const { mutateAsync } = useMutation({
-    mutation: () => komgaClient.POST('/api/v1/books/import', { body: importBatch.value }),
-  })
-
-  mutateAsync()
+  postImportBooks()
     .then(() => {
       selectedImportableBooks.value.forEach((it) => {
         it.imported = true
@@ -597,9 +597,6 @@ function doImportBooks() {
           (error?.cause as ErrorCause)?.message || intl.formatMessage(commonMessages.networkError),
       })
     })
-    .finally(() => {
-      importing.value = false
-    })
 }
 </script>