Merge branch 'master' into translations

[bookstack] / app / Repos / BookRepo.php

diff --git a/app/Repos/BookRepo.php b/app/Repos/BookRepo.php
index b0530b4f5ebe99e9fd017704293b7898f2c8a6e1..b14cf0dabc205e24979e911b69df89fc9a000f88 100644 (file)
--- a/app/Repos/BookRepo.php
+++ b/app/Repos/BookRepo.php
@@ -2,6 +2,7 @@
 
 use Alpha\B;
 use BookStack\Exceptions\NotFoundException;
 
 use Alpha\B;
 use BookStack\Exceptions\NotFoundException;

+use Illuminate\Database\Eloquent\Collection;

 use Illuminate\Support\Str;
 use BookStack\Book;
 use Views;
 use Illuminate\Support\Str;
 use BookStack\Book;
 use Views;


@@ -108,7 +109,7 @@ class BookRepo extends EntityRepo
     public function getBySlug($slug)
     {
         $book = $this->bookQuery()->where('slug', '=', $slug)->first();
     public function getBySlug($slug)
     {
         $book = $this->bookQuery()->where('slug', '=', $slug)->first();

-        if ($book === null) throw new NotFoundException('Book not found');
+        if ($book === null) throw new NotFoundException(trans('errors.book_not_found'));

         return $book;
     }
 
         return $book;
     }
 


@@ -131,8 +132,8 @@ class BookRepo extends EntityRepo
     {
         $book = $this->book->newInstance($input);
         $book->slug = $this->findSuitableSlug($book->name);
     {
         $book = $this->book->newInstance($input);
         $book->slug = $this->findSuitableSlug($book->name);

-        $book->created_by = auth()->user()->id;
-        $book->updated_by = auth()->user()->id;
+        $book->created_by = user()->id;
+        $book->updated_by = user()->id;

         $book->save();
         $this->permissionService->buildJointPermissionsForEntity($book);
         return $book;
         $book->save();
         $this->permissionService->buildJointPermissionsForEntity($book);
         return $book;


@@ -146,9 +147,11 @@ class BookRepo extends EntityRepo
      */
     public function updateFromInput(Book $book, $input)
     {
      */
     public function updateFromInput(Book $book, $input)
     {

+        if ($book->name !== $input['name']) {
+            $book->slug = $this->findSuitableSlug($input['name'], $book->id);
+        }

         $book->fill($input);
         $book->fill($input);

-        $book->slug = $this->findSuitableSlug($book->name, $book->id);
-        $book->updated_by = auth()->user()->id;
+        $book->updated_by = user()->id;

         $book->save();
         $this->permissionService->buildJointPermissionsForEntity($book);
         return $book;
         $book->save();
         $this->permissionService->buildJointPermissionsForEntity($book);
         return $book;


@@ -173,15 +176,6 @@ class BookRepo extends EntityRepo
         $book->delete();
     }
 
         $book->delete();
     }
 

-    /**
-     * Alias method to update the book jointPermissions in the PermissionService.
-     * @param Book $book
-     */
-    public function updateBookPermissions(Book $book)
-    {
-        $this->permissionService->buildJointPermissionsForEntity($book);
-    }
-

     /**
      * Get the next child element priority.
      * @param Book $book
     /**
      * Get the next child element priority.
      * @param Book $book


@@ -216,12 +210,9 @@ class BookRepo extends EntityRepo
      */
     public function findSuitableSlug($name, $currentId = false)
     {
      */
     public function findSuitableSlug($name, $currentId = false)
     {

-        $originalSlug = Str::slug($name);
-        $slug = $originalSlug;
-        $count = 2;
+        $slug = $this->nameToSlug($name);

         while ($this->doesSlugExist($slug, $currentId)) {
         while ($this->doesSlugExist($slug, $currentId)) {

-            $slug = $originalSlug . '-' . $count;
-            $count++;
+            $slug .= '-' . substr(md5(rand(1, 500)), 0, 3);

         }
         return $slug;
     }
         }
         return $slug;
     }


@@ -229,7 +220,7 @@ class BookRepo extends EntityRepo
     /**
      * Get all child objects of a book.
      * Returns a sorted collection of Pages and Chapters.
     /**
      * Get all child objects of a book.
      * Returns a sorted collection of Pages and Chapters.

-     * Loads the bookslug onto child elements to prevent access database access for getting the slug.
+     * Loads the book slug onto child elements to prevent access database access for getting the slug.

      * @param Book $book
      * @param bool $filterDrafts
      * @return mixed
      * @param Book $book
      * @param bool $filterDrafts
      * @return mixed


@@ -245,13 +236,16 @@ class BookRepo extends EntityRepo
 
         $pages = $pageQuery->get();
 
 
         $pages = $pageQuery->get();
 

-        $chapterQuery = $book->chapters()->with(['pages' => function($query) use ($filterDrafts) {
+        $chapterQuery = $book->chapters()->with(['pages' => function ($query) use ($filterDrafts) {

             $this->permissionService->enforcePageRestrictions($query, 'view');
             if ($filterDrafts) $query->where('draft', '=', false);
         }]);
         $chapterQuery = $this->permissionService->enforceChapterRestrictions($chapterQuery, 'view');
         $chapters = $chapterQuery->get();
             $this->permissionService->enforcePageRestrictions($query, 'view');
             if ($filterDrafts) $query->where('draft', '=', false);
         }]);
         $chapterQuery = $this->permissionService->enforceChapterRestrictions($chapterQuery, 'view');
         $chapters = $chapterQuery->get();

-        $children = $pages->merge($chapters);
+        $children = $pages->values();
+        foreach ($chapters as $chapter) {
+            $children->push($chapter);
+        }

         $bookSlug = $book->slug;
 
         $children->each(function ($child) use ($bookSlug) {
         $bookSlug = $book->slug;
 
         $children->each(function ($child) use ($bookSlug) {


@@ -260,7 +254,7 @@ class BookRepo extends EntityRepo
                 $child->pages->each(function ($page) use ($bookSlug) {
                     $page->setAttribute('bookSlug', $bookSlug);
                 });
                 $child->pages->each(function ($page) use ($bookSlug) {
                     $page->setAttribute('bookSlug', $bookSlug);
                 });

-                $child->pages = $child->pages->sortBy(function($child, $key) {
+                $child->pages = $child->pages->sortBy(function ($child, $key) {

                     $score = $child->priority;
                     if ($child->draft) $score -= 100;
                     return $score;
                     $score = $child->priority;
                     if ($child->draft) $score -= 100;
                     return $score;


@@ -269,7 +263,7 @@ class BookRepo extends EntityRepo
         });
 
         // Sort items with drafts first then by priority.
         });
 
         // Sort items with drafts first then by priority.

-        return $children->sortBy(function($child, $key) {
+        return $children->sortBy(function ($child, $key) {

             $score = $child->priority;
             if ($child->isA('page') && $child->draft) $score -= 100;
             return $score;
             $score = $child->priority;
             if ($child->isA('page') && $child->draft) $score -= 100;
             return $score;