FileUploadController

Complexity

Total Complexity

10

Size/Duplication

Total Lines	62
Duplicated Lines	0 %

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	3	1
B	validateAndUpload()	0	41	6
A	uploadFiles()	0	10	3

<?php
use LaravelRealtimeChat\Repositories\Team\TeamRepository;
use LaravelRealtimeChat\Repositories\Task\TaskRepository;
class FileUploadController extends AuthController {
  private $taskRepository;

        public function __construct(TeamRepository $teamRepository, TaskRepository $taskRepository) {
                $this->teamRepository = $teamRepository;

The property teamRepository is declared private in AuthController and cannot be accessed from this context.

                $this->taskRepository = $taskRepository;
        }
        public function uploadFiles($name) {

                switch ($name) {
                        case "chat":$folderName = "userUploads";
                                $this->validateAndUpload($folderName,false);//set true for db operations,else false
                                break;

                        case "task":$folderName = "userTaskFiles";
                                $this->validateAndUpload($folderName,true);
                                break;
                }
        }


 function validateAndUpload($folderName,$dbSave) {

It is generally recommended to explicitly declare the visibility for methods.

        $successStatus;

The variable $successStatus seems to be never defined.

        $fileErr;

The variable $fileErr seems to be never defined.

        // Build the input for our validation
        $input      = Input::all();

The type Input was not found. Maybe you did not declare it correctly or list all dependencies?

        // Within the ruleset, make sure we let the validator know that this
        // file should be an image
        $rules      = array(
             'file' => 'max:100000000',
        );
        $validation = Validator::make($input, $rules);

        if ($validation->fails()) {
                return Response::make($validation->errors->first(), 400);
        }

        $file = Input::file('file');
        foreach ($file as $files) {

                $upload_success = $files->move('fusionmate/public/plugins/' . $folderName . '', $files->getClientOriginalName());

                $imgPath = $files->getClientOriginalName();

                if ($upload_success) {
                        if($dbSave){
                             $this->taskRepository->saveTaskFiles($imgPath);   
                        }
                        $fileErr = false;
                }
                else {
                        $fileErr = true;
                }
        }
        if (!$fileErr) {

The variable $fileErr seems to be defined by a foreach iteration on line 42. Are you sure the iterator is never empty, otherwise this variable is not defined?

                $successStatus = Response::json('success', 200);
        }
        else {
                $successStatus = Response::json('error', 400);
        }
        return $successStatus;
}
}