Modernize third party services' logos

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

diff --git a/app/Entities/Repos/ChapterRepo.php b/app/Entities/Repos/ChapterRepo.php
index c6f3a2d2f0fc093c37b6e541081c20c977468c75..68330dd57bf5a02135db0765280fccd730e09bbe 100644 (file)
--- a/app/Entities/Repos/ChapterRepo.php
+++ b/app/Entities/Repos/ChapterRepo.php
@@ -1,25 +1,23 @@
-<?php namespace BookStack\Entities\Repos;
+<?php

 
 

-use BookStack\Entities\Book;
-use BookStack\Entities\Chapter;
-use BookStack\Entities\Managers\BookContents;
-use BookStack\Entities\Managers\TrashCan;
+namespace BookStack\Entities\Repos;
+
+use BookStack\Actions\ActivityType;
+use BookStack\Entities\Models\Book;
+use BookStack\Entities\Models\Chapter;
+use BookStack\Entities\Tools\BookContents;
+use BookStack\Entities\Tools\TrashCan;

 use BookStack\Exceptions\MoveOperationException;
 use BookStack\Exceptions\NotFoundException;
 use BookStack\Exceptions\MoveOperationException;
 use BookStack\Exceptions\NotFoundException;

-use BookStack\Exceptions\NotifyException;
+use BookStack\Facades\Activity;

 use Exception;
 use Exception;

-use Illuminate\Contracts\Container\BindingResolutionException;
-use Illuminate\Database\Eloquent\Builder;
-use Illuminate\Support\Collection;

 
 class ChapterRepo
 {
 
 class ChapterRepo
 {

-

     protected $baseRepo;
 
     /**
      * ChapterRepo constructor.
     protected $baseRepo;
 
     /**
      * ChapterRepo constructor.

-     * @param $baseRepo

      */
     public function __construct(BaseRepo $baseRepo)
     {
      */
     public function __construct(BaseRepo $baseRepo)
     {


@@ -28,6 +26,7 @@ class ChapterRepo
 
     /**
      * Get a chapter via the slug.
 
     /**
      * Get a chapter via the slug.

+     *

      * @throws NotFoundException
      */
     public function getBySlug(string $bookSlug, string $chapterSlug): Chapter
      * @throws NotFoundException
      */
     public function getBySlug(string $bookSlug, string $chapterSlug): Chapter


@@ -50,6 +49,8 @@ class ChapterRepo
         $chapter->book_id = $parentBook->id;
         $chapter->priority = (new BookContents($parentBook))->getLastPriority() + 1;
         $this->baseRepo->create($chapter, $input);
         $chapter->book_id = $parentBook->id;
         $chapter->priority = (new BookContents($parentBook))->getLastPriority() + 1;
         $this->baseRepo->create($chapter, $input);

+        Activity::addForEntity($chapter, ActivityType::CHAPTER_CREATE);
+

         return $chapter;
     }
 
         return $chapter;
     }
 


@@ -59,31 +60,29 @@ class ChapterRepo
     public function update(Chapter $chapter, array $input): Chapter
     {
         $this->baseRepo->update($chapter, $input);
     public function update(Chapter $chapter, array $input): Chapter
     {
         $this->baseRepo->update($chapter, $input);

-        return $chapter;
-    }
+        Activity::addForEntity($chapter, ActivityType::CHAPTER_UPDATE);

 
 

-    /**
-     * Update the permissions of a chapter.
-     */
-    public function updatePermissions(Chapter $chapter, bool $restricted, Collection $permissions = null)
-    {
-        $this->baseRepo->updatePermissions($chapter, $restricted, $permissions);
+        return $chapter;

     }
 
     /**
      * Remove a chapter from the system.
     }
 
     /**
      * Remove a chapter from the system.

+     *

      * @throws Exception
      */
     public function destroy(Chapter $chapter)
     {
         $trashCan = new TrashCan();
      * @throws Exception
      */
     public function destroy(Chapter $chapter)
     {
         $trashCan = new TrashCan();

-        $trashCan->destroyChapter($chapter);
+        $trashCan->softDestroyChapter($chapter);
+        Activity::addForEntity($chapter, ActivityType::CHAPTER_DELETE);
+        $trashCan->autoClearOld();

     }
 
     /**
      * Move the given chapter into a new parent book.
      * The $parentIdentifier must be a string of the following format:
     }
 
     /**
      * Move the given chapter into a new parent book.
      * The $parentIdentifier must be a string of the following format:

-     * 'book:<id>' (book:5)
+     * 'book:<id>' (book:5).
+     *

      * @throws MoveOperationException
      */
     public function move(Chapter $chapter, string $parentIdentifier): Book
      * @throws MoveOperationException
      */
     public function move(Chapter $chapter, string $parentIdentifier): Book


@@ -96,6 +95,7 @@ class ChapterRepo
             throw new MoveOperationException('Chapters can only be moved into books');
         }
 
             throw new MoveOperationException('Chapters can only be moved into books');
         }
 

+        /** @var Book $parent */

         $parent = Book::visible()->where('id', '=', $entityId)->first();
         if ($parent === null) {
             throw new MoveOperationException('Book to move chapter into not found');
         $parent = Book::visible()->where('id', '=', $entityId)->first();
         if ($parent === null) {
             throw new MoveOperationException('Book to move chapter into not found');


@@ -103,6 +103,8 @@ class ChapterRepo
 
         $chapter->changeBook($parent->id);
         $chapter->rebuildPermissions();
 
         $chapter->changeBook($parent->id);
         $chapter->rebuildPermissions();

+        Activity::addForEntity($chapter, ActivityType::CHAPTER_MOVE);
+

         return $parent;
     }
 }
         return $parent;
     }
 }