Arcanesoft\Media\Http\Controllers\MediaApiController

Complexity

Total Complexity

12

Size/Duplication

Total Lines	155
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

8 Methods

Rating	Name	Duplication	Size	Complexity
A	rename()	0	10	1
A	move()	0	12	1
A	download()	0	4	1
A	delete()	0	18	3
A	newFolder()	0	9	1
A	upload()	0	10	1
A	directories()	0	12	2
A	all()	0	11	2

<?php namespace Arcanesoft\Media\Http\Controllers;

use Arcanesoft\Media\Entities\MediaItem;
use Arcanesoft\Media\Http\Requests\NewFolderRequest;
use Arcanesoft\Media\Http\Requests\RenameMediaRequest;
use Arcanesoft\Media\Http\Requests\UploadMediaRequest;
use Illuminate\Http\Request;

/**
 * Class     MediaApiController
 *
 * @package  Arcanesoft\Media\Http\Controllers
 * @author   ARCANEDEV <arcanedev.maroc@gmail.com>
 */
class MediaApiController extends Controller
{
    /* -----------------------------------------------------------------
     |  Main Methods
     | -----------------------------------------------------------------
     */

    /**
     * @param  \Illuminate\Http\Request  $request
     *
     * @return \Illuminate\Http\JsonResponse
     */
    public function all(Request $request)
    {
        // TODO: Add authorization check

        $location = $request->get('location', '/');

        if ( ! $this->manager->exists($location))
            return static::jsonResponseError(['message' => 'Location not found'], 404);

        return static::jsonResponse(
            $this->manager->all($location)->values()->toArray()
        );
    }

    /**
     * Get all the directories based on the given location.
     *
     * @param  \Illuminate\Http\Request  $request
     *
     * @return \Illuminate\Http\JsonResponse
     */
    public function directories(Request $request)
    {
        // TODO: Add authorization check
        $location = $request->get('location', '/');

        if ($this->manager->exists($location)) {
            return static::jsonResponse(
                $this->manager->directories($location)->values()->toArray()
            );
        }

        return static::jsonResponseError(['message' => 'Location not found'], 404);
    }

    /**
     * Upload a media.
     *
     * @param  \Arcanesoft\Media\Http\Requests\UploadMediaRequest  $request
     *
     * @return \Illuminate\Http\JsonResponse
     */
    public function upload(UploadMediaRequest $request)
    {
        // TODO: Add authorization check

        $url = $this->manager->putFile(
            $request->get('location'),

It seems like $request->get('location') can also be of type null; however, parameter $path of Arcanesoft\Media\MediaManager::putFile() does only seem to accept string, maybe add an additional type check?

            $request->file('files')[0]
        );

        return static::jsonResponse(compact('url'));
    }

    /**
     * Create a new folder.
     *
     * @param  \Arcanesoft\Media\Http\Requests\NewFolderRequest  $request
     *
     * @return \Illuminate\Http\JsonResponse
     */
    public function newFolder(NewFolderRequest $request)
    {
        // TODO: Add authorization check

        $this->manager->makeDirectory(
            $path = $request->get('path')

It seems like $path = $request->get('path') can also be of type null; however, parameter $path of Arcanesoft\Media\MediaManager::makeDirectory() does only seem to accept string, maybe add an additional type check?

        );

        return static::jsonResponse(compact('path'));
    }

    /**
     * Move a media.
     *
     * @param  \Illuminate\Http\Request  $request
     *
     * @return \Illuminate\Http\JsonResponse
     */
    public function move(Request $request)
    {
        $from        = $request->get('path');
        $destination = $request->get('destination');

        $parts = explode('/', $from);
        $file  = array_pop($parts);
        $to    = implode('/', array_merge($parts, [$destination], [$file]));

        $this->manager->move($from, $to);

It seems like $from can also be of type null; however, parameter $from of Arcanesoft\Media\MediaManager::move() does only seem to accept string, maybe add an additional type check?

        return static::jsonResponse();
    }

    /**
     * Rename a media.
     *
     * @param  \Arcanesoft\Media\Http\Requests\RenameMediaRequest  $request
     *
     * @return \Illuminate\Http\JsonResponse
     */
    public function rename(RenameMediaRequest $request)
    {
        // TODO: Add authorization check

        $this->manager->rename(
            $request->get('old_path'),

It seems like $request->get('old_path') can also be of type null; however, parameter $oldName of Arcanesoft\Media\MediaManager::rename() does only seem to accept string, maybe add an additional type check?

            $request->get('new_path')

It seems like $request->get('new_path') can also be of type null; however, parameter $newName of Arcanesoft\Media\MediaManager::rename() does only seem to accept string, maybe add an additional type check?

        );

        return static::jsonResponse();
    }

    /**
     * Delete a media.
     *
     * @param  \Illuminate\Http\Request  $request
     *
     * @return \Illuminate\Http\JsonResponse
     */
    public function delete(Request $request)
    {
        // TODO: Add authorization check

        $type = $request->get('type');
        $path = $request->get('path');

        if ($type === MediaItem::TYPE_FILE){
            $this->manager->deleteFile($path);
        }
        elseif ($type === MediaItem::TYPE_DIRECTORY) {
            $this->manager->deleteDirectory($path);

It seems like $path can also be of type null; however, parameter $directory of Arcanesoft\Media\MediaManager::deleteDirectory() does only seem to accept string, maybe add an additional type check?

        }
        else {
            // TODO: Throw an exception ?
        }

        return static::jsonResponse();
    }

    public function download(Request $request)
    {
        return $this->manager->download(
            $request->get('path')

It seems like $request->get('path') can also be of type null; however, parameter $path of Arcanesoft\Media\MediaManager::download() does only seem to accept string, maybe add an additional type check?

        );
    }
}