Updated dev-docker setup to not alter phpunit.xml

[bookstack] / app / Http / Controllers / Images / ImageController.php

diff --git a/app/Http/Controllers/Images/ImageController.php b/app/Http/Controllers/Images/ImageController.php
index 024003f87d13c4a40611b7942ce83d3cbdcf9a43..ecc36bf67e24ad531f83326ed32d22bf4f97f63d 100644 (file)
--- a/app/Http/Controllers/Images/ImageController.php
+++ b/app/Http/Controllers/Images/ImageController.php
@@ -1,13 +1,13 @@
 <?php namespace BookStack\Http\Controllers\Images;
 
 <?php namespace BookStack\Http\Controllers\Images;
 

-use BookStack\Entities\Repos\EntityRepo;

 use BookStack\Exceptions\ImageUploadException;
 use BookStack\Http\Controllers\Controller;
 use BookStack\Exceptions\ImageUploadException;
 use BookStack\Http\Controllers\Controller;

-use BookStack\Repos\PageRepo;

 use BookStack\Uploads\Image;
 use BookStack\Uploads\ImageRepo;
 use BookStack\Uploads\Image;
 use BookStack\Uploads\ImageRepo;

+use Exception;

 use Illuminate\Filesystem\Filesystem as File;
 use Illuminate\Http\Request;
 use Illuminate\Filesystem\Filesystem as File;
 use Illuminate\Http\Request;

+use Illuminate\Validation\ValidationException;

 
 class ImageController extends Controller
 {
 
 class ImageController extends Controller
 {


@@ -17,22 +17,16 @@ class ImageController extends Controller
 
     /**
      * ImageController constructor.
 
     /**
      * ImageController constructor.

-     * @param Image $image
-     * @param File $file
-     * @param ImageRepo $imageRepo

      */
     public function __construct(Image $image, File $file, ImageRepo $imageRepo)
     {
         $this->image = $image;
         $this->file = $file;
         $this->imageRepo = $imageRepo;
      */
     public function __construct(Image $image, File $file, ImageRepo $imageRepo)
     {
         $this->image = $image;
         $this->file = $file;
         $this->imageRepo = $imageRepo;

-        parent::__construct();

     }
 
     /**
      * Provide an image file from storage.
     }
 
     /**
      * Provide an image file from storage.

-     * @param string $path
-     * @return mixed

      */
     public function showImage(string $path)
     {
      */
     public function showImage(string $path)
     {


@@ -47,13 +41,10 @@ class ImageController extends Controller
 
     /**
      * Update image details
 
     /**
      * Update image details

-     * @param integer $id
-     * @param Request $request
-     * @return \Illuminate\Http\JsonResponse

      * @throws ImageUploadException
      * @throws ImageUploadException

-     * @throws \Exception
+     * @throws ValidationException

      */
      */

-    public function update($id, Request $request)
+    public function update(Request $request, string $id)

     {
         $this->validate($request, [
             'name' => 'required|min:2|string'
     {
         $this->validate($request, [
             'name' => 'required|min:2|string'


@@ -64,42 +55,50 @@ class ImageController extends Controller
         $this->checkOwnablePermission('image-update', $image);
 
         $image = $this->imageRepo->updateImageDetails($image, $request->all());
         $this->checkOwnablePermission('image-update', $image);
 
         $image = $this->imageRepo->updateImageDetails($image, $request->all());

-        return response()->json($image);
+
+        $this->imageRepo->loadThumbs($image);
+        return view('components.image-manager-form', [
+            'image' => $image,
+            'dependantPages' => null,
+        ]);

     }
 
     /**
     }
 
     /**

-     * Show the usage of an image on pages.
-     * @param \BookStack\Entities\Repos\EntityRepo $entityRepo
-     * @param $id
-     * @return \Illuminate\Http\JsonResponse
+     * Get the form for editing the given image.
+     * @throws Exception

      */
      */

-    public function usage(EntityRepo $entityRepo, $id)
+    public function edit(Request $request, string $id)

     {
         $image = $this->imageRepo->getById($id);
         $this->checkImagePermission($image);
     {
         $image = $this->imageRepo->getById($id);
         $this->checkImagePermission($image);

-        $pageSearch = $entityRepo->searchForImage($image->url);
-        return response()->json($pageSearch);
+
+        if ($request->has('delete')) {
+            $dependantPages = $this->imageRepo->getPagesUsingImage($image);
+        }
+
+        $this->imageRepo->loadThumbs($image);
+        return view('components.image-manager-form', [
+            'image' => $image,
+            'dependantPages' => $dependantPages ?? null,
+        ]);

     }
 
     /**
      * Deletes an image and all thumbnail/image files
     }
 
     /**
      * Deletes an image and all thumbnail/image files

-     * @param int $id
-     * @return \Illuminate\Http\JsonResponse
-     * @throws \Exception
+     * @throws Exception

      */
      */

-    public function destroy($id)
+    public function destroy(string $id)

     {
         $image = $this->imageRepo->getById($id);
         $this->checkOwnablePermission('image-delete', $image);
         $this->checkImagePermission($image);
 
         $this->imageRepo->destroyImage($image);
     {
         $image = $this->imageRepo->getById($id);
         $this->checkOwnablePermission('image-delete', $image);
         $this->checkImagePermission($image);
 
         $this->imageRepo->destroyImage($image);

-        return response()->json(trans('components.images_deleted'));
+        return response('');

     }
 
     /**
      * Check related page permission and ensure type is drawio or gallery.
     }
 
     /**
      * Check related page permission and ensure type is drawio or gallery.

-     * @param Image $image

      */
     protected function checkImagePermission(Image $image)
     {
      */
     protected function checkImagePermission(Image $image)
     {