itstructure / laravel-media-file-uploader

src/Http/Controllers/UploadController.php

<?php

namespace Itstructure\MFU\Http\Controllers;

use Throwable;
use Illuminate\Http\Request;
use Symfony\Component\HttpKernel\Exception\HttpException;
use Itstructure\MFU\Facades\{Uploader, Previewer};
use Itstructure\MFU\Models\Mediafile;

/**
 * Class UploadController
 * @package Itstructure\MFU\Http\Controllers
 */
class UploadController extends BaseController
{
    /**
     * @param Request $request
     * @return \Illuminate\Http\JsonResponse
     * @throws HttpException
     */
    public function upload(Request $request)
    {
        try {
            $data = $request->post('data');
            $file = $request->file('file');

            if (!Uploader::upload($data, $file)) {

The method upload() does not exist on Itstructure\MFU\Facades\Uploader. Since you implemented __callStatic, consider adding a @method annotation.

                return response()->json([
                    'success' => false,
                    'errors' => Uploader::hasErrors()

The method hasErrors() does not exist on Itstructure\MFU\Facades\Uploader. Since you implemented __callStatic, consider adding a @method annotation.

                        ? Uploader::getErrors()->getMessages()

The method getErrors() does not exist on Itstructure\MFU\Facades\Uploader. Since you implemented __callStatic, consider adding a @method annotation.

                        : []
                ]);
            }
            return response()->json([
                'success' => true,
                'id' => Uploader::getId()

The method getId() does not exist on Itstructure\MFU\Facades\Uploader. Since you implemented __callStatic, consider adding a @method annotation.

            ]);

        } catch (Throwable $exception) {
            return abort(400, $exception->getMessage());

Are you sure the usage of abort(400, $exception->getMessage()) is correct as it seems to always return null.

The expression return abort(400, $exception->getMessage()) returns the type void which is incompatible with the documented return type Illuminate\Http\JsonResponse.

        }
    }

    /**
     * @param Request $request
     * @return \Illuminate\Http\JsonResponse
     * @throws HttpException
     */
    public function update(Request $request)
    {
        try {
            $id = $request->post('id');
            $data = $request->post('data');
            $file = $request->hasFile('file') ? $request->file('file') : null;

            if (!Uploader::update($id, $data, $file)) {

The method update() does not exist on Itstructure\MFU\Facades\Uploader. Since you implemented __callStatic, consider adding a @method annotation.

                return response()->json([
                    'success' => false,
                    'errors' => Uploader::hasErrors()
                        ? Uploader::getErrors()->getMessages()
                        : []
                ]);
            }
            return response()->json([
                'success' => true
            ]);

        } catch (Throwable $exception) {
            return abort(400, $exception->getMessage());

Are you sure the usage of abort(400, $exception->getMessage()) is correct as it seems to always return null.

The expression return abort(400, $exception->getMessage()) returns the type void which is incompatible with the documented return type Illuminate\Http\JsonResponse.

        }
    }

    /**
     * @param Request $request
     * @return \Illuminate\Http\JsonResponse
     * @throws HttpException
     */
    public function delete(Request $request)
    {
        try {
            return response()->json([
                'success' => Uploader::delete($request->post('id'))

The method delete() does not exist on Itstructure\MFU\Facades\Uploader. Since you implemented __callStatic, consider adding a @method annotation.

            ]);

        } catch (Throwable $exception) {
            return abort(400, $exception->getMessage());

The expression return abort(400, $exception->getMessage()) returns the type void which is incompatible with the documented return type Illuminate\Http\JsonResponse.

Are you sure the usage of abort(400, $exception->getMessage()) is correct as it seems to always return null.

        }
    }

    /**
     * @param Request $request
     * @return string
     * @throws HttpException
     */
    public function preview(Request $request)
    {
        try {
            $id = $request->post('id');
            $location = $request->post('location') ?? \Itstructure\MFU\Services\Previewer::LOCATION_FILE_INFO;
            $mediaFile = Mediafile::find($id);
            return Previewer::getPreviewHtml($mediaFile, $location);

The method getPreviewHtml() does not exist on Itstructure\MFU\Facades\Previewer. Since you implemented __callStatic, consider adding a @method annotation.

        } catch (Throwable $exception) {
            return abort(400, $exception->getMessage());

The expression return abort(400, $exception->getMessage()) returns the type void which is incompatible with the documented return type string.

Are you sure the usage of abort(400, $exception->getMessage()) is correct as it seems to always return null.

        }
    }
}