Fixed issue where books titles could be leaked via shelf home view

- Also added test to cover
Fixes #1425

diff --git a/app/Http/Controllers/HomeController.php b/app/Http/Controllers/HomeController.php
index c5f3cd02a4bcee24e88eca0110dcc62c7825a41c..ba93bfe6517ddd7a29664aa0eee45976217d98e3 100644 (file)
--- a/app/Http/Controllers/HomeController.php
+++ b/app/Http/Controllers/HomeController.php
@@ -67,6 +67,9 @@ class HomeController extends Controller
 
         if ($homepageOption === 'bookshelves') {
             $shelves = $this->entityRepo->getAllPaginated('bookshelf', 18, $commonData['sort'], $commonData['order']);
+            foreach ($shelves as $shelf) {
+                $shelf->books = $this->entityRepo->getBookshelfChildren($shelf);
+            }
             $data = array_merge($commonData, ['shelves' => $shelves]);
             return view('common.home-shelves', $data);
         }

diff --git a/tests/HomepageTest.php b/tests/HomepageTest.php
index 2c8b8d5c032929e100b512c86ec91ccc544b0c50..286d4cf60aca6a3d06cb24d6d65be9580bac6b95 100644 (file)
--- a/tests/HomepageTest.php
+++ b/tests/HomepageTest.php
@@ -1,5 +1,7 @@
 <?php namespace Tests;
 
+use BookStack\Entities\Bookshelf;
+
 class HomepageTest extends TestCase
 {
 
@@ -89,4 +91,33 @@ class HomepageTest extends TestCase
         $this->setSettings(['app-homepage-type' => false]);
         $this->test_default_homepage_visible();
     }
+
+    public function test_shelves_list_homepage_adheres_to_book_visibility_permissions()
+    {
+        $editor = $this->getEditor();
+        setting()->putUser($editor, 'bookshelves_view_type', 'list');
+        $this->setSettings(['app-homepage-type' => 'bookshelves']);
+        $this->asEditor();
+
+        $shelf = Bookshelf::query()->first();
+        $book = $shelf->books()->first();
+
+        // Ensure initially visible
+        $homeVisit = $this->get('/');
+        $homeVisit->assertElementContains('.content-wrap', $shelf->name);
+        $homeVisit->assertElementContains('.content-wrap', $book->name);
+
+        // Ensure book no longer visible without view permission
+        $editor->roles()->detach();
+        $this->giveUserPermissions($editor, ['bookshelf-view-all']);
+        $homeVisit = $this->get('/');
+        $homeVisit->assertElementContains('.content-wrap', $shelf->name);
+        $homeVisit->assertElementNotContains('.content-wrap', $book->name);
+
+        // Ensure is visible again with entity-level view permission
+        $this->setEntityRestrictions($book, ['view'], [$editor->roles()->first()]);
+        $homeVisit = $this->get('/');
+        $homeVisit->assertElementContains('.content-wrap', $shelf->name);
+        $homeVisit->assertElementContains('.content-wrap', $book->name);
+    }
 }