Fixed OIDC Logout

[bookstack] / app / Entities / Models / Chapter.php

diff --git a/app/Entities/Models/Chapter.php b/app/Entities/Models/Chapter.php
index 4514bef036177faa24a6ed422c5bc30ee9a4df25..98889ce3f38a430c15d281d64b20b05491fc9bb9 100644 (file)
--- a/app/Entities/Models/Chapter.php
+++ b/app/Entities/Models/Chapter.php
@@ -1,65 +1,50 @@
-<?php namespace BookStack\Entities\Models;
+<?php

 
 

-use BookStack\Entities\Models\BookChild;
-use BookStack\Entities\Models\Page;
+namespace BookStack\Entities\Models;
+
+use Illuminate\Database\Eloquent\Factories\HasFactory;
+use Illuminate\Database\Eloquent\Relations\HasMany;

 use Illuminate\Support\Collection;
 
 /**
 use Illuminate\Support\Collection;
 
 /**

- * Class Chapter
+ * Class Chapter.
+ *

  * @property Collection<Page> $pages
  * @property Collection<Page> $pages

+ * @property string           $description

  */
 class Chapter extends BookChild
 {
  */
 class Chapter extends BookChild
 {

-    public $searchFactor = 1.3;
+    use HasFactory;
+
+    public $searchFactor = 1.2;

 
 

-    protected $fillable = ['name', 'description', 'priority', 'book_id'];
-    protected $hidden = ['restricted', 'pivot'];
+    protected $fillable = ['name', 'description', 'priority'];
+    protected $hidden = ['pivot', 'deleted_at'];

 
     /**
      * Get the pages that this chapter contains.
 
     /**
      * Get the pages that this chapter contains.

-     * @param string $dir
-     * @return mixed
+     *
+     * @return HasMany<Page>

      */
      */

-    public function pages($dir = 'ASC')
+    public function pages(string $dir = 'ASC'): HasMany

     {
         return $this->hasMany(Page::class)->orderBy('priority', $dir);
     }
 
     /**
      * Get the url of this chapter.
     {
         return $this->hasMany(Page::class)->orderBy('priority', $dir);
     }
 
     /**
      * Get the url of this chapter.

-     * @param string|bool $path
-     * @return string
-     */
-    public function getUrl($path = false)
-    {
-        $bookSlug = $this->getAttribute('bookSlug') ? $this->getAttribute('bookSlug') : $this->book->slug;
-        $fullPath = '/books/' . urlencode($bookSlug) . '/chapter/' . urlencode($this->slug);
-
-        if ($path !== false) {
-            $fullPath .= '/' . trim($path, '/');
-        }
-
-        return url($fullPath);
-    }
-
-    /**
-     * Get an excerpt of this chapter's description to the specified length or less.
-     * @param int $length
-     * @return string

      */
      */

-    public function getExcerpt(int $length = 100)
+    public function getUrl(string $path = ''): string

     {
     {

-        $description = $this->text ?? $this->description;
-        return mb_strlen($description) > $length ? mb_substr($description, 0, $length-3) . '...' : $description;
-    }
+        $parts = [
+            'books',
+            urlencode($this->book_slug ?? $this->book->slug),
+            'chapter',
+            urlencode($this->slug),
+            trim($path, '/'),
+        ];

 
 

-    /**
-     * Check if this chapter has any child pages.
-     * @return bool
-     */
-    public function hasChildren()
-    {
-        return count($this->pages) > 0;
+        return url('/' . implode('/', $parts));

     }
 
     /**
     }
 
     /**


@@ -67,9 +52,19 @@ class Chapter extends BookChild
      */
     public function getVisiblePages(): Collection
     {
      */
     public function getVisiblePages(): Collection
     {

-        return $this->pages()->visible()
+        return $this->pages()
+        ->scopes('visible')

         ->orderBy('draft', 'desc')
         ->orderBy('priority', 'asc')
         ->get();
     }
         ->orderBy('draft', 'desc')
         ->orderBy('priority', 'asc')
         ->get();
     }

+
+    /**
+     * Get a visible chapter by its book and page slugs.
+     * @throws \Illuminate\Database\Eloquent\ModelNotFoundException
+     */
+    public static function getBySlugs(string $bookSlug, string $chapterSlug): self
+    {
+        return static::visible()->whereSlugs($bookSlug, $chapterSlug)->firstOrFail();
+    }

 }
 }