Altered mysql start command in workflows

[bookstack] / app / Http / Controllers / Api / BookshelfApiController.php

diff --git a/app/Http/Controllers/Api/BookshelfApiController.php b/app/Http/Controllers/Api/BookshelfApiController.php
index e6379fdec88eddfda8251cdb950b5138bd0eac57..bd4f23a1093257cecd971a030d5c48109fc3a313 100644 (file)
--- a/app/Http/Controllers/Api/BookshelfApiController.php
+++ b/app/Http/Controllers/Api/BookshelfApiController.php
@@ -1,16 +1,16 @@
-<?php namespace BookStack\Http\Controllers\Api;
+<?php

 
 

-use BookStack\Facades\Activity;
+namespace BookStack\Http\Controllers\Api;
+
+use BookStack\Entities\Models\Bookshelf;

 use BookStack\Entities\Repos\BookshelfRepo;
 use BookStack\Entities\Repos\BookshelfRepo;

-use BookStack\Entities\Bookshelf;

 use Exception;
 use Exception;

+use Illuminate\Database\Eloquent\Relations\BelongsToMany;

 use Illuminate\Http\Request;
 use Illuminate\Validation\ValidationException;
 
 use Illuminate\Http\Request;
 use Illuminate\Validation\ValidationException;
 

-

 class BookshelfApiController extends ApiController
 {
 class BookshelfApiController extends ApiController
 {

-

     /**
      * @var BookshelfRepo
      */
     /**
      * @var BookshelfRepo
      */


@@ -18,18 +18,19 @@ class BookshelfApiController extends ApiController
 
     protected $rules = [
         'create' => [
 
     protected $rules = [
         'create' => [

-            'name' => 'required|string|max:255',
-            'description' => 'string|max:1000',
+            'name'        => ['required', 'string', 'max:255'],
+            'description' => ['string', 'max:1000'],
+            'books'       => ['array'],

         ],
         'update' => [
         ],
         'update' => [

-            'name' => 'string|min:1|max:255',
-            'description' => 'string|max:1000',
+            'name'        => ['string', 'min:1', 'max:255'],
+            'description' => ['string', 'max:1000'],
+            'books'       => ['array'],

         ],
     ];
 
     /**
      * BookshelfApiController constructor.
         ],
     ];
 
     /**
      * BookshelfApiController constructor.

-     * @param BookshelfRepo $bookshelfRepo

      */
     public function __construct(BookshelfRepo $bookshelfRepo)
     {
      */
     public function __construct(BookshelfRepo $bookshelfRepo)
     {


@@ -42,13 +43,17 @@ class BookshelfApiController extends ApiController
     public function list()
     {
         $shelves = Bookshelf::visible();
     public function list()
     {
         $shelves = Bookshelf::visible();

+

         return $this->apiListingResponse($shelves, [
         return $this->apiListingResponse($shelves, [

-            'id', 'name', 'slug', 'description', 'created_at', 'updated_at', 'created_by', 'updated_by', 'image_id',
+            'id', 'name', 'slug', 'description', 'created_at', 'updated_at', 'created_by', 'updated_by', 'owned_by', 'image_id',

         ]);
     }
 
     /**
      * Create a new shelf in the system.
         ]);
     }
 
     /**
      * Create a new shelf in the system.

+     * An array of books IDs can be provided in the request. These
+     * will be added to the shelf in the same order as provided.
+     *

      * @throws ValidationException
      */
     public function create(Request $request)
      * @throws ValidationException
      */
     public function create(Request $request)


@@ -57,9 +62,7 @@ class BookshelfApiController extends ApiController
         $requestData = $this->validate($request, $this->rules['create']);
 
         $bookIds = $request->get('books', []);
         $requestData = $this->validate($request, $this->rules['create']);
 
         $bookIds = $request->get('books', []);

-
-        $shelf = $this->bookshelfRepo->create($requestData,$bookIds);
-        Activity::add($shelf, 'bookshelf_create', $shelf->id);
+        $shelf = $this->bookshelfRepo->create($requestData, $bookIds);

 
         return response()->json($shelf);
     }
 
         return response()->json($shelf);
     }


@@ -69,12 +72,22 @@ class BookshelfApiController extends ApiController
      */
     public function read(string $id)
     {
      */
     public function read(string $id)
     {

-        $shelf = Bookshelf::visible()->with(['tags', 'cover', 'createdBy', 'updatedBy'])->findOrFail($id);
+        $shelf = Bookshelf::visible()->with([
+            'tags', 'cover', 'createdBy', 'updatedBy', 'ownedBy',
+            'books' => function (BelongsToMany $query) {
+                $query->scopes('visible')->get(['id', 'name', 'slug']);
+            },
+        ])->findOrFail($id);
+

         return response()->json($shelf);
     }
 
     /**
      * Update the details of a single shelf.
         return response()->json($shelf);
     }
 
     /**
      * Update the details of a single shelf.

+     * An array of books IDs can be provided in the request. These
+     * will be added to the shelf in the same order as provided and overwrite
+     * any existing book assignments.
+     *

      * @throws ValidationException
      */
     public function update(Request $request, string $id)
      * @throws ValidationException
      */
     public function update(Request $request, string $id)


@@ -83,21 +96,17 @@ class BookshelfApiController extends ApiController
         $this->checkOwnablePermission('bookshelf-update', $shelf);
 
         $requestData = $this->validate($request, $this->rules['update']);
         $this->checkOwnablePermission('bookshelf-update', $shelf);
 
         $requestData = $this->validate($request, $this->rules['update']);

+        $bookIds = $request->get('books', null);

 
 

-        $bookIds = $request->get('books', []);
-
-        $shelf = $this->bookshelfRepo->update($shelf, $requestData,$bookIds);
-        Activity::add($shelf, 'bookshelf_update', $shelf->id);
+        $shelf = $this->bookshelfRepo->update($shelf, $requestData, $bookIds);

 
         return response()->json($shelf);
     }
 
 
         return response()->json($shelf);
     }
 

-
-

     /**
     /**

-     * Delete a single shelf from the system.
-     * @param string $id
-     * @return \Illuminate\Contracts\Routing\ResponseFactory|\Illuminate\Http\Response
+     * Delete a single shelf.
+     * This will typically send the shelf to the recycle bin.
+     *

      * @throws Exception
      */
     public function delete(string $id)
      * @throws Exception
      */
     public function delete(string $id)


@@ -106,8 +115,7 @@ class BookshelfApiController extends ApiController
         $this->checkOwnablePermission('bookshelf-delete', $shelf);
 
         $this->bookshelfRepo->destroy($shelf);
         $this->checkOwnablePermission('bookshelf-delete', $shelf);
 
         $this->bookshelfRepo->destroy($shelf);

-        Activity::addMessage('bookshelf-delete', $shelf->name);

 
         return response('', 204);
     }
 
         return response('', 204);
     }

-}
\ No newline at end of file
+}