Perms: Fixed some issues made when adding transactions

[bookstack] / app / Entities / Repos / BookRepo.php

diff --git a/app/Entities/Repos/BookRepo.php b/app/Entities/Repos/BookRepo.php
index 91bc9a1b41eebafb74cfe8aeaae1ff0344528d86..6d28d5d6aabe1796a0f219d641dc5abf816badd3 100644 (file)
--- a/app/Entities/Repos/BookRepo.php
+++ b/app/Entities/Repos/BookRepo.php
@@ -1,46 +1,93 @@
 <?php
 
 <?php
 

-

 namespace BookStack\Entities\Repos;
 
 namespace BookStack\Entities\Repos;
 

-use BookStack\Entities\Book;
-use BookStack\Exceptions\NotFoundException;
-use BookStack\Exceptions\NotifyException;
+use BookStack\Activity\ActivityType;
+use BookStack\Activity\TagRepo;
+use BookStack\Entities\Models\Book;
+use BookStack\Entities\Tools\TrashCan;
+use BookStack\Exceptions\ImageUploadException;
+use BookStack\Facades\Activity;
+use BookStack\Sorting\SortRule;
+use BookStack\Uploads\ImageRepo;
+use BookStack\Util\DatabaseTransaction;
+use Exception;
+use Illuminate\Http\UploadedFile;

 
 

-class BookRepo extends EntityRepo
+class BookRepo

 {
 {

+    public function __construct(
+        protected BaseRepo $baseRepo,
+        protected TagRepo $tagRepo,
+        protected ImageRepo $imageRepo,
+        protected TrashCan $trashCan,
+    ) {
+    }

 
     /**
 
     /**

-     * Fetch a book by its slug.
-     * @param string $slug
-     * @return Book
-     * @throws NotFoundException
+     * Create a new book in the system.

      */
      */

-    public function getBySlug(string $slug): Book
+    public function create(array $input): Book

     {
     {

-        /** @var Book $book */
-        $book = $this->getEntityBySlug('book', $slug);
-        return $book;
+        return (new DatabaseTransaction(function () use ($input) {
+            $book = new Book();
+
+            $this->baseRepo->create($book, $input);
+            $this->baseRepo->updateCoverImage($book, $input['image'] ?? null);
+            $this->baseRepo->updateDefaultTemplate($book, intval($input['default_template_id'] ?? null));
+            Activity::add(ActivityType::BOOK_CREATE, $book);
+
+            $defaultBookSortSetting = intval(setting('sorting-book-default', '0'));
+            if ($defaultBookSortSetting && SortRule::query()->find($defaultBookSortSetting)) {
+                $book->sort_rule_id = $defaultBookSortSetting;
+                $book->save();
+            }
+
+            return $book;
+        }))->run();

     }
 
     /**
     }
 
     /**

-     * Destroy the provided book and all its child entities.
-     * @param Book $book
-     * @throws NotifyException
-     * @throws \Throwable
+     * Update the given book.

      */
      */

-    public function destroyBook(Book $book)
+    public function update(Book $book, array $input): Book

     {
     {

-        foreach ($book->pages as $page) {
-            $this->destroyPage($page);
+        $this->baseRepo->update($book, $input);
+
+        if (array_key_exists('default_template_id', $input)) {
+            $this->baseRepo->updateDefaultTemplate($book, intval($input['default_template_id']));

         }
 
         }
 

-        foreach ($book->chapters as $chapter) {
-            $this->destroyChapter($chapter);
+        if (array_key_exists('image', $input)) {
+            $this->baseRepo->updateCoverImage($book, $input['image'], $input['image'] === null);

         }
 
         }
 

-        $this->destroyEntityCommonRelations($book);
-        $book->delete();
+        Activity::add(ActivityType::BOOK_UPDATE, $book);
+
+        return $book;

     }
 
     }
 

-}
\ No newline at end of file
+    /**
+     * Update the given book's cover image, or clear it.
+     *
+     * @throws ImageUploadException
+     * @throws Exception
+     */
+    public function updateCoverImage(Book $book, ?UploadedFile $coverImage, bool $removeImage = false)
+    {
+        $this->baseRepo->updateCoverImage($book, $coverImage, $removeImage);
+    }
+
+    /**
+     * Remove a book from the system.
+     *
+     * @throws Exception
+     */
+    public function destroy(Book $book)
+    {
+        $this->trashCan->softDestroyBook($book);
+        Activity::add(ActivityType::BOOK_DELETE, $book);
+
+        $this->trashCan->autoClearOld();
+    }
+}