diff --git a/app/src/main/java/io/legado/app/ui/book/info/BookInfoActivity.kt b/app/src/main/java/io/legado/app/ui/book/info/BookInfoActivity.kt
index 4d74cc0bb..aa18be7c8 100644
--- a/app/src/main/java/io/legado/app/ui/book/info/BookInfoActivity.kt
+++ b/app/src/main/java/io/legado/app/ui/book/info/BookInfoActivity.kt
@@ -255,7 +255,7 @@ class BookInfoActivity : VMBaseActivity<BookInfoViewModel>(R.layout.activity_boo
         viewModel.groupData.postValue(group)
         viewModel.bookData.value?.group = group.groupId
         if (viewModel.inBookshelf) {
-            viewModel.saveBook { }
+            viewModel.saveBook()
         }
     }
 
diff --git a/app/src/main/java/io/legado/app/ui/book/info/BookInfoViewModel.kt b/app/src/main/java/io/legado/app/ui/book/info/BookInfoViewModel.kt
index 7673458de..bace352ff 100644
--- a/app/src/main/java/io/legado/app/ui/book/info/BookInfoViewModel.kt
+++ b/app/src/main/java/io/legado/app/ui/book/info/BookInfoViewModel.kt
@@ -148,7 +148,7 @@ class BookInfoViewModel(application: Application) : BaseViewModel(application) {
         }
     }
 
-    fun saveBook(success: (() -> Unit)?) {
+    fun saveBook(success: (() -> Unit)? = null) {
         execute {
             bookData.value?.let { book ->
                 App.db.bookDao().insert(book)