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 f37cf8581..129bc38af 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
@@ -144,15 +144,11 @@ class BookInfoActivity :
     override fun onCompatOptionsItemSelected(item: MenuItem): Boolean {
         when (item.itemId) {
             R.id.menu_edit -> {
-               // if (viewModel.inBookshelf) {
-                    viewModel.bookData.value?.let {
-                        infoEditResult.launch {
-                            putExtra("bookUrl", it.bookUrl)
-                        }
+                viewModel.bookData.value?.let {
+                    infoEditResult.launch {
+                        putExtra("bookUrl", it.bookUrl)
                     }
-               // } else {
-                //    toastOnUi(R.string.after_add_bookshelf)
-                //}
+                }
             }
             R.id.menu_share_it -> {
                 viewModel.bookData.value?.let {
@@ -183,13 +179,11 @@ class BookInfoActivity :
                 sendToClip(it)
             } ?: toastOnUi(R.string.no_book)
             R.id.menu_can_update -> {
-                if (viewModel.inBookshelf) {
-                    viewModel.bookData.value?.let {
-                        it.canUpdate = !it.canUpdate
+                viewModel.bookData.value?.let {
+                    it.canUpdate = !it.canUpdate
+                    if (viewModel.inBookshelf) {
                         viewModel.saveBook(it)
                     }
-                } else {
-                    toastOnUi(R.string.after_add_bookshelf)
                 }
             }
             R.id.menu_clear_cache -> viewModel.clearCache()